{"action":"created","issue":{"url":"https://api.github.com/repos/flycheck/flycheck/issues/661","labels_url":"https://api.github.com/repos/flycheck/flycheck/issues/661/labels{/name}","comments_url":"https://api.github.com/repos/flycheck/flycheck/issues/661/comments","events_url":"https://api.github.com/repos/flycheck/flycheck/issues/661/events","html_url":"https://github.com/flycheck/flycheck/pull/661","id":87294449,"number":661,"title":"Add clang check","user":{"login":"stsquad","id":22458,"avatar_url":"https://avatars.githubusercontent.com/u/22458?v=3","gravatar_id":"","url":"https://api.github.com/users/stsquad","html_url":"https://github.com/stsquad","followers_url":"https://api.github.com/users/stsquad/followers","following_url":"https://api.github.com/users/stsquad/following{/other_user}","gists_url":"https://api.github.com/users/stsquad/gists{/gist_id}","starred_url":"https://api.github.com/users/stsquad/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/stsquad/subscriptions","organizations_url":"https://api.github.com/users/stsquad/orgs","repos_url":"https://api.github.com/users/stsquad/repos","events_url":"https://api.github.com/users/stsquad/events{/privacy}","received_events_url":"https://api.github.com/users/stsquad/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":9,"created_at":"2015-06-11T10:24:46Z","updated_at":"2015-06-12T05:52:53Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/flycheck/flycheck/pulls/661","html_url":"https://github.com/flycheck/flycheck/pull/661","diff_url":"https://github.com/flycheck/flycheck/pull/661.diff","patch_url":"https://github.com/flycheck/flycheck/pull/661.patch"},"body":"Here is v2 of the pull request (github wouldn't let me re-open the old one).\r\n\r\nChanges:\r\n  - adds some documentation\r\n  - don't chain to the \"legacy\" checkers (which would get confusing)\r\n  - add a proper predicate for testing for the compilation database\r\n"},"comment":{"url":"https://api.github.com/repos/flycheck/flycheck/issues/comments/111368489","html_url":"https://github.com/flycheck/flycheck/pull/661#issuecomment-111368489","issue_url":"https://api.github.com/repos/flycheck/flycheck/issues/661","id":111368489,"user":{"login":"lunaryorn","id":224922,"avatar_url":"https://avatars.githubusercontent.com/u/224922?v=3","gravatar_id":"","url":"https://api.github.com/users/lunaryorn","html_url":"https://github.com/lunaryorn","followers_url":"https://api.github.com/users/lunaryorn/followers","following_url":"https://api.github.com/users/lunaryorn/following{/other_user}","gists_url":"https://api.github.com/users/lunaryorn/gists{/gist_id}","starred_url":"https://api.github.com/users/lunaryorn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lunaryorn/subscriptions","organizations_url":"https://api.github.com/users/lunaryorn/orgs","repos_url":"https://api.github.com/users/lunaryorn/repos","events_url":"https://api.github.com/users/lunaryorn/events{/privacy}","received_events_url":"https://api.github.com/users/lunaryorn/received_events","type":"User","site_admin":false},"created_at":"2015-06-12T05:52:53Z","updated_at":"2015-06-12T05:52:53Z","body":"There's no place to do this in Flycheck. `flycheck-disabled-checkers` is a user option.  \r\n\r\nI'm not so sure whether this checker is a good fit for Flycheck proper.  I think it should be in a separate extension package, e.g. `flycheck-cxx`.  It won't work with manual generation of the database, which goes against the idea of automatic syntax checking of Flycheck proper, and it really needs `idle-change` being removed from `flycheck-check-syntax-automatically` to work fine, because even if you disabled other C/C++ checkers, you'd still have the problem that Flycheck would still attempt a syntax check on unmodified buffers, which would clear all errors reported by this checker.\r\n\r\nIn other words, this checker doesn't really work well with Flycheck's default configuration, which more or less blocks it from being part of Flycheck itself.\r\n\r\nIn a separate package, you'd be free to write a hook function which would check for the existence of a database upfront, and then explicitly enable the checker via `flycheck-checker` and disable `idle-change` buffer locally to entirely any issues with conflicting checkers."}}