GitHub PullRequestReviewCommentEvent at 2014-01-15 09:12:58
Repository: facs
Actor: brainstorm
Payload: {"comment":{"url":"https://api.github.com/repos/SciLifeLab/facs/pulls/comments/8889875","id":8889875,"diff_hunk":"@@ -120,13 +120,14 @@ char *query (char *query, char *bloom_filter, double tole_rate, double sampling_\n   isodate(timestamp);\n   bloom *bl_2 = NEW (bloom);\n   F_set *File_head = make_list (bloom_filter, list);\n-  /*initialize for python interface*/\n+  /*initialization for python interface*/\n   File_head->hits = 0;\n   File_head->all_k = 0;\n   File_head->reads_num = 0;\n   File_head->reads_contam = 0;\n   File_head->filename = bloom_filter;           //extra initialization for python interface\n-  load_bloom (File_head->filename, bl_2);\t//load a bloom filter\n+  if (load_bloom (File_head->filename, bl_2)<=0)\t//load a bloom filter","path":"facs/query.c","position":12,"original_position":12,"commit_id":"ba18dd159e5ff50b7162e0d32a604998b29b7f93","original_commit_id":"ba18dd159e5ff50b7162e0d32a604998b29b7f93","user":{"login":"brainstorm","id":175587,"avatar_url":"https://gravatar.com/avatar/840a3abaf67e9fef8c58225d2e20608a?d=https%3A%2F%2Fidenticons.github.com%2Fb9ca974fba7b1bb8709b64a6dc8120c6.png&r=x","gravatar_id":"840a3abaf67e9fef8c58225d2e20608a","url":"https://api.github.com/users/brainstorm","html_url":"https://github.com/brainstorm","followers_url":"https://api.github.com/users/brainstorm/followers","following_url":"https://api.github.com/users/brainstorm/following{/other_user}","gists_url":"https://api.github.com/users/brainstorm/gists{/gist_id}","starred_url":"https://api.github.com/users/brainstorm/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/brainstorm/subscriptions","organizations_url":"https://api.github.com/users/brainstorm/orgs","repos_url":"https://api.github.com/users/brainstorm/repos","events_url":"https://api.github.com/users/brainstorm/events{/privacy}","received_events_url":"https://api.github.com/users/brainstorm/received_events","type":"User","site_admin":false},"body":"Here there should better better error control, like above with `perror` and proper return codes instead of just `exit()`, @tzcoolman, please fix this.","created_at":"2014-01-15T09:12:58Z","updated_at":"2014-01-15T09:12:58Z","html_url":"https://github.com/SciLifeLab/facs/pull/101#discussion_r8889875","pull_request_url":"https://api.github.com/repos/SciLifeLab/facs/pulls/101","_links":{"self":{"href":"https://api.github.com/repos/SciLifeLab/facs/pulls/comments/8889875"},"html":{"href":"https://github.com/SciLifeLab/facs/pull/101#discussion_r8889875"},"pull_request":{"href":"https://api.github.com/repos/SciLifeLab/facs/pulls/101"}}}}