{"action":"created","issue":{"url":"https://api.github.com/repos/alpha0010/ClangLib/issues/5","labels_url":"https://api.github.com/repos/alpha0010/ClangLib/issues/5/labels{/name}","comments_url":"https://api.github.com/repos/alpha0010/ClangLib/issues/5/comments","events_url":"https://api.github.com/repos/alpha0010/ClangLib/issues/5/events","html_url":"https://github.com/alpha0010/ClangLib/pull/5","id":112603745,"number":5,"title":"Rework when the Compile command is being assembled","user":{"login":"yvesdm3000","id":1717312,"avatar_url":"https://avatars.githubusercontent.com/u/1717312?v=3","gravatar_id":"","url":"https://api.github.com/users/yvesdm3000","html_url":"https://github.com/yvesdm3000","followers_url":"https://api.github.com/users/yvesdm3000/followers","following_url":"https://api.github.com/users/yvesdm3000/following{/other_user}","gists_url":"https://api.github.com/users/yvesdm3000/gists{/gist_id}","starred_url":"https://api.github.com/users/yvesdm3000/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/yvesdm3000/subscriptions","organizations_url":"https://api.github.com/users/yvesdm3000/orgs","repos_url":"https://api.github.com/users/yvesdm3000/repos","events_url":"https://api.github.com/users/yvesdm3000/events{/privacy}","received_events_url":"https://api.github.com/users/yvesdm3000/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/alpha0010/ClangLib/labels/bug","name":"bug","color":"fc2929"}],"state":"closed","locked":false,"assignee":{"login":"alpha0010","id":4990928,"avatar_url":"https://avatars.githubusercontent.com/u/4990928?v=3","gravatar_id":"","url":"https://api.github.com/users/alpha0010","html_url":"https://github.com/alpha0010","followers_url":"https://api.github.com/users/alpha0010/followers","following_url":"https://api.github.com/users/alpha0010/following{/other_user}","gists_url":"https://api.github.com/users/alpha0010/gists{/gist_id}","starred_url":"https://api.github.com/users/alpha0010/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/alpha0010/subscriptions","organizations_url":"https://api.github.com/users/alpha0010/orgs","repos_url":"https://api.github.com/users/alpha0010/repos","events_url":"https://api.github.com/users/alpha0010/events{/privacy}","received_events_url":"https://api.github.com/users/alpha0010/received_events","type":"User","site_admin":false},"milestone":null,"comments":3,"created_at":"2015-10-21T14:35:12Z","updated_at":"2015-11-03T01:29:36Z","closed_at":"2015-11-03T01:29:36Z","pull_request":{"url":"https://api.github.com/repos/alpha0010/ClangLib/pulls/5","html_url":"https://github.com/alpha0010/ClangLib/pull/5","diff_url":"https://github.com/alpha0010/ClangLib/pull/5.diff","patch_url":"https://github.com/alpha0010/ClangLib/pull/5.patch"},"body":"Rework how the Compile command is being assembled to work around the fact that wxExecute allows the app to run OnTimer. The Compile-Command-string, needed for Clang to run is being built in an OnTimer event which causes a deadlock.\r\n\r\nIt's essentially when an wxExecute is ran, C::B allows events to be fired to keep the UI happy. That is a good thing, but it also allows OnTimer events to be fired. Previous code then performs another wxExecute to resolve backticks for the Compile command string trough an OnTimer event, which causes a deadlock in wxExecute.\r\nThis change works around this by moving the function that performs this wxExecute outside the OnTimer scope."},"comment":{"url":"https://api.github.com/repos/alpha0010/ClangLib/issues/comments/153211567","html_url":"https://github.com/alpha0010/ClangLib/pull/5#issuecomment-153211567","issue_url":"https://api.github.com/repos/alpha0010/ClangLib/issues/5","id":153211567,"user":{"login":"alpha0010","id":4990928,"avatar_url":"https://avatars.githubusercontent.com/u/4990928?v=3","gravatar_id":"","url":"https://api.github.com/users/alpha0010","html_url":"https://github.com/alpha0010","followers_url":"https://api.github.com/users/alpha0010/followers","following_url":"https://api.github.com/users/alpha0010/following{/other_user}","gists_url":"https://api.github.com/users/alpha0010/gists{/gist_id}","starred_url":"https://api.github.com/users/alpha0010/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/alpha0010/subscriptions","organizations_url":"https://api.github.com/users/alpha0010/orgs","repos_url":"https://api.github.com/users/alpha0010/repos","events_url":"https://api.github.com/users/alpha0010/events{/privacy}","received_events_url":"https://api.github.com/users/alpha0010/received_events","type":"User","site_admin":false},"created_at":"2015-11-03T01:29:36Z","updated_at":"2015-11-03T01:29:36Z","body":"Thanks for the fix.\r\n(I assume the other two commits were not intended for this pull.)"}}