{"action":"created","comment":{"url":"https://api.github.com/repos/EionRobb/skype4pidgin/pulls/comments/92610887","pull_request_review_id":13129542,"id":92610887,"diff_hunk":"@@ -1437,20 +1437,24 @@ skypeweb_buddy_unblock(PurpleConnection *pc, const char *name)\n void\r\n skypeweb_set_mood_message(SkypeWebAccount *sa, const gchar *mood)\r\n {\r\n-\tJsonObject *obj, *payload;\r\n-\tgchar *post;\r\n-\t\r\n-\tg_return_if_fail(mood);\r\n-\t\r\n-\tobj = json_object_new();\r\n-\tpayload = json_object_new();\r\n-\t\r\n-\tjson_object_set_string_member(payload, \"mood\", mood);\r\n-\tjson_object_set_object_member(obj, \"payload\", payload);\r\n-\tpost = skypeweb_jsonobj_to_string(obj);\r\n-\t\r\n-\tskypeweb_post_or_get(sa, SKYPEWEB_METHOD_POST | SKYPEWEB_METHOD_SSL, SKYPEWEB_CONTACTS_HOST, \"/users/self/profile/partial\", post, NULL, NULL, TRUE);\r\n-\t\r\n-\tg_free(post);\r\n-\tjson_object_unref(obj);\r\n+\tif (mood != NULL) {\r","path":"skypeweb/skypeweb_contacts.c","position":29,"original_position":29,"commit_id":"dc4c9e7cf85d2dd9ae89835e25e2718d3fa68ad4","original_commit_id":"dc4c9e7cf85d2dd9ae89835e25e2718d3fa68ad4","user":{"login":"dequis","id":94108,"avatar_url":"https://avatars.githubusercontent.com/u/94108?v=3","gravatar_id":"","url":"https://api.github.com/users/dequis","html_url":"https://github.com/dequis","followers_url":"https://api.github.com/users/dequis/followers","following_url":"https://api.github.com/users/dequis/following{/other_user}","gists_url":"https://api.github.com/users/dequis/gists{/gist_id}","starred_url":"https://api.github.com/users/dequis/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dequis/subscriptions","organizations_url":"https://api.github.com/users/dequis/orgs","repos_url":"https://api.github.com/users/dequis/repos","events_url":"https://api.github.com/users/dequis/events{/privacy}","received_events_url":"https://api.github.com/users/dequis/received_events","type":"User","site_admin":false},"body":"You could flip this condition and return immediately instead of indenting the whole function","created_at":"2016-12-15T13:45:41Z","updated_at":"2016-12-15T13:45:41Z","html_url":"https://github.com/EionRobb/skype4pidgin/pull/510#discussion_r92610887","pull_request_url":"https://api.github.com/repos/EionRobb/skype4pidgin/pulls/510","_links":{"self":{"href":"https://api.github.com/repos/EionRobb/skype4pidgin/pulls/comments/92610887"},"html":{"href":"https://github.com/EionRobb/skype4pidgin/pull/510#discussion_r92610887"},"pull_request":{"href":"https://api.github.com/repos/EionRobb/skype4pidgin/pulls/510"}}},"pull_request":{"url":"https://api.github.com/repos/EionRobb/skype4pidgin/pulls/510","id":98109931,"html_url":"https://github.com/EionRobb/skype4pidgin/pull/510","diff_url":"https://github.com/EionRobb/skype4pidgin/pull/510.diff","patch_url":"https://github.com/EionRobb/skype4pidgin/pull/510.patch","issue_url":"https://api.github.com/repos/EionRobb/skype4pidgin/issues/510","number":510,"state":"open","locked":false,"title":"Bug fix when load a chat ","user":{"login":"intika","id":6892180,"avatar_url":"https://avatars.githubusercontent.com/u/6892180?v=3","gravatar_id":"","url":"https://api.github.com/users/intika","html_url":"https://github.com/intika","followers_url":"https://api.github.com/users/intika/followers","following_url":"https://api.github.com/users/intika/following{/other_user}","gists_url":"https://api.github.com/users/intika/gists{/gist_id}","starred_url":"https://api.github.com/users/intika/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/intika/subscriptions","organizations_url":"https://api.github.com/users/intika/orgs","repos_url":"https://api.github.com/users/intika/repos","events_url":"https://api.github.com/users/intika/events{/privacy}","received_events_url":"https://api.github.com/users/intika/received_events","type":"User","site_admin":false},"body":"Fix this :\r\nFatal error when loading a chat \r\ng_log: skypeweb_set_mood_message: assertion `mood' failed","created_at":"2016-12-15T09:00:40Z","updated_at":"2016-12-15T13:45:41Z","closed_at":null,"merged_at":null,"merge_commit_sha":"af64815acf5d0d51496fba91d0d3609a6648d204","assignee":null,"assignees":[],"milestone":null,"commits_url":"https://api.github.com/repos/EionRobb/skype4pidgin/pulls/510/commits","review_comments_url":"https://api.github.com/repos/EionRobb/skype4pidgin/pulls/510/comments","review_comment_url":"https://api.github.com/repos/EionRobb/skype4pidgin/pulls/comments{/number}","comments_url":"https://api.github.com/repos/EionRobb/skype4pidgin/issues/510/comments","statuses_url":"https://api.github.com/repos/EionRobb/skype4pidgin/statuses/dc4c9e7cf85d2dd9ae89835e25e2718d3fa68ad4","head":{"label":"intika:master","ref":"master","sha":"dc4c9e7cf85d2dd9ae89835e25e2718d3fa68ad4","user":{"login":"intika","id":6892180,"avatar_url":"https://avatars.githubusercontent.com/u/6892180?v=3","gravatar_id":"","url":"https://api.github.com/users/intika","html_url":"https://github.com/intika","followers_url":"https://api.github.com/users/intika/followers","following_url":"https://api.github.com/users/intika/following{/other_user}","gists_url":"https://api.github.com/users/intika/gists{/gist_id}","starred_url":"https://api.github.com/users/intika/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/intika/subscriptions","organizations_url":"https://api.github.com/users/intika/orgs","repos_url":"https://api.github.com/users/intika/repos","events_url":"https://api.github.com/users/intika/events{/privacy}","received_events_url":"https://api.github.com/users/intika/received_events","type":"User","site_admin":false},"repo":{"id":70266327,"name":"skype4pidgin","full_name":"intika/skype4pidgin","owner":{"login":"intika","id":6892180,"avatar_url":"https://avatars.githubusercontent.com/u/6892180?v=3","gravatar_id":"","url":"https://api.github.com/users/intika","html_url":"https://github.com/intika","followers_url":"https://api.github.com/users/intika/followers","following_url":"https://api.github.com/users/intika/following{/other_user}","gists_url":"https://api.github.com/users/intika/gists{/gist_id}","starred_url":"https://api.github.com/users/intika/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/intika/subscriptions","organizations_url":"https://api.github.com/users/intika/orgs","repos_url":"https://api.github.com/users/intika/repos","events_url":"https://api.github.com/users/intika/events{/privacy}","received_events_url":"https://api.github.com/users/intika/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/intika/skype4pidgin","description":"Skype Plugin for Pidgin, libpurple and more","fork":true,"url":"https://api.github.com/repos/intika/skype4pidgin","forks_url":"https://api.github.com/repos/intika/skype4pidgin/forks","keys_url":"https://api.github.com/repos/intika/skype4pidgin/keys{/key_id}","collaborators_url":"https://api.github.com/repos/intika/skype4pidgin/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/intika/skype4pidgin/teams","hooks_url":"https://api.github.com/repos/intika/skype4pidgin/hooks","issue_events_url":"https://api.github.com/repos/intika/skype4pidgin/issues/events{/number}","events_url":"https://api.github.com/repos/intika/skype4pidgin/events","assignees_url":"https://api.github.com/repos/intika/skype4pidgin/assignees{/user}","branches_url":"https://api.github.com/repos/intika/skype4pidgin/branches{/branch}","tags_url":"https://api.github.com/repos/intika/skype4pidgin/tags","blobs_url":"https://api.github.com/repos/intika/skype4pidgin/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/intika/skype4pidgin/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/intika/skype4pidgin/git/refs{/sha}","trees_url":"https://api.github.com/repos/intika/skype4pidgin/git/trees{/sha}","statuses_url":"https://api.github.com/repos/intika/skype4pidgin/statuses/{sha}","languages_url":"https://api.github.com/repos/intika/skype4pidgin/languages","stargazers_url":"https://api.github.com/repos/intika/skype4pidgin/stargazers","contributors_url":"https://api.github.com/repos/intika/skype4pidgin/contributors","subscribers_url":"https://api.github.com/repos/intika/skype4pidgin/subscribers","subscription_url":"https://api.github.com/repos/intika/skype4pidgin/subscription","commits_url":"https://api.github.com/repos/intika/skype4pidgin/commits{/sha}","git_commits_url":"https://api.github.com/repos/intika/skype4pidgin/git/commits{/sha}","comments_url":"https://api.github.com/repos/intika/skype4pidgin/comments{/number}","issue_comment_url":"https://api.github.com/repos/intika/skype4pidgin/issues/comments{/number}","contents_url":"https://api.github.com/repos/intika/skype4pidgin/contents/{+path}","compare_url":"https://api.github.com/repos/intika/skype4pidgin/compare/{base}...{head}","merges_url":"https://api.github.com/repos/intika/skype4pidgin/merges","archive_url":"https://api.github.com/repos/intika/skype4pidgin/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/intika/skype4pidgin/downloads","issues_url":"https://api.github.com/repos/intika/skype4pidgin/issues{/number}","pulls_url":"https://api.github.com/repos/intika/skype4pidgin/pulls{/number}","milestones_url":"https://api.github.com/repos/intika/skype4pidgin/milestones{/number}","notifications_url":"https://api.github.com/repos/intika/skype4pidgin/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/intika/skype4pidgin/labels{/name}","releases_url":"https://api.github.com/repos/intika/skype4pidgin/releases{/id}","deployments_url":"https://api.github.com/repos/intika/skype4pidgin/deployments","created_at":"2016-10-07T17:00:58Z","updated_at":"2016-10-07T17:00:59Z","pushed_at":"2016-12-15T08:57:27Z","git_url":"git://github.com/intika/skype4pidgin.git","ssh_url":"git@github.com:intika/skype4pidgin.git","clone_url":"https://github.com/intika/skype4pidgin.git","svn_url":"https://github.com/intika/skype4pidgin","homepage":"https://github.com/EionRobb/skype4pidgin/tree/master/skypeweb#skypeweb-plugin-for-pidgin","size":1796,"stargazers_count":0,"watchers_count":0,"language":"C","has_issues":false,"has_downloads":false,"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":"EionRobb:master","ref":"master","sha":"278b062fcb526a6a4b487073685be0f0722f5af5","user":{"login":"EionRobb","id":1063865,"avatar_url":"https://avatars.githubusercontent.com/u/1063865?v=3","gravatar_id":"","url":"https://api.github.com/users/EionRobb","html_url":"https://github.com/EionRobb","followers_url":"https://api.github.com/users/EionRobb/followers","following_url":"https://api.github.com/users/EionRobb/following{/other_user}","gists_url":"https://api.github.com/users/EionRobb/gists{/gist_id}","starred_url":"https://api.github.com/users/EionRobb/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/EionRobb/subscriptions","organizations_url":"https://api.github.com/users/EionRobb/orgs","repos_url":"https://api.github.com/users/EionRobb/repos","events_url":"https://api.github.com/users/EionRobb/events{/privacy}","received_events_url":"https://api.github.com/users/EionRobb/received_events","type":"User","site_admin":false},"repo":{"id":32111896,"name":"skype4pidgin","full_name":"EionRobb/skype4pidgin","owner":{"login":"EionRobb","id":1063865,"avatar_url":"https://avatars.githubusercontent.com/u/1063865?v=3","gravatar_id":"","url":"https://api.github.com/users/EionRobb","html_url":"https://github.com/EionRobb","followers_url":"https://api.github.com/users/EionRobb/followers","following_url":"https://api.github.com/users/EionRobb/following{/other_user}","gists_url":"https://api.github.com/users/EionRobb/gists{/gist_id}","starred_url":"https://api.github.com/users/EionRobb/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/EionRobb/subscriptions","organizations_url":"https://api.github.com/users/EionRobb/orgs","repos_url":"https://api.github.com/users/EionRobb/repos","events_url":"https://api.github.com/users/EionRobb/events{/privacy}","received_events_url":"https://api.github.com/users/EionRobb/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/EionRobb/skype4pidgin","description":"Skype Plugin for Pidgin, libpurple and more","fork":false,"url":"https://api.github.com/repos/EionRobb/skype4pidgin","forks_url":"https://api.github.com/repos/EionRobb/skype4pidgin/forks","keys_url":"https://api.github.com/repos/EionRobb/skype4pidgin/keys{/key_id}","collaborators_url":"https://api.github.com/repos/EionRobb/skype4pidgin/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/EionRobb/skype4pidgin/teams","hooks_url":"https://api.github.com/repos/EionRobb/skype4pidgin/hooks","issue_events_url":"https://api.github.com/repos/EionRobb/skype4pidgin/issues/events{/number}","events_url":"https://api.github.com/repos/EionRobb/skype4pidgin/events","assignees_url":"https://api.github.com/repos/EionRobb/skype4pidgin/assignees{/user}","branches_url":"https://api.github.com/repos/EionRobb/skype4pidgin/branches{/branch}","tags_url":"https://api.github.com/repos/EionRobb/skype4pidgin/tags","blobs_url":"https://api.github.com/repos/EionRobb/skype4pidgin/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/EionRobb/skype4pidgin/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/EionRobb/skype4pidgin/git/refs{/sha}","trees_url":"https://api.github.com/repos/EionRobb/skype4pidgin/git/trees{/sha}","statuses_url":"https://api.github.com/repos/EionRobb/skype4pidgin/statuses/{sha}","languages_url":"https://api.github.com/repos/EionRobb/skype4pidgin/languages","stargazers_url":"https://api.github.com/repos/EionRobb/skype4pidgin/stargazers","contributors_url":"https://api.github.com/repos/EionRobb/skype4pidgin/contributors","subscribers_url":"https://api.github.com/repos/EionRobb/skype4pidgin/subscribers","subscription_url":"https://api.github.com/repos/EionRobb/skype4pidgin/subscription","commits_url":"https://api.github.com/repos/EionRobb/skype4pidgin/commits{/sha}","git_commits_url":"https://api.github.com/repos/EionRobb/skype4pidgin/git/commits{/sha}","comments_url":"https://api.github.com/repos/EionRobb/skype4pidgin/comments{/number}","issue_comment_url":"https://api.github.com/repos/EionRobb/skype4pidgin/issues/comments{/number}","contents_url":"https://api.github.com/repos/EionRobb/skype4pidgin/contents/{+path}","compare_url":"https://api.github.com/repos/EionRobb/skype4pidgin/compare/{base}...{head}","merges_url":"https://api.github.com/repos/EionRobb/skype4pidgin/merges","archive_url":"https://api.github.com/repos/EionRobb/skype4pidgin/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/EionRobb/skype4pidgin/downloads","issues_url":"https://api.github.com/repos/EionRobb/skype4pidgin/issues{/number}","pulls_url":"https://api.github.com/repos/EionRobb/skype4pidgin/pulls{/number}","milestones_url":"https://api.github.com/repos/EionRobb/skype4pidgin/milestones{/number}","notifications_url":"https://api.github.com/repos/EionRobb/skype4pidgin/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/EionRobb/skype4pidgin/labels{/name}","releases_url":"https://api.github.com/repos/EionRobb/skype4pidgin/releases{/id}","deployments_url":"https://api.github.com/repos/EionRobb/skype4pidgin/deployments","created_at":"2015-03-13T00:44:46Z","updated_at":"2016-12-15T09:41:56Z","pushed_at":"2016-12-15T09:00:40Z","git_url":"git://github.com/EionRobb/skype4pidgin.git","ssh_url":"git@github.com:EionRobb/skype4pidgin.git","clone_url":"https://github.com/EionRobb/skype4pidgin.git","svn_url":"https://github.com/EionRobb/skype4pidgin","homepage":"https://github.com/EionRobb/skype4pidgin/tree/master/skypeweb#skypeweb-plugin-for-pidgin","size":2848,"stargazers_count":427,"watchers_count":427,"language":"C","has_issues":true,"has_downloads":false,"has_wiki":true,"has_pages":false,"forks_count":64,"mirror_url":null,"open_issues_count":68,"forks":64,"open_issues":68,"watchers":427,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/EionRobb/skype4pidgin/pulls/510"},"html":{"href":"https://github.com/EionRobb/skype4pidgin/pull/510"},"issue":{"href":"https://api.github.com/repos/EionRobb/skype4pidgin/issues/510"},"comments":{"href":"https://api.github.com/repos/EionRobb/skype4pidgin/issues/510/comments"},"review_comments":{"href":"https://api.github.com/repos/EionRobb/skype4pidgin/pulls/510/comments"},"review_comment":{"href":"https://api.github.com/repos/EionRobb/skype4pidgin/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/EionRobb/skype4pidgin/pulls/510/commits"},"statuses":{"href":"https://api.github.com/repos/EionRobb/skype4pidgin/statuses/dc4c9e7cf85d2dd9ae89835e25e2718d3fa68ad4"}}}}
{"actor":{"display_login":"dequis"}}