{"action":"created","comment":{"url":"https://api.github.com/repos/HandBrake/HandBrake/pulls/comments/106016498","pull_request_review_id":26920837,"id":106016498,"diff_hunk":"@@ -892,26 +897,54 @@ int hb_compile_kernel_file( const char *filename, GPUEnv *gpu_info,\n                                                    length, buildLog, &length);\n         }\n \n-        fd1 = fopen( \"kernel-build.log\", \"w+\" );\n-        if( fd1 != NULL ) {\n-            fwrite( buildLog, sizeof(char), length, fd1 );\n-            fclose( fd1 );\n+        fd1 = fopen(\"kernel-build.log\", \"w+\");\n+        if (fd1 != NULL) {\n+            fwrite(buildLog, sizeof(char), length, fd1);\n+            fclose(fd1);\n         }\n \n-        free( buildLog );\n-        return(0);\n+        ret_value = 0;\n+        goto to_exit;\n     }\n \n-    strcpy( gpu_env.kernelSrcFile[idx], filename );\n+    strcpy(gpu_env.kernelSrcFile[idx], filename);\n \n-    if (binaryExisted != 1)\n+    gpu_info->file_count += 1;\n+\n+to_exit:\n+    if (source_str)","path":"libhb/openclwrapper.c","position":222,"original_position":222,"commit_id":"1fb2b4ede46c50578c143b8e5d278bcc99903be6","original_commit_id":"1fb2b4ede46c50578c143b8e5d278bcc99903be6","user":{"login":"bradleysepos","id":70239,"avatar_url":"https://avatars2.githubusercontent.com/u/70239?v=3","gravatar_id":"","url":"https://api.github.com/users/bradleysepos","html_url":"https://github.com/bradleysepos","followers_url":"https://api.github.com/users/bradleysepos/followers","following_url":"https://api.github.com/users/bradleysepos/following{/other_user}","gists_url":"https://api.github.com/users/bradleysepos/gists{/gist_id}","starred_url":"https://api.github.com/users/bradleysepos/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bradleysepos/subscriptions","organizations_url":"https://api.github.com/users/bradleysepos/orgs","repos_url":"https://api.github.com/users/bradleysepos/repos","events_url":"https://api.github.com/users/bradleysepos/events{/privacy}","received_events_url":"https://api.github.com/users/bradleysepos/received_events","type":"User","site_admin":false},"body":"Nit: `if (source_str != NULL)`\r\n\r\nAlso lines 923, 929, 935, 941.","created_at":"2017-03-14T20:20:25Z","updated_at":"2017-03-14T20:20:26Z","html_url":"https://github.com/HandBrake/HandBrake/pull/628#discussion_r106016498","pull_request_url":"https://api.github.com/repos/HandBrake/HandBrake/pulls/628","_links":{"self":{"href":"https://api.github.com/repos/HandBrake/HandBrake/pulls/comments/106016498"},"html":{"href":"https://github.com/HandBrake/HandBrake/pull/628#discussion_r106016498"},"pull_request":{"href":"https://api.github.com/repos/HandBrake/HandBrake/pulls/628"}}},"pull_request":{"url":"https://api.github.com/repos/HandBrake/HandBrake/pulls/628","id":110683689,"html_url":"https://github.com/HandBrake/HandBrake/pull/628","diff_url":"https://github.com/HandBrake/HandBrake/pull/628.diff","patch_url":"https://github.com/HandBrake/HandBrake/pull/628.patch","issue_url":"https://api.github.com/repos/HandBrake/HandBrake/issues/628","number":628,"state":"open","locked":false,"title":"opencl: cleaning more leaks","user":{"login":"maximd33","id":3846214,"avatar_url":"https://avatars2.githubusercontent.com/u/3846214?v=3","gravatar_id":"","url":"https://api.github.com/users/maximd33","html_url":"https://github.com/maximd33","followers_url":"https://api.github.com/users/maximd33/followers","following_url":"https://api.github.com/users/maximd33/following{/other_user}","gists_url":"https://api.github.com/users/maximd33/gists{/gist_id}","starred_url":"https://api.github.com/users/maximd33/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/maximd33/subscriptions","organizations_url":"https://api.github.com/users/maximd33/orgs","repos_url":"https://api.github.com/users/maximd33/repos","events_url":"https://api.github.com/users/maximd33/events{/privacy}","received_events_url":"https://api.github.com/users/maximd33/received_events","type":"User","site_admin":false},"body":"","created_at":"2017-03-14T18:06:18Z","updated_at":"2017-03-14T20:20:26Z","closed_at":null,"merged_at":null,"merge_commit_sha":"47bddf0f638aa6113a758862d3254c142a6aafcc","assignee":{"login":"bradleysepos","id":70239,"avatar_url":"https://avatars2.githubusercontent.com/u/70239?v=3","gravatar_id":"","url":"https://api.github.com/users/bradleysepos","html_url":"https://github.com/bradleysepos","followers_url":"https://api.github.com/users/bradleysepos/followers","following_url":"https://api.github.com/users/bradleysepos/following{/other_user}","gists_url":"https://api.github.com/users/bradleysepos/gists{/gist_id}","starred_url":"https://api.github.com/users/bradleysepos/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bradleysepos/subscriptions","organizations_url":"https://api.github.com/users/bradleysepos/orgs","repos_url":"https://api.github.com/users/bradleysepos/repos","events_url":"https://api.github.com/users/bradleysepos/events{/privacy}","received_events_url":"https://api.github.com/users/bradleysepos/received_events","type":"User","site_admin":false},"assignees":[{"login":"bradleysepos","id":70239,"avatar_url":"https://avatars2.githubusercontent.com/u/70239?v=3","gravatar_id":"","url":"https://api.github.com/users/bradleysepos","html_url":"https://github.com/bradleysepos","followers_url":"https://api.github.com/users/bradleysepos/followers","following_url":"https://api.github.com/users/bradleysepos/following{/other_user}","gists_url":"https://api.github.com/users/bradleysepos/gists{/gist_id}","starred_url":"https://api.github.com/users/bradleysepos/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bradleysepos/subscriptions","organizations_url":"https://api.github.com/users/bradleysepos/orgs","repos_url":"https://api.github.com/users/bradleysepos/repos","events_url":"https://api.github.com/users/bradleysepos/events{/privacy}","received_events_url":"https://api.github.com/users/bradleysepos/received_events","type":"User","site_admin":false}],"milestone":null,"commits_url":"https://api.github.com/repos/HandBrake/HandBrake/pulls/628/commits","review_comments_url":"https://api.github.com/repos/HandBrake/HandBrake/pulls/628/comments","review_comment_url":"https://api.github.com/repos/HandBrake/HandBrake/pulls/comments{/number}","comments_url":"https://api.github.com/repos/HandBrake/HandBrake/issues/628/comments","statuses_url":"https://api.github.com/repos/HandBrake/HandBrake/statuses/1fb2b4ede46c50578c143b8e5d278bcc99903be6","head":{"label":"maximd33:ocl_fix4","ref":"ocl_fix4","sha":"1fb2b4ede46c50578c143b8e5d278bcc99903be6","user":{"login":"maximd33","id":3846214,"avatar_url":"https://avatars2.githubusercontent.com/u/3846214?v=3","gravatar_id":"","url":"https://api.github.com/users/maximd33","html_url":"https://github.com/maximd33","followers_url":"https://api.github.com/users/maximd33/followers","following_url":"https://api.github.com/users/maximd33/following{/other_user}","gists_url":"https://api.github.com/users/maximd33/gists{/gist_id}","starred_url":"https://api.github.com/users/maximd33/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/maximd33/subscriptions","organizations_url":"https://api.github.com/users/maximd33/orgs","repos_url":"https://api.github.com/users/maximd33/repos","events_url":"https://api.github.com/users/maximd33/events{/privacy}","received_events_url":"https://api.github.com/users/maximd33/received_events","type":"User","site_admin":false},"repo":{"id":42952089,"name":"HandBrake","full_name":"maximd33/HandBrake","owner":{"login":"maximd33","id":3846214,"avatar_url":"https://avatars2.githubusercontent.com/u/3846214?v=3","gravatar_id":"","url":"https://api.github.com/users/maximd33","html_url":"https://github.com/maximd33","followers_url":"https://api.github.com/users/maximd33/followers","following_url":"https://api.github.com/users/maximd33/following{/other_user}","gists_url":"https://api.github.com/users/maximd33/gists{/gist_id}","starred_url":"https://api.github.com/users/maximd33/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/maximd33/subscriptions","organizations_url":"https://api.github.com/users/maximd33/orgs","repos_url":"https://api.github.com/users/maximd33/repos","events_url":"https://api.github.com/users/maximd33/events{/privacy}","received_events_url":"https://api.github.com/users/maximd33/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/maximd33/HandBrake","description":"HandBrake's main development repository","fork":true,"url":"https://api.github.com/repos/maximd33/HandBrake","forks_url":"https://api.github.com/repos/maximd33/HandBrake/forks","keys_url":"https://api.github.com/repos/maximd33/HandBrake/keys{/key_id}","collaborators_url":"https://api.github.com/repos/maximd33/HandBrake/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/maximd33/HandBrake/teams","hooks_url":"https://api.github.com/repos/maximd33/HandBrake/hooks","issue_events_url":"https://api.github.com/repos/maximd33/HandBrake/issues/events{/number}","events_url":"https://api.github.com/repos/maximd33/HandBrake/events","assignees_url":"https://api.github.com/repos/maximd33/HandBrake/assignees{/user}","branches_url":"https://api.github.com/repos/maximd33/HandBrake/branches{/branch}","tags_url":"https://api.github.com/repos/maximd33/HandBrake/tags","blobs_url":"https://api.github.com/repos/maximd33/HandBrake/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/maximd33/HandBrake/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/maximd33/HandBrake/git/refs{/sha}","trees_url":"https://api.github.com/repos/maximd33/HandBrake/git/trees{/sha}","statuses_url":"https://api.github.com/repos/maximd33/HandBrake/statuses/{sha}","languages_url":"https://api.github.com/repos/maximd33/HandBrake/languages","stargazers_url":"https://api.github.com/repos/maximd33/HandBrake/stargazers","contributors_url":"https://api.github.com/repos/maximd33/HandBrake/contributors","subscribers_url":"https://api.github.com/repos/maximd33/HandBrake/subscribers","subscription_url":"https://api.github.com/repos/maximd33/HandBrake/subscription","commits_url":"https://api.github.com/repos/maximd33/HandBrake/commits{/sha}","git_commits_url":"https://api.github.com/repos/maximd33/HandBrake/git/commits{/sha}","comments_url":"https://api.github.com/repos/maximd33/HandBrake/comments{/number}","issue_comment_url":"https://api.github.com/repos/maximd33/HandBrake/issues/comments{/number}","contents_url":"https://api.github.com/repos/maximd33/HandBrake/contents/{+path}","compare_url":"https://api.github.com/repos/maximd33/HandBrake/compare/{base}...{head}","merges_url":"https://api.github.com/repos/maximd33/HandBrake/merges","archive_url":"https://api.github.com/repos/maximd33/HandBrake/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/maximd33/HandBrake/downloads","issues_url":"https://api.github.com/repos/maximd33/HandBrake/issues{/number}","pulls_url":"https://api.github.com/repos/maximd33/HandBrake/pulls{/number}","milestones_url":"https://api.github.com/repos/maximd33/HandBrake/milestones{/number}","notifications_url":"https://api.github.com/repos/maximd33/HandBrake/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/maximd33/HandBrake/labels{/name}","releases_url":"https://api.github.com/repos/maximd33/HandBrake/releases{/id}","deployments_url":"https://api.github.com/repos/maximd33/HandBrake/deployments","created_at":"2015-09-22T18:15:08Z","updated_at":"2016-03-30T14:17:27Z","pushed_at":"2017-03-14T18:04:48Z","git_url":"git://github.com/maximd33/HandBrake.git","ssh_url":"git@github.com:maximd33/HandBrake.git","clone_url":"https://github.com/maximd33/HandBrake.git","svn_url":"https://github.com/maximd33/HandBrake","homepage":"https://handbrake.fr","size":70214,"stargazers_count":0,"watchers_count":0,"language":"C","has_issues":false,"has_downloads":true,"has_wiki":false,"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":"HandBrake:master","ref":"master","sha":"35cf0e22cd51157e249414e824b2cc66151104d4","user":{"login":"HandBrake","id":627269,"avatar_url":"https://avatars1.githubusercontent.com/u/627269?v=3","gravatar_id":"","url":"https://api.github.com/users/HandBrake","html_url":"https://github.com/HandBrake","followers_url":"https://api.github.com/users/HandBrake/followers","following_url":"https://api.github.com/users/HandBrake/following{/other_user}","gists_url":"https://api.github.com/users/HandBrake/gists{/gist_id}","starred_url":"https://api.github.com/users/HandBrake/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/HandBrake/subscriptions","organizations_url":"https://api.github.com/users/HandBrake/orgs","repos_url":"https://api.github.com/users/HandBrake/repos","events_url":"https://api.github.com/users/HandBrake/events{/privacy}","received_events_url":"https://api.github.com/users/HandBrake/received_events","type":"Organization","site_admin":false},"repo":{"id":41215835,"name":"HandBrake","full_name":"HandBrake/HandBrake","owner":{"login":"HandBrake","id":627269,"avatar_url":"https://avatars1.githubusercontent.com/u/627269?v=3","gravatar_id":"","url":"https://api.github.com/users/HandBrake","html_url":"https://github.com/HandBrake","followers_url":"https://api.github.com/users/HandBrake/followers","following_url":"https://api.github.com/users/HandBrake/following{/other_user}","gists_url":"https://api.github.com/users/HandBrake/gists{/gist_id}","starred_url":"https://api.github.com/users/HandBrake/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/HandBrake/subscriptions","organizations_url":"https://api.github.com/users/HandBrake/orgs","repos_url":"https://api.github.com/users/HandBrake/repos","events_url":"https://api.github.com/users/HandBrake/events{/privacy}","received_events_url":"https://api.github.com/users/HandBrake/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/HandBrake/HandBrake","description":"HandBrake's main development repository ","fork":false,"url":"https://api.github.com/repos/HandBrake/HandBrake","forks_url":"https://api.github.com/repos/HandBrake/HandBrake/forks","keys_url":"https://api.github.com/repos/HandBrake/HandBrake/keys{/key_id}","collaborators_url":"https://api.github.com/repos/HandBrake/HandBrake/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/HandBrake/HandBrake/teams","hooks_url":"https://api.github.com/repos/HandBrake/HandBrake/hooks","issue_events_url":"https://api.github.com/repos/HandBrake/HandBrake/issues/events{/number}","events_url":"https://api.github.com/repos/HandBrake/HandBrake/events","assignees_url":"https://api.github.com/repos/HandBrake/HandBrake/assignees{/user}","branches_url":"https://api.github.com/repos/HandBrake/HandBrake/branches{/branch}","tags_url":"https://api.github.com/repos/HandBrake/HandBrake/tags","blobs_url":"https://api.github.com/repos/HandBrake/HandBrake/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/HandBrake/HandBrake/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/HandBrake/HandBrake/git/refs{/sha}","trees_url":"https://api.github.com/repos/HandBrake/HandBrake/git/trees{/sha}","statuses_url":"https://api.github.com/repos/HandBrake/HandBrake/statuses/{sha}","languages_url":"https://api.github.com/repos/HandBrake/HandBrake/languages","stargazers_url":"https://api.github.com/repos/HandBrake/HandBrake/stargazers","contributors_url":"https://api.github.com/repos/HandBrake/HandBrake/contributors","subscribers_url":"https://api.github.com/repos/HandBrake/HandBrake/subscribers","subscription_url":"https://api.github.com/repos/HandBrake/HandBrake/subscription","commits_url":"https://api.github.com/repos/HandBrake/HandBrake/commits{/sha}","git_commits_url":"https://api.github.com/repos/HandBrake/HandBrake/git/commits{/sha}","comments_url":"https://api.github.com/repos/HandBrake/HandBrake/comments{/number}","issue_comment_url":"https://api.github.com/repos/HandBrake/HandBrake/issues/comments{/number}","contents_url":"https://api.github.com/repos/HandBrake/HandBrake/contents/{+path}","compare_url":"https://api.github.com/repos/HandBrake/HandBrake/compare/{base}...{head}","merges_url":"https://api.github.com/repos/HandBrake/HandBrake/merges","archive_url":"https://api.github.com/repos/HandBrake/HandBrake/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/HandBrake/HandBrake/downloads","issues_url":"https://api.github.com/repos/HandBrake/HandBrake/issues{/number}","pulls_url":"https://api.github.com/repos/HandBrake/HandBrake/pulls{/number}","milestones_url":"https://api.github.com/repos/HandBrake/HandBrake/milestones{/number}","notifications_url":"https://api.github.com/repos/HandBrake/HandBrake/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/HandBrake/HandBrake/labels{/name}","releases_url":"https://api.github.com/repos/HandBrake/HandBrake/releases{/id}","deployments_url":"https://api.github.com/repos/HandBrake/HandBrake/deployments","created_at":"2015-08-22T16:32:28Z","updated_at":"2017-03-14T12:50:54Z","pushed_at":"2017-03-14T18:06:19Z","git_url":"git://github.com/HandBrake/HandBrake.git","ssh_url":"git@github.com:HandBrake/HandBrake.git","clone_url":"https://github.com/HandBrake/HandBrake.git","svn_url":"https://github.com/HandBrake/HandBrake","homepage":"https://handbrake.fr","size":70364,"stargazers_count":1836,"watchers_count":1836,"language":"C","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":256,"mirror_url":null,"open_issues_count":72,"forks":256,"open_issues":72,"watchers":1836,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/HandBrake/HandBrake/pulls/628"},"html":{"href":"https://github.com/HandBrake/HandBrake/pull/628"},"issue":{"href":"https://api.github.com/repos/HandBrake/HandBrake/issues/628"},"comments":{"href":"https://api.github.com/repos/HandBrake/HandBrake/issues/628/comments"},"review_comments":{"href":"https://api.github.com/repos/HandBrake/HandBrake/pulls/628/comments"},"review_comment":{"href":"https://api.github.com/repos/HandBrake/HandBrake/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/HandBrake/HandBrake/pulls/628/commits"},"statuses":{"href":"https://api.github.com/repos/HandBrake/HandBrake/statuses/1fb2b4ede46c50578c143b8e5d278bcc99903be6"}}}}
{"actor":{"display_login":"bradleysepos"}}