{"action":"created","issue":{"url":"https://api.github.com/repos/iojs/io.js/issues/691","labels_url":"https://api.github.com/repos/iojs/io.js/issues/691/labels{/name}","comments_url":"https://api.github.com/repos/iojs/io.js/issues/691/comments","events_url":"https://api.github.com/repos/iojs/io.js/issues/691/events","html_url":"https://github.com/iojs/io.js/pull/691","id":56260883,"number":691,"title":"lib: replace sprintf-like format to template string literal","user":{"login":"yosuke-furukawa","id":555645,"avatar_url":"https://avatars.githubusercontent.com/u/555645?v=3","gravatar_id":"","url":"https://api.github.com/users/yosuke-furukawa","html_url":"https://github.com/yosuke-furukawa","followers_url":"https://api.github.com/users/yosuke-furukawa/followers","following_url":"https://api.github.com/users/yosuke-furukawa/following{/other_user}","gists_url":"https://api.github.com/users/yosuke-furukawa/gists{/gist_id}","starred_url":"https://api.github.com/users/yosuke-furukawa/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/yosuke-furukawa/subscriptions","organizations_url":"https://api.github.com/users/yosuke-furukawa/orgs","repos_url":"https://api.github.com/users/yosuke-furukawa/repos","events_url":"https://api.github.com/users/yosuke-furukawa/events{/privacy}","received_events_url":"https://api.github.com/users/yosuke-furukawa/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":7,"created_at":"2015-02-02T16:48:37Z","updated_at":"2015-02-03T17:29:29Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/iojs/io.js/pulls/691","html_url":"https://github.com/iojs/io.js/pull/691","diff_url":"https://github.com/iojs/io.js/pull/691.diff","patch_url":"https://github.com/iojs/io.js/pull/691.patch"},"body":"Template String Literal is faster than sprintf-like format string.\r\n\r\nhttp://jsperf.com/template-literal-vs-string-plus/2\r\n![2015-02-03 1 33 05](https://cloud.githubusercontent.com/assets/555645/6004240/a71bdab4-ab44-11e4-897d-5bf0d2ffb18c.png)\r\n\r\nAnd, I think template string literal is easier to understand than sprintf.\r\n"},"comment":{"url":"https://api.github.com/repos/iojs/io.js/issues/comments/72694898","html_url":"https://github.com/iojs/io.js/pull/691#issuecomment-72694898","issue_url":"https://api.github.com/repos/iojs/io.js/issues/691","id":72694898,"user":{"login":"yosuke-furukawa","id":555645,"avatar_url":"https://avatars.githubusercontent.com/u/555645?v=3","gravatar_id":"","url":"https://api.github.com/users/yosuke-furukawa","html_url":"https://github.com/yosuke-furukawa","followers_url":"https://api.github.com/users/yosuke-furukawa/followers","following_url":"https://api.github.com/users/yosuke-furukawa/following{/other_user}","gists_url":"https://api.github.com/users/yosuke-furukawa/gists{/gist_id}","starred_url":"https://api.github.com/users/yosuke-furukawa/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/yosuke-furukawa/subscriptions","organizations_url":"https://api.github.com/users/yosuke-furukawa/orgs","repos_url":"https://api.github.com/users/yosuke-furukawa/repos","events_url":"https://api.github.com/users/yosuke-furukawa/events{/privacy}","received_events_url":"https://api.github.com/users/yosuke-furukawa/received_events","type":"User","site_admin":false},"created_at":"2015-02-03T17:29:29Z","updated_at":"2015-02-03T17:29:29Z","body":"Thank you for comments. \r\n\r\nI think template string literals have 2 pros.\r\n1. faster than util.format.\r\n2. improve readability compared to string concatenation.\r\n\r\nSo I suggest this pull request.\r\n\r\n@bnoordhuis @charmander \r\nThanks for review.\r\nWhen I fixed your review comments, Closure linter reports `Missing space before \":\"` messages. the linter does not recognize ``${foo}: var`` style strings. I reported this issue to [closure linter](https://code.google.com/p/closure-linter/issues/detail?id=86). \r\n\r\nCurrently, I would like to replace some sprintf-like strings that are not reported by closure linter.\r\nWhen the issue of closure linter is fixed or we change our linter tool, I would like to replace all sprintf-like strings to template string literals."}}