{"action":"created","issue":{"url":"https://api.github.com/repos/bitcoin/bitcoin/issues/5681","labels_url":"https://api.github.com/repos/bitcoin/bitcoin/issues/5681/labels{/name}","comments_url":"https://api.github.com/repos/bitcoin/bitcoin/issues/5681/comments","events_url":"https://api.github.com/repos/bitcoin/bitcoin/issues/5681/events","html_url":"https://github.com/bitcoin/bitcoin/pull/5681","id":54805794,"number":5681,"title":"MOVEONLY-ish: Do not include main.h from any other header","user":{"login":"jtimon","id":1008458,"avatar_url":"https://avatars.githubusercontent.com/u/1008458?v=3","gravatar_id":"","url":"https://api.github.com/users/jtimon","html_url":"https://github.com/jtimon","followers_url":"https://api.github.com/users/jtimon/followers","following_url":"https://api.github.com/users/jtimon/following{/other_user}","gists_url":"https://api.github.com/users/jtimon/gists{/gist_id}","starred_url":"https://api.github.com/users/jtimon/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jtimon/subscriptions","organizations_url":"https://api.github.com/users/jtimon/orgs","repos_url":"https://api.github.com/users/jtimon/repos","events_url":"https://api.github.com/users/jtimon/events{/privacy}","received_events_url":"https://api.github.com/users/jtimon/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/bitcoin/bitcoin/labels/Improvement","name":"Improvement","color":"E6F6D6"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":20,"created_at":"2015-01-19T19:59:33Z","updated_at":"2015-03-15T20:28:11Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/bitcoin/bitcoin/pulls/5681","html_url":"https://github.com/bitcoin/bitcoin/pull/5681","diff_url":"https://github.com/bitcoin/bitcoin/pull/5681.diff","patch_url":"https://github.com/bitcoin/bitcoin/pull/5681.patch"},"body":"Moving some code, including main.h from qt/peertablemodel.h and wallet.h can be avoided.\r\nAlso, main.h doesn't need to include many things that can be included in main.cpp.\r\nAfter that, using the compiler to make sure only files that depend on main.h include it, many missing includes show up.\r\nMany files should require less memory for being compiled after this.\r\n"},"comment":{"url":"https://api.github.com/repos/bitcoin/bitcoin/issues/comments/81226558","html_url":"https://github.com/bitcoin/bitcoin/pull/5681#issuecomment-81226558","issue_url":"https://api.github.com/repos/bitcoin/bitcoin/issues/5681","id":81226558,"user":{"login":"theuni","id":417043,"avatar_url":"https://avatars.githubusercontent.com/u/417043?v=3","gravatar_id":"","url":"https://api.github.com/users/theuni","html_url":"https://github.com/theuni","followers_url":"https://api.github.com/users/theuni/followers","following_url":"https://api.github.com/users/theuni/following{/other_user}","gists_url":"https://api.github.com/users/theuni/gists{/gist_id}","starred_url":"https://api.github.com/users/theuni/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/theuni/subscriptions","organizations_url":"https://api.github.com/users/theuni/orgs","repos_url":"https://api.github.com/users/theuni/repos","events_url":"https://api.github.com/users/theuni/events{/privacy}","received_events_url":"https://api.github.com/users/theuni/received_events","type":"User","site_admin":false},"created_at":"2015-03-15T20:28:10Z","updated_at":"2015-03-15T20:28:10Z","body":"utACK. I'd like to see the node info moved out as well, but that makes sense to do separately."}}