{"action":"created","issue":{"url":"https://api.github.com/repos/iojs/io.js/issues/1351","labels_url":"https://api.github.com/repos/iojs/io.js/issues/1351/labels{/name}","comments_url":"https://api.github.com/repos/iojs/io.js/issues/1351/comments","events_url":"https://api.github.com/repos/iojs/io.js/issues/1351/events","html_url":"https://github.com/iojs/io.js/pull/1351","id":66521785,"number":1351,"title":"http: basic cleanup / mini-opt writeHead/server","user":{"login":"brendanashworth","id":3038254,"avatar_url":"https://avatars.githubusercontent.com/u/3038254?v=3","gravatar_id":"","url":"https://api.github.com/users/brendanashworth","html_url":"https://github.com/brendanashworth","followers_url":"https://api.github.com/users/brendanashworth/followers","following_url":"https://api.github.com/users/brendanashworth/following{/other_user}","gists_url":"https://api.github.com/users/brendanashworth/gists{/gist_id}","starred_url":"https://api.github.com/users/brendanashworth/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/brendanashworth/subscriptions","organizations_url":"https://api.github.com/users/brendanashworth/orgs","repos_url":"https://api.github.com/users/brendanashworth/repos","events_url":"https://api.github.com/users/brendanashworth/events{/privacy}","received_events_url":"https://api.github.com/users/brendanashworth/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/iojs/io.js/labels/http","name":"http","color":"c7def8"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-04-06T04:12:48Z","updated_at":"2015-04-06T11:49:36Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/iojs/io.js/pulls/1351","html_url":"https://github.com/iojs/io.js/pull/1351","diff_url":"https://github.com/iojs/io.js/pull/1351.diff","patch_url":"https://github.com/iojs/io.js/pull/1351.patch"},"body":"This contains a few changes, which include:\r\n- renaming of argument to function (harmless) to match docs and improve\r\n  readability\r\n- removing a useless variable and inlining it with the better\r\n  alternative (headers)\r\n- change .toString() to JS string coersion, which is faster\r\n- alias writeHeader() instead of using .apply()\r\n- remove unnecessary key lookups\r\n- remove unnecessary redeclaration of variable\r\n\r\nSorry, this is a redo of #1293 - I tried to squash / rebase onto `HEAD` / push `--force` and ended up screwing up the history irreversibly. CC'ing those who joined in on the discussion: @Fishrock123 @mscdex @monsanto @dominic @benjamingr"},"comment":{"url":"https://api.github.com/repos/iojs/io.js/issues/comments/90027034","html_url":"https://github.com/iojs/io.js/pull/1351#issuecomment-90027034","issue_url":"https://api.github.com/repos/iojs/io.js/issues/1351","id":90027034,"user":{"login":"tellnes","id":613729,"avatar_url":"https://avatars.githubusercontent.com/u/613729?v=3","gravatar_id":"","url":"https://api.github.com/users/tellnes","html_url":"https://github.com/tellnes","followers_url":"https://api.github.com/users/tellnes/followers","following_url":"https://api.github.com/users/tellnes/following{/other_user}","gists_url":"https://api.github.com/users/tellnes/gists{/gist_id}","starred_url":"https://api.github.com/users/tellnes/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tellnes/subscriptions","organizations_url":"https://api.github.com/users/tellnes/orgs","repos_url":"https://api.github.com/users/tellnes/repos","events_url":"https://api.github.com/users/tellnes/events{/privacy}","received_events_url":"https://api.github.com/users/tellnes/received_events","type":"User","site_admin":false},"created_at":"2015-04-06T11:49:36Z","updated_at":"2015-04-06T11:49:36Z","body":"LGTM"}}