{"action":"created","issue":{"url":"https://api.github.com/repos/geany/geany/issues/1160","repository_url":"https://api.github.com/repos/geany/geany","labels_url":"https://api.github.com/repos/geany/geany/issues/1160/labels{/name}","comments_url":"https://api.github.com/repos/geany/geany/issues/1160/comments","events_url":"https://api.github.com/repos/geany/geany/issues/1160/events","html_url":"https://github.com/geany/geany/pull/1160","id":169007391,"number":1160,"title":"Ctags sync, part 1","user":{"login":"techee","id":713965,"avatar_url":"https://avatars.githubusercontent.com/u/713965?v=3","gravatar_id":"","url":"https://api.github.com/users/techee","html_url":"https://github.com/techee","followers_url":"https://api.github.com/users/techee/followers","following_url":"https://api.github.com/users/techee/following{/other_user}","gists_url":"https://api.github.com/users/techee/gists{/gist_id}","starred_url":"https://api.github.com/users/techee/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/techee/subscriptions","organizations_url":"https://api.github.com/users/techee/orgs","repos_url":"https://api.github.com/users/techee/repos","events_url":"https://api.github.com/users/techee/events{/privacy}","received_events_url":"https://api.github.com/users/techee/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"b4n","id":793526,"avatar_url":"https://avatars.githubusercontent.com/u/793526?v=3","gravatar_id":"","url":"https://api.github.com/users/b4n","html_url":"https://github.com/b4n","followers_url":"https://api.github.com/users/b4n/followers","following_url":"https://api.github.com/users/b4n/following{/other_user}","gists_url":"https://api.github.com/users/b4n/gists{/gist_id}","starred_url":"https://api.github.com/users/b4n/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/b4n/subscriptions","organizations_url":"https://api.github.com/users/b4n/orgs","repos_url":"https://api.github.com/users/b4n/repos","events_url":"https://api.github.com/users/b4n/events{/privacy}","received_events_url":"https://api.github.com/users/b4n/received_events","type":"User","site_admin":false},"assignees":[{"login":"b4n","id":793526,"avatar_url":"https://avatars.githubusercontent.com/u/793526?v=3","gravatar_id":"","url":"https://api.github.com/users/b4n","html_url":"https://github.com/b4n","followers_url":"https://api.github.com/users/b4n/followers","following_url":"https://api.github.com/users/b4n/following{/other_user}","gists_url":"https://api.github.com/users/b4n/gists{/gist_id}","starred_url":"https://api.github.com/users/b4n/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/b4n/subscriptions","organizations_url":"https://api.github.com/users/b4n/orgs","repos_url":"https://api.github.com/users/b4n/repos","events_url":"https://api.github.com/users/b4n/events{/privacy}","received_events_url":"https://api.github.com/users/b4n/received_events","type":"User","site_admin":false}],"milestone":null,"comments":21,"created_at":"2016-08-02T22:50:05Z","updated_at":"2016-08-04T13:18:59Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/geany/geany/pulls/1160","html_url":"https://github.com/geany/geany/pull/1160","diff_url":"https://github.com/geany/geany/pull/1160.diff","patch_url":"https://github.com/geany/geany/pull/1160.patch"},"body":"The purpose of this pull request is to bring parsers as close to uctags parsers as possible in terms of their interaction with the \"main\" part of ctags (this means, no changes in parser's actual logic are made in this patch set). This involves various renames of functions in the \"API\" of \"main\" so that Geany parsers call the same functions as the uctags ones. Additional modifications include using tab-only indentation style (some files used combination of tabs and spaces) and unification of includes with uctags so some stuff had to be moved around.\r\n\r\nI was concentrated mostly on how the parsers look - in some cases I didn't backport the full implementation of the \"main\" functionality and used just some dummy implementation (in cases where the actual implementation isn't used by Geany).\r\n\r\nAt this point, parsers can be synced one by one independently of the sync work inside \"main\" (some are already identical with the uctags ones!). The work on syncing \"main\" shouldn't affect the work on syncing parsers (there are just tiny bits missing like cork support but these will be 1-line diffs in parsers).\r\n\r\nIt would be good to merge this soon - the patches have a huge potential to introduce merge conflicts with any other change made in the ctags directory and resolving conflicts will be quite painful.\r\n"},"comment":{"url":"https://api.github.com/repos/geany/geany/issues/comments/237549978","html_url":"https://github.com/geany/geany/pull/1160#issuecomment-237549978","issue_url":"https://api.github.com/repos/geany/geany/issues/1160","id":237549978,"user":{"login":"b4n","id":793526,"avatar_url":"https://avatars.githubusercontent.com/u/793526?v=3","gravatar_id":"","url":"https://api.github.com/users/b4n","html_url":"https://github.com/b4n","followers_url":"https://api.github.com/users/b4n/followers","following_url":"https://api.github.com/users/b4n/following{/other_user}","gists_url":"https://api.github.com/users/b4n/gists{/gist_id}","starred_url":"https://api.github.com/users/b4n/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/b4n/subscriptions","organizations_url":"https://api.github.com/users/b4n/orgs","repos_url":"https://api.github.com/users/b4n/repos","events_url":"https://api.github.com/users/b4n/events{/privacy}","received_events_url":"https://api.github.com/users/b4n/received_events","type":"User","site_admin":false},"created_at":"2016-08-04T13:18:59Z","updated_at":"2016-08-04T13:18:59Z","body":"@elextr @techee see, 6bc579798716dbe94a171a016f9989c92c4b67e2 was quicker to review than some previous patches 'cause `git show -b 6bc579798716dbe94a171a016f9989c92c4b67e2` was very straightforward :)  (about 62 lines changes (vim settings), instead of about 7446)"}}
{"actor":{"display_login":"b4n"}}