{"action":"created","issue":{"url":"https://api.github.com/repos/berkerpeksag/astor/issues/25","labels_url":"https://api.github.com/repos/berkerpeksag/astor/issues/25/labels{/name}","comments_url":"https://api.github.com/repos/berkerpeksag/astor/issues/25/comments","events_url":"https://api.github.com/repos/berkerpeksag/astor/issues/25/events","html_url":"https://github.com/berkerpeksag/astor/pull/25","id":77228886,"number":25,"title":"Added new anti8 example utility; used it to find and fix bugs","user":{"login":"pmaupin","id":2379062,"avatar_url":"https://avatars.githubusercontent.com/u/2379062?v=3","gravatar_id":"","url":"https://api.github.com/users/pmaupin","html_url":"https://github.com/pmaupin","followers_url":"https://api.github.com/users/pmaupin/followers","following_url":"https://api.github.com/users/pmaupin/following{/other_user}","gists_url":"https://api.github.com/users/pmaupin/gists{/gist_id}","starred_url":"https://api.github.com/users/pmaupin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pmaupin/subscriptions","organizations_url":"https://api.github.com/users/pmaupin/orgs","repos_url":"https://api.github.com/users/pmaupin/repos","events_url":"https://api.github.com/users/pmaupin/events{/privacy}","received_events_url":"https://api.github.com/users/pmaupin/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-05-17T06:14:24Z","updated_at":"2015-05-17T11:11:18Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/berkerpeksag/astor/pulls/25","html_url":"https://github.com/berkerpeksag/astor/pull/25","diff_url":"https://github.com/berkerpeksag/astor/pull/25.diff","patch_url":"https://github.com/berkerpeksag/astor/pull/25.patch"},"body":"Berker:\r\n\r\nI'm still pretty new at this git thing, so I probably didn't put this in as many separate commits as some would prefer.  Feel free to slice and dice it anyway you want when updating the master.  (I realize now that I should have done a branch before doing the work, as well, so that's something to try to do next time.)\r\n\r\nAnyway, I wrote an example utility to be able to easily compare two source trees (e.g. before and after PEP8 modifications), and ran it on the 2.7 and 3.4 standard libraries, and exposed several bugs in codegen in the process.\r\n\r\nFor each bug, I wrote a failing testcase and then fixed the bug.\r\n\r\nIt might be worthwhile to do the same thing for 3.5 before your release, if the goal of the release is 3.5 support.  All you have to do is run the anti8 script with a single parameter that is the path to the system libraries.\r\n\r\nIt's well past my bedtime, so I don't think I'll have any more code for your new version.\r\n\r\nThanks,\r\nPat\r\n"},"comment":{"url":"https://api.github.com/repos/berkerpeksag/astor/issues/comments/102784512","html_url":"https://github.com/berkerpeksag/astor/pull/25#issuecomment-102784512","issue_url":"https://api.github.com/repos/berkerpeksag/astor/issues/25","id":102784512,"user":{"login":"berkerpeksag","id":26338,"avatar_url":"https://avatars.githubusercontent.com/u/26338?v=3","gravatar_id":"","url":"https://api.github.com/users/berkerpeksag","html_url":"https://github.com/berkerpeksag","followers_url":"https://api.github.com/users/berkerpeksag/followers","following_url":"https://api.github.com/users/berkerpeksag/following{/other_user}","gists_url":"https://api.github.com/users/berkerpeksag/gists{/gist_id}","starred_url":"https://api.github.com/users/berkerpeksag/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/berkerpeksag/subscriptions","organizations_url":"https://api.github.com/users/berkerpeksag/orgs","repos_url":"https://api.github.com/users/berkerpeksag/repos","events_url":"https://api.github.com/users/berkerpeksag/events{/privacy}","received_events_url":"https://api.github.com/users/berkerpeksag/received_events","type":"User","site_admin":false},"created_at":"2015-05-17T11:11:18Z","updated_at":"2015-05-17T11:11:18Z","body":"This looks great, thanks! Perhaps we can make anti8 a public module in astor and add a CLI for it:\r\n\r\n```sh\r\n$ astor anti8 <path>\r\n# or \r\n$ python -m astor.anti8 <path>\r\n```"}}