{"action":"created","comment":{"url":"https://api.github.com/repos/ppy/osu/pulls/comments/92943232","pull_request_review_id":13465869,"id":92943232,"diff_hunk":"@@ -58,6 +61,11 @@ public void UpdateBeatmap(WorkingBeatmap beatmap)\n             BeatmapSetInfo beatmapSetInfo = beatmap.BeatmapSetInfo;\r\n             BeatmapInfo beatmapInfo = beatmap.BeatmapInfo;\r\n \r\n+            string bpm = GetBPMRange(beatmap);\r\n+            string length = \"\" + TimeSpan.FromMilliseconds((beatmap.Beatmap.HitObjects.Last().EndTime - beatmap.Beatmap.HitObjects.First().StartTime)).ToString(@\"m\\:ss\");\r\n+            string hitCircles = \"\" + beatmap.Beatmap.HitObjects.Count(b => b.GetType().ToString().Equals(\"osu.Game.Modes.Osu.Objects.HitCircle\"));\r","path":"osu.Game/Screens/Select/BeatmapInfoWedge.cs","position":16,"original_position":16,"commit_id":"492581fa50b9725da134f8f071e70293f1fbf78f","original_commit_id":"492581fa50b9725da134f8f071e70293f1fbf78f","user":{"login":"Tom94","id":4923655,"avatar_url":"https://avatars.githubusercontent.com/u/4923655?v=3","gravatar_id":"","url":"https://api.github.com/users/Tom94","html_url":"https://github.com/Tom94","followers_url":"https://api.github.com/users/Tom94/followers","following_url":"https://api.github.com/users/Tom94/following{/other_user}","gists_url":"https://api.github.com/users/Tom94/gists{/gist_id}","starred_url":"https://api.github.com/users/Tom94/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Tom94/subscriptions","organizations_url":"https://api.github.com/users/Tom94/orgs","repos_url":"https://api.github.com/users/Tom94/repos","events_url":"https://api.github.com/users/Tom94/events{/privacy}","received_events_url":"https://api.github.com/users/Tom94/received_events","type":"User","site_admin":false},"body":"Why are you naming hitobjects `b`? `h` would fit way better. Also use `h is HitCircle` instead of this convoluted string comparison. Same for the other places in the code which are similar.","created_at":"2016-12-18T15:05:58Z","updated_at":"2016-12-18T15:05:58Z","html_url":"https://github.com/ppy/osu/pull/250#discussion_r92943232","pull_request_url":"https://api.github.com/repos/ppy/osu/pulls/250","_links":{"self":{"href":"https://api.github.com/repos/ppy/osu/pulls/comments/92943232"},"html":{"href":"https://github.com/ppy/osu/pull/250#discussion_r92943232"},"pull_request":{"href":"https://api.github.com/repos/ppy/osu/pulls/250"}}},"pull_request":{"url":"https://api.github.com/repos/ppy/osu/pulls/250","id":98273633,"html_url":"https://github.com/ppy/osu/pull/250","diff_url":"https://github.com/ppy/osu/pull/250.diff","patch_url":"https://github.com/ppy/osu/pull/250.patch","issue_url":"https://api.github.com/repos/ppy/osu/issues/250","number":250,"state":"open","locked":false,"title":"more beatmap info in song select","user":{"login":"n4b3l","id":24597250,"avatar_url":"https://avatars.githubusercontent.com/u/24597250?v=3","gravatar_id":"","url":"https://api.github.com/users/n4b3l","html_url":"https://github.com/n4b3l","followers_url":"https://api.github.com/users/n4b3l/followers","following_url":"https://api.github.com/users/n4b3l/following{/other_user}","gists_url":"https://api.github.com/users/n4b3l/gists{/gist_id}","starred_url":"https://api.github.com/users/n4b3l/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/n4b3l/subscriptions","organizations_url":"https://api.github.com/users/n4b3l/orgs","repos_url":"https://api.github.com/users/n4b3l/repos","events_url":"https://api.github.com/users/n4b3l/events{/privacy}","received_events_url":"https://api.github.com/users/n4b3l/received_events","type":"User","site_admin":false},"body":"","created_at":"2016-12-16T03:34:55Z","updated_at":"2016-12-18T15:05:58Z","closed_at":null,"merged_at":null,"merge_commit_sha":"f318a46675d08d261f2600c92e1aa24bc6af043b","assignee":null,"assignees":[],"milestone":null,"commits_url":"https://api.github.com/repos/ppy/osu/pulls/250/commits","review_comments_url":"https://api.github.com/repos/ppy/osu/pulls/250/comments","review_comment_url":"https://api.github.com/repos/ppy/osu/pulls/comments{/number}","comments_url":"https://api.github.com/repos/ppy/osu/issues/250/comments","statuses_url":"https://api.github.com/repos/ppy/osu/statuses/492581fa50b9725da134f8f071e70293f1fbf78f","head":{"label":"n4b3l:more-beatmap-info","ref":"more-beatmap-info","sha":"492581fa50b9725da134f8f071e70293f1fbf78f","user":{"login":"n4b3l","id":24597250,"avatar_url":"https://avatars.githubusercontent.com/u/24597250?v=3","gravatar_id":"","url":"https://api.github.com/users/n4b3l","html_url":"https://github.com/n4b3l","followers_url":"https://api.github.com/users/n4b3l/followers","following_url":"https://api.github.com/users/n4b3l/following{/other_user}","gists_url":"https://api.github.com/users/n4b3l/gists{/gist_id}","starred_url":"https://api.github.com/users/n4b3l/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/n4b3l/subscriptions","organizations_url":"https://api.github.com/users/n4b3l/orgs","repos_url":"https://api.github.com/users/n4b3l/repos","events_url":"https://api.github.com/users/n4b3l/events{/privacy}","received_events_url":"https://api.github.com/users/n4b3l/received_events","type":"User","site_admin":false},"repo":{"id":76617367,"name":"osu","full_name":"n4b3l/osu","owner":{"login":"n4b3l","id":24597250,"avatar_url":"https://avatars.githubusercontent.com/u/24597250?v=3","gravatar_id":"","url":"https://api.github.com/users/n4b3l","html_url":"https://github.com/n4b3l","followers_url":"https://api.github.com/users/n4b3l/followers","following_url":"https://api.github.com/users/n4b3l/following{/other_user}","gists_url":"https://api.github.com/users/n4b3l/gists{/gist_id}","starred_url":"https://api.github.com/users/n4b3l/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/n4b3l/subscriptions","organizations_url":"https://api.github.com/users/n4b3l/orgs","repos_url":"https://api.github.com/users/n4b3l/repos","events_url":"https://api.github.com/users/n4b3l/events{/privacy}","received_events_url":"https://api.github.com/users/n4b3l/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/n4b3l/osu","description":"rhythm is just a *click* away!","fork":true,"url":"https://api.github.com/repos/n4b3l/osu","forks_url":"https://api.github.com/repos/n4b3l/osu/forks","keys_url":"https://api.github.com/repos/n4b3l/osu/keys{/key_id}","collaborators_url":"https://api.github.com/repos/n4b3l/osu/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/n4b3l/osu/teams","hooks_url":"https://api.github.com/repos/n4b3l/osu/hooks","issue_events_url":"https://api.github.com/repos/n4b3l/osu/issues/events{/number}","events_url":"https://api.github.com/repos/n4b3l/osu/events","assignees_url":"https://api.github.com/repos/n4b3l/osu/assignees{/user}","branches_url":"https://api.github.com/repos/n4b3l/osu/branches{/branch}","tags_url":"https://api.github.com/repos/n4b3l/osu/tags","blobs_url":"https://api.github.com/repos/n4b3l/osu/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/n4b3l/osu/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/n4b3l/osu/git/refs{/sha}","trees_url":"https://api.github.com/repos/n4b3l/osu/git/trees{/sha}","statuses_url":"https://api.github.com/repos/n4b3l/osu/statuses/{sha}","languages_url":"https://api.github.com/repos/n4b3l/osu/languages","stargazers_url":"https://api.github.com/repos/n4b3l/osu/stargazers","contributors_url":"https://api.github.com/repos/n4b3l/osu/contributors","subscribers_url":"https://api.github.com/repos/n4b3l/osu/subscribers","subscription_url":"https://api.github.com/repos/n4b3l/osu/subscription","commits_url":"https://api.github.com/repos/n4b3l/osu/commits{/sha}","git_commits_url":"https://api.github.com/repos/n4b3l/osu/git/commits{/sha}","comments_url":"https://api.github.com/repos/n4b3l/osu/comments{/number}","issue_comment_url":"https://api.github.com/repos/n4b3l/osu/issues/comments{/number}","contents_url":"https://api.github.com/repos/n4b3l/osu/contents/{+path}","compare_url":"https://api.github.com/repos/n4b3l/osu/compare/{base}...{head}","merges_url":"https://api.github.com/repos/n4b3l/osu/merges","archive_url":"https://api.github.com/repos/n4b3l/osu/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/n4b3l/osu/downloads","issues_url":"https://api.github.com/repos/n4b3l/osu/issues{/number}","pulls_url":"https://api.github.com/repos/n4b3l/osu/pulls{/number}","milestones_url":"https://api.github.com/repos/n4b3l/osu/milestones{/number}","notifications_url":"https://api.github.com/repos/n4b3l/osu/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/n4b3l/osu/labels{/name}","releases_url":"https://api.github.com/repos/n4b3l/osu/releases{/id}","deployments_url":"https://api.github.com/repos/n4b3l/osu/deployments","created_at":"2016-12-16T03:20:30Z","updated_at":"2016-12-16T03:20:31Z","pushed_at":"2016-12-18T10:49:46Z","git_url":"git://github.com/n4b3l/osu.git","ssh_url":"git@github.com:n4b3l/osu.git","clone_url":"https://github.com/n4b3l/osu.git","svn_url":"https://github.com/n4b3l/osu","homepage":"https://osu.ppy.sh","size":1628,"stargazers_count":0,"watchers_count":0,"language":"C#","has_issues":false,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"ppy:master","ref":"master","sha":"d39da80b1b369e41121adb35918cac4ba970e12a","user":{"login":"ppy","id":995763,"avatar_url":"https://avatars.githubusercontent.com/u/995763?v=3","gravatar_id":"","url":"https://api.github.com/users/ppy","html_url":"https://github.com/ppy","followers_url":"https://api.github.com/users/ppy/followers","following_url":"https://api.github.com/users/ppy/following{/other_user}","gists_url":"https://api.github.com/users/ppy/gists{/gist_id}","starred_url":"https://api.github.com/users/ppy/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ppy/subscriptions","organizations_url":"https://api.github.com/users/ppy/orgs","repos_url":"https://api.github.com/users/ppy/repos","events_url":"https://api.github.com/users/ppy/events{/privacy}","received_events_url":"https://api.github.com/users/ppy/received_events","type":"Organization","site_admin":false},"repo":{"id":22456440,"name":"osu","full_name":"ppy/osu","owner":{"login":"ppy","id":995763,"avatar_url":"https://avatars.githubusercontent.com/u/995763?v=3","gravatar_id":"","url":"https://api.github.com/users/ppy","html_url":"https://github.com/ppy","followers_url":"https://api.github.com/users/ppy/followers","following_url":"https://api.github.com/users/ppy/following{/other_user}","gists_url":"https://api.github.com/users/ppy/gists{/gist_id}","starred_url":"https://api.github.com/users/ppy/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ppy/subscriptions","organizations_url":"https://api.github.com/users/ppy/orgs","repos_url":"https://api.github.com/users/ppy/repos","events_url":"https://api.github.com/users/ppy/events{/privacy}","received_events_url":"https://api.github.com/users/ppy/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/ppy/osu","description":"rhythm is just a *click* away!","fork":false,"url":"https://api.github.com/repos/ppy/osu","forks_url":"https://api.github.com/repos/ppy/osu/forks","keys_url":"https://api.github.com/repos/ppy/osu/keys{/key_id}","collaborators_url":"https://api.github.com/repos/ppy/osu/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/ppy/osu/teams","hooks_url":"https://api.github.com/repos/ppy/osu/hooks","issue_events_url":"https://api.github.com/repos/ppy/osu/issues/events{/number}","events_url":"https://api.github.com/repos/ppy/osu/events","assignees_url":"https://api.github.com/repos/ppy/osu/assignees{/user}","branches_url":"https://api.github.com/repos/ppy/osu/branches{/branch}","tags_url":"https://api.github.com/repos/ppy/osu/tags","blobs_url":"https://api.github.com/repos/ppy/osu/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/ppy/osu/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/ppy/osu/git/refs{/sha}","trees_url":"https://api.github.com/repos/ppy/osu/git/trees{/sha}","statuses_url":"https://api.github.com/repos/ppy/osu/statuses/{sha}","languages_url":"https://api.github.com/repos/ppy/osu/languages","stargazers_url":"https://api.github.com/repos/ppy/osu/stargazers","contributors_url":"https://api.github.com/repos/ppy/osu/contributors","subscribers_url":"https://api.github.com/repos/ppy/osu/subscribers","subscription_url":"https://api.github.com/repos/ppy/osu/subscription","commits_url":"https://api.github.com/repos/ppy/osu/commits{/sha}","git_commits_url":"https://api.github.com/repos/ppy/osu/git/commits{/sha}","comments_url":"https://api.github.com/repos/ppy/osu/comments{/number}","issue_comment_url":"https://api.github.com/repos/ppy/osu/issues/comments{/number}","contents_url":"https://api.github.com/repos/ppy/osu/contents/{+path}","compare_url":"https://api.github.com/repos/ppy/osu/compare/{base}...{head}","merges_url":"https://api.github.com/repos/ppy/osu/merges","archive_url":"https://api.github.com/repos/ppy/osu/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/ppy/osu/downloads","issues_url":"https://api.github.com/repos/ppy/osu/issues{/number}","pulls_url":"https://api.github.com/repos/ppy/osu/pulls{/number}","milestones_url":"https://api.github.com/repos/ppy/osu/milestones{/number}","notifications_url":"https://api.github.com/repos/ppy/osu/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/ppy/osu/labels{/name}","releases_url":"https://api.github.com/repos/ppy/osu/releases{/id}","deployments_url":"https://api.github.com/repos/ppy/osu/deployments","created_at":"2014-07-31T04:26:30Z","updated_at":"2016-12-18T07:11:58Z","pushed_at":"2016-12-18T15:04:11Z","git_url":"git://github.com/ppy/osu.git","ssh_url":"git@github.com:ppy/osu.git","clone_url":"https://github.com/ppy/osu.git","svn_url":"https://github.com/ppy/osu","homepage":"https://osu.ppy.sh","size":1596,"stargazers_count":570,"watchers_count":570,"language":"C#","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":123,"mirror_url":null,"open_issues_count":26,"forks":123,"open_issues":26,"watchers":570,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/ppy/osu/pulls/250"},"html":{"href":"https://github.com/ppy/osu/pull/250"},"issue":{"href":"https://api.github.com/repos/ppy/osu/issues/250"},"comments":{"href":"https://api.github.com/repos/ppy/osu/issues/250/comments"},"review_comments":{"href":"https://api.github.com/repos/ppy/osu/pulls/250/comments"},"review_comment":{"href":"https://api.github.com/repos/ppy/osu/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/ppy/osu/pulls/250/commits"},"statuses":{"href":"https://api.github.com/repos/ppy/osu/statuses/492581fa50b9725da134f8f071e70293f1fbf78f"}}}}
{"actor":{"display_login":"Tom94"}}