{"action":"created","issue":{"url":"https://api.github.com/repos/iojs/io.js/issues/1196","labels_url":"https://api.github.com/repos/iojs/io.js/issues/1196/labels{/name}","comments_url":"https://api.github.com/repos/iojs/io.js/issues/1196/comments","events_url":"https://api.github.com/repos/iojs/io.js/issues/1196/events","html_url":"https://github.com/iojs/io.js/pull/1196","id":62818888,"number":1196,"title":"test: cache lazy properties, fix style nits","user":{"login":"rvagg","id":495647,"avatar_url":"https://avatars.githubusercontent.com/u/495647?v=3","gravatar_id":"","url":"https://api.github.com/users/rvagg","html_url":"https://github.com/rvagg","followers_url":"https://api.github.com/users/rvagg/followers","following_url":"https://api.github.com/users/rvagg/following{/other_user}","gists_url":"https://api.github.com/users/rvagg/gists{/gist_id}","starred_url":"https://api.github.com/users/rvagg/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rvagg/subscriptions","organizations_url":"https://api.github.com/users/rvagg/orgs","repos_url":"https://api.github.com/users/rvagg/repos","events_url":"https://api.github.com/users/rvagg/events{/privacy}","received_events_url":"https://api.github.com/users/rvagg/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-03-18T22:32:00Z","updated_at":"2015-03-19T00:29:16Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/iojs/io.js/pulls/1196","html_url":"https://github.com/iojs/io.js/pull/1196","diff_url":"https://github.com/iojs/io.js/pull/1196.diff","patch_url":"https://github.com/iojs/io.js/pull/1196.patch"},"body":"_(addressing the concerns I raised too late in #1167)_\r\n\r\n`inFreeBSDJail` involves an `execSync()` and is used by `localhost_ipv4` so will be unnecessarily expensive, so cache both values and reuse rather than re-evaluate each time.\r\n\r\nRenamed `localhost_ipv4` to `localhostIPv4` for style consistency.\r\n\r\n/R=@jbergstroem \r\n/R=@bnoordhuis "},"comment":{"url":"https://api.github.com/repos/iojs/io.js/issues/comments/83233547","html_url":"https://github.com/iojs/io.js/pull/1196#issuecomment-83233547","issue_url":"https://api.github.com/repos/iojs/io.js/issues/1196","id":83233547,"user":{"login":"rvagg","id":495647,"avatar_url":"https://avatars.githubusercontent.com/u/495647?v=3","gravatar_id":"","url":"https://api.github.com/users/rvagg","html_url":"https://github.com/rvagg","followers_url":"https://api.github.com/users/rvagg/followers","following_url":"https://api.github.com/users/rvagg/following{/other_user}","gists_url":"https://api.github.com/users/rvagg/gists{/gist_id}","starred_url":"https://api.github.com/users/rvagg/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rvagg/subscriptions","organizations_url":"https://api.github.com/users/rvagg/orgs","repos_url":"https://api.github.com/users/rvagg/repos","events_url":"https://api.github.com/users/rvagg/events{/privacy}","received_events_url":"https://api.github.com/users/rvagg/received_events","type":"User","site_admin":false},"created_at":"2015-03-19T00:29:16Z","updated_at":"2015-03-19T00:29:16Z","body":"@jbergstroem ptal, I've changed the logic around the default fallback"}}