{"action":"created","comment":{"url":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/pulls/comments/242337154","pull_request_review_id":185824748,"id":242337154,"node_id":"MDI0OlB1bGxSZXF1ZXN0UmV2aWV3Q29tbWVudDI0MjMzNzE1NA==","diff_hunk":"@@ -799,9 +800,9 @@ layers can be categorized into two categories:\n  * Implicit Layers\n  * Explicit Layers\n \n-The main difference between the two is that Implicit Layers are automatically\n-enabled, unless overridden, and Explicit Layers must be enabled.  Remember,\n-Implicit Layers are not present on all Operating Systems (like Android).\n+The main difference between the two is that implicitlLayers are automatically","path":"loader/LoaderAndLayerInterface.md","position":376,"original_position":376,"commit_id":"22cc01db8da2501bb312b0757adf7420a7ad9c75","original_commit_id":"22cc01db8da2501bb312b0757adf7420a7ad9c75","user":{"login":"lunarpapillo","id":45772930,"node_id":"MDQ6VXNlcjQ1NzcyOTMw","avatar_url":"https://avatars1.githubusercontent.com/u/45772930?v=4","gravatar_id":"","url":"https://api.github.com/users/lunarpapillo","html_url":"https://github.com/lunarpapillo","followers_url":"https://api.github.com/users/lunarpapillo/followers","following_url":"https://api.github.com/users/lunarpapillo/following{/other_user}","gists_url":"https://api.github.com/users/lunarpapillo/gists{/gist_id}","starred_url":"https://api.github.com/users/lunarpapillo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lunarpapillo/subscriptions","organizations_url":"https://api.github.com/users/lunarpapillo/orgs","repos_url":"https://api.github.com/users/lunarpapillo/repos","events_url":"https://api.github.com/users/lunarpapillo/events{/privacy}","received_events_url":"https://api.github.com/users/lunarpapillo/received_events","type":"User","site_admin":false},"body":"Yup, sure do.  Will fix.","created_at":"2018-12-17T22:16:12Z","updated_at":"2018-12-17T22:16:12Z","html_url":"https://github.com/KhronosGroup/Vulkan-Loader/pull/119#discussion_r242337154","pull_request_url":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/pulls/119","author_association":"NONE","_links":{"self":{"href":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/pulls/comments/242337154"},"html":{"href":"https://github.com/KhronosGroup/Vulkan-Loader/pull/119#discussion_r242337154"},"pull_request":{"href":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/pulls/119"}},"in_reply_to_id":242261199},"pull_request":{"url":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/pulls/119","id":239237055,"node_id":"MDExOlB1bGxSZXF1ZXN0MjM5MjM3MDU1","html_url":"https://github.com/KhronosGroup/Vulkan-Loader/pull/119","diff_url":"https://github.com/KhronosGroup/Vulkan-Loader/pull/119.diff","patch_url":"https://github.com/KhronosGroup/Vulkan-Loader/pull/119.patch","issue_url":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/issues/119","number":119,"state":"open","locked":false,"title":"Minor grammar and consistency improvements","user":{"login":"lunarpapillo","id":45772930,"node_id":"MDQ6VXNlcjQ1NzcyOTMw","avatar_url":"https://avatars1.githubusercontent.com/u/45772930?v=4","gravatar_id":"","url":"https://api.github.com/users/lunarpapillo","html_url":"https://github.com/lunarpapillo","followers_url":"https://api.github.com/users/lunarpapillo/followers","following_url":"https://api.github.com/users/lunarpapillo/following{/other_user}","gists_url":"https://api.github.com/users/lunarpapillo/gists{/gist_id}","starred_url":"https://api.github.com/users/lunarpapillo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lunarpapillo/subscriptions","organizations_url":"https://api.github.com/users/lunarpapillo/orgs","repos_url":"https://api.github.com/users/lunarpapillo/repos","events_url":"https://api.github.com/users/lunarpapillo/events{/privacy}","received_events_url":"https://api.github.com/users/lunarpapillo/received_events","type":"User","site_admin":false},"body":"LoaderAndLayerInterface.md:\r\n- entry-point => entry point\r\n- MacOS => macOS (except in API function names)\r\n- on the applications surfaces => on the application's surfaces\r\n- Because layers are optionally, => Because layers are optional,\r\n- a child of a one of the above => a child of one of the above\r\n- action / operation => action or operation\r\n- fixed indentation of the dispatch table description to what I think was intended\r\n- the application and/or system can specify => the application and the system can each specify\r\n- linking to the .dll and .so versions of the loader => linking to the dynamic versions of the loader\r\n  (because .dylib is also a dynamic link)\r\n- platform specific => platform-specific\r\n- dlsym() => `dlsym` (to match usage elsewhere)\r\n- setup => set up (when used as a verb)\r\n- don't break link references across lines\r\n  e.g. [XX XX XX XX](YY) should never be broken in the YY clause\r\n- 3 enabled layers => three enabled layers (spell numbers up to 10)\r\n- backwards compatibility => backward compatibility\r\n- Windows paths use \"\\\" instead of \"/\"\r\n- libXXX.YYY => `libXXX.YYY` (to match usage elsewhere)\r\n- Layers => layers (to match usage elsewhere, where \"Layers\" isn't a capitalized term)\r\n- bottommost => bottom-most (for consistency elsewhere)\r\n- the VK_LAYER_XXX ... => the VK_LAYER_XXX layer (for consistency elsewhere)\r\n- either end all list or table items with periods, or none of them\r\n- Khronos approved => Khronos-approved\r\n- hand-shake => handshake\r\n- logical device (This depends => logical device (this depends\r\n- other-hand => other hand\r\n- filtering out of => filtering of\r\n- JSON formatted => JSON-formatted\r\n- /path/to/something => `/path/to/something` (to match usage elsewhere)\r\n- system wide registries => system registries\r\n- fixed some \"*\" entries (which generally must be escaed, \"\\*\", in markdown)\r\n- environment variable enabled layer => environment-variable-enabled layer\r\n- The implementation of each intercept functions is => The implementation fo each intercept function is\r\n- corrected some \"it's\" to \"its\", and vice versa\r\n- the extension suffix of the file ends with \".json\" => the extension suffix of the file is \".json\"\r\n- added commas to fix illegal JSON file content\r\n- semi-colon separated => semi-colon-separated\r\n- other minor grammar and consistency fixes","created_at":"2018-12-17T18:10:47Z","updated_at":"2018-12-17T22:16:12Z","closed_at":null,"merged_at":null,"merge_commit_sha":"567e5e49b06caefafe17e9aba08cabc3001fa93c","assignee":null,"assignees":[],"requested_reviewers":[],"requested_teams":[],"labels":[],"milestone":null,"commits_url":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/pulls/119/commits","review_comments_url":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/pulls/119/comments","review_comment_url":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/pulls/comments{/number}","comments_url":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/issues/119/comments","statuses_url":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/statuses/22cc01db8da2501bb312b0757adf7420a7ad9c75","head":{"label":"lunarpapillo:bob_review","ref":"bob_review","sha":"22cc01db8da2501bb312b0757adf7420a7ad9c75","user":{"login":"lunarpapillo","id":45772930,"node_id":"MDQ6VXNlcjQ1NzcyOTMw","avatar_url":"https://avatars1.githubusercontent.com/u/45772930?v=4","gravatar_id":"","url":"https://api.github.com/users/lunarpapillo","html_url":"https://github.com/lunarpapillo","followers_url":"https://api.github.com/users/lunarpapillo/followers","following_url":"https://api.github.com/users/lunarpapillo/following{/other_user}","gists_url":"https://api.github.com/users/lunarpapillo/gists{/gist_id}","starred_url":"https://api.github.com/users/lunarpapillo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lunarpapillo/subscriptions","organizations_url":"https://api.github.com/users/lunarpapillo/orgs","repos_url":"https://api.github.com/users/lunarpapillo/repos","events_url":"https://api.github.com/users/lunarpapillo/events{/privacy}","received_events_url":"https://api.github.com/users/lunarpapillo/received_events","type":"User","site_admin":false},"repo":{"id":162170585,"node_id":"MDEwOlJlcG9zaXRvcnkxNjIxNzA1ODU=","name":"Vulkan-Loader","full_name":"lunarpapillo/Vulkan-Loader","private":false,"owner":{"login":"lunarpapillo","id":45772930,"node_id":"MDQ6VXNlcjQ1NzcyOTMw","avatar_url":"https://avatars1.githubusercontent.com/u/45772930?v=4","gravatar_id":"","url":"https://api.github.com/users/lunarpapillo","html_url":"https://github.com/lunarpapillo","followers_url":"https://api.github.com/users/lunarpapillo/followers","following_url":"https://api.github.com/users/lunarpapillo/following{/other_user}","gists_url":"https://api.github.com/users/lunarpapillo/gists{/gist_id}","starred_url":"https://api.github.com/users/lunarpapillo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lunarpapillo/subscriptions","organizations_url":"https://api.github.com/users/lunarpapillo/orgs","repos_url":"https://api.github.com/users/lunarpapillo/repos","events_url":"https://api.github.com/users/lunarpapillo/events{/privacy}","received_events_url":"https://api.github.com/users/lunarpapillo/received_events","type":"User","site_admin":false},"html_url":"https://github.com/lunarpapillo/Vulkan-Loader","description":"Vulkan Loader","fork":true,"url":"https://api.github.com/repos/lunarpapillo/Vulkan-Loader","forks_url":"https://api.github.com/repos/lunarpapillo/Vulkan-Loader/forks","keys_url":"https://api.github.com/repos/lunarpapillo/Vulkan-Loader/keys{/key_id}","collaborators_url":"https://api.github.com/repos/lunarpapillo/Vulkan-Loader/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/lunarpapillo/Vulkan-Loader/teams","hooks_url":"https://api.github.com/repos/lunarpapillo/Vulkan-Loader/hooks","issue_events_url":"https://api.github.com/repos/lunarpapillo/Vulkan-Loader/issues/events{/number}","events_url":"https://api.github.com/repos/lunarpapillo/Vulkan-Loader/events","assignees_url":"https://api.github.com/repos/lunarpapillo/Vulkan-Loader/assignees{/user}","branches_url":"https://api.github.com/repos/lunarpapillo/Vulkan-Loader/branches{/branch}","tags_url":"https://api.github.com/repos/lunarpapillo/Vulkan-Loader/tags","blobs_url":"https://api.github.com/repos/lunarpapillo/Vulkan-Loader/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/lunarpapillo/Vulkan-Loader/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/lunarpapillo/Vulkan-Loader/git/refs{/sha}","trees_url":"https://api.github.com/repos/lunarpapillo/Vulkan-Loader/git/trees{/sha}","statuses_url":"https://api.github.com/repos/lunarpapillo/Vulkan-Loader/statuses/{sha}","languages_url":"https://api.github.com/repos/lunarpapillo/Vulkan-Loader/languages","stargazers_url":"https://api.github.com/repos/lunarpapillo/Vulkan-Loader/stargazers","contributors_url":"https://api.github.com/repos/lunarpapillo/Vulkan-Loader/contributors","subscribers_url":"https://api.github.com/repos/lunarpapillo/Vulkan-Loader/subscribers","subscription_url":"https://api.github.com/repos/lunarpapillo/Vulkan-Loader/subscription","commits_url":"https://api.github.com/repos/lunarpapillo/Vulkan-Loader/commits{/sha}","git_commits_url":"https://api.github.com/repos/lunarpapillo/Vulkan-Loader/git/commits{/sha}","comments_url":"https://api.github.com/repos/lunarpapillo/Vulkan-Loader/comments{/number}","issue_comment_url":"https://api.github.com/repos/lunarpapillo/Vulkan-Loader/issues/comments{/number}","contents_url":"https://api.github.com/repos/lunarpapillo/Vulkan-Loader/contents/{+path}","compare_url":"https://api.github.com/repos/lunarpapillo/Vulkan-Loader/compare/{base}...{head}","merges_url":"https://api.github.com/repos/lunarpapillo/Vulkan-Loader/merges","archive_url":"https://api.github.com/repos/lunarpapillo/Vulkan-Loader/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/lunarpapillo/Vulkan-Loader/downloads","issues_url":"https://api.github.com/repos/lunarpapillo/Vulkan-Loader/issues{/number}","pulls_url":"https://api.github.com/repos/lunarpapillo/Vulkan-Loader/pulls{/number}","milestones_url":"https://api.github.com/repos/lunarpapillo/Vulkan-Loader/milestones{/number}","notifications_url":"https://api.github.com/repos/lunarpapillo/Vulkan-Loader/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/lunarpapillo/Vulkan-Loader/labels{/name}","releases_url":"https://api.github.com/repos/lunarpapillo/Vulkan-Loader/releases{/id}","deployments_url":"https://api.github.com/repos/lunarpapillo/Vulkan-Loader/deployments","created_at":"2018-12-17T18:03:08Z","updated_at":"2018-12-17T18:03:11Z","pushed_at":"2018-12-17T18:09:39Z","git_url":"git://github.com/lunarpapillo/Vulkan-Loader.git","ssh_url":"git@github.com:lunarpapillo/Vulkan-Loader.git","clone_url":"https://github.com/lunarpapillo/Vulkan-Loader.git","svn_url":"https://github.com/lunarpapillo/Vulkan-Loader","homepage":"","size":47818,"stargazers_count":0,"watchers_count":0,"language":"C","has_issues":false,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"archived":false,"open_issues_count":0,"license":{"key":"other","name":"Other","spdx_id":"NOASSERTION","url":null,"node_id":"MDc6TGljZW5zZTA="},"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"KhronosGroup:master","ref":"master","sha":"171f2a6b44546c8d89273233ac771f8c05f11f62","user":{"login":"KhronosGroup","id":1608701,"node_id":"MDEyOk9yZ2FuaXphdGlvbjE2MDg3MDE=","avatar_url":"https://avatars3.githubusercontent.com/u/1608701?v=4","gravatar_id":"","url":"https://api.github.com/users/KhronosGroup","html_url":"https://github.com/KhronosGroup","followers_url":"https://api.github.com/users/KhronosGroup/followers","following_url":"https://api.github.com/users/KhronosGroup/following{/other_user}","gists_url":"https://api.github.com/users/KhronosGroup/gists{/gist_id}","starred_url":"https://api.github.com/users/KhronosGroup/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/KhronosGroup/subscriptions","organizations_url":"https://api.github.com/users/KhronosGroup/orgs","repos_url":"https://api.github.com/users/KhronosGroup/repos","events_url":"https://api.github.com/users/KhronosGroup/events{/privacy}","received_events_url":"https://api.github.com/users/KhronosGroup/received_events","type":"Organization","site_admin":false},"repo":{"id":132043422,"node_id":"MDEwOlJlcG9zaXRvcnkxMzIwNDM0MjI=","name":"Vulkan-Loader","full_name":"KhronosGroup/Vulkan-Loader","private":false,"owner":{"login":"KhronosGroup","id":1608701,"node_id":"MDEyOk9yZ2FuaXphdGlvbjE2MDg3MDE=","avatar_url":"https://avatars3.githubusercontent.com/u/1608701?v=4","gravatar_id":"","url":"https://api.github.com/users/KhronosGroup","html_url":"https://github.com/KhronosGroup","followers_url":"https://api.github.com/users/KhronosGroup/followers","following_url":"https://api.github.com/users/KhronosGroup/following{/other_user}","gists_url":"https://api.github.com/users/KhronosGroup/gists{/gist_id}","starred_url":"https://api.github.com/users/KhronosGroup/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/KhronosGroup/subscriptions","organizations_url":"https://api.github.com/users/KhronosGroup/orgs","repos_url":"https://api.github.com/users/KhronosGroup/repos","events_url":"https://api.github.com/users/KhronosGroup/events{/privacy}","received_events_url":"https://api.github.com/users/KhronosGroup/received_events","type":"Organization","site_admin":false},"html_url":"https://github.com/KhronosGroup/Vulkan-Loader","description":"Vulkan Loader","fork":false,"url":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader","forks_url":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/forks","keys_url":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/keys{/key_id}","collaborators_url":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/teams","hooks_url":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/hooks","issue_events_url":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/issues/events{/number}","events_url":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/events","assignees_url":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/assignees{/user}","branches_url":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/branches{/branch}","tags_url":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/tags","blobs_url":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/git/refs{/sha}","trees_url":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/git/trees{/sha}","statuses_url":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/statuses/{sha}","languages_url":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/languages","stargazers_url":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/stargazers","contributors_url":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/contributors","subscribers_url":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/subscribers","subscription_url":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/subscription","commits_url":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/commits{/sha}","git_commits_url":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/git/commits{/sha}","comments_url":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/comments{/number}","issue_comment_url":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/issues/comments{/number}","contents_url":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/contents/{+path}","compare_url":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/compare/{base}...{head}","merges_url":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/merges","archive_url":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/downloads","issues_url":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/issues{/number}","pulls_url":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/pulls{/number}","milestones_url":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/milestones{/number}","notifications_url":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/labels{/name}","releases_url":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/releases{/id}","deployments_url":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/deployments","created_at":"2018-05-03T19:59:26Z","updated_at":"2018-12-13T13:46:56Z","pushed_at":"2018-12-17T18:10:48Z","git_url":"git://github.com/KhronosGroup/Vulkan-Loader.git","ssh_url":"git@github.com:KhronosGroup/Vulkan-Loader.git","clone_url":"https://github.com/KhronosGroup/Vulkan-Loader.git","svn_url":"https://github.com/KhronosGroup/Vulkan-Loader","homepage":"","size":47816,"stargazers_count":47,"watchers_count":47,"language":"C","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":28,"mirror_url":null,"archived":false,"open_issues_count":31,"license":{"key":"other","name":"Other","spdx_id":"NOASSERTION","url":null,"node_id":"MDc6TGljZW5zZTA="},"forks":28,"open_issues":31,"watchers":47,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/pulls/119"},"html":{"href":"https://github.com/KhronosGroup/Vulkan-Loader/pull/119"},"issue":{"href":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/issues/119"},"comments":{"href":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/issues/119/comments"},"review_comments":{"href":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/pulls/119/comments"},"review_comment":{"href":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/pulls/119/commits"},"statuses":{"href":"https://api.github.com/repos/KhronosGroup/Vulkan-Loader/statuses/22cc01db8da2501bb312b0757adf7420a7ad9c75"}},"author_association":"NONE"}}
{"actor":{"display_login":"lunarpapillo"}}