{"action":"created","issue":{"url":"https://api.github.com/repos/rails/sprockets/issues/87","labels_url":"https://api.github.com/repos/rails/sprockets/issues/87/labels{/name}","comments_url":"https://api.github.com/repos/rails/sprockets/issues/87/comments","events_url":"https://api.github.com/repos/rails/sprockets/issues/87/events","html_url":"https://github.com/rails/sprockets/pull/87","id":99611715,"number":87,"title":"Save String allocations by freezing ❄️ them (3.0 series)","user":{"login":"elia","id":1051,"avatar_url":"https://avatars.githubusercontent.com/u/1051?v=3","gravatar_id":"","url":"https://api.github.com/users/elia","html_url":"https://github.com/elia","followers_url":"https://api.github.com/users/elia/followers","following_url":"https://api.github.com/users/elia/following{/other_user}","gists_url":"https://api.github.com/users/elia/gists{/gist_id}","starred_url":"https://api.github.com/users/elia/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/elia/subscriptions","organizations_url":"https://api.github.com/users/elia/orgs","repos_url":"https://api.github.com/users/elia/repos","events_url":"https://api.github.com/users/elia/events{/privacy}","received_events_url":"https://api.github.com/users/elia/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-08-07T09:13:45Z","updated_at":"2015-08-07T13:13:45Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/rails/sprockets/pulls/87","html_url":"https://github.com/rails/sprockets/pull/87","diff_url":"https://github.com/rails/sprockets/pull/87.diff","patch_url":"https://github.com/rails/sprockets/pull/87.patch"},"body":"Inspired by the great work done by @schneems on rails.\r\n\r\nHere are some numbers taken from the current (Spree) app I’m working on:\r\n\r\n```\r\n'file-digest'   39k > 11k\r\n'\\1'            11k > 59\r\n'NilClass'      10k > (less than 24)\r\n```\r\n\r\nPS. if this is ok I'll port the changes to master too"},"comment":{"url":"https://api.github.com/repos/rails/sprockets/issues/comments/128696067","html_url":"https://github.com/rails/sprockets/pull/87#issuecomment-128696067","issue_url":"https://api.github.com/repos/rails/sprockets/issues/87","id":128696067,"user":{"login":"elia","id":1051,"avatar_url":"https://avatars.githubusercontent.com/u/1051?v=3","gravatar_id":"","url":"https://api.github.com/users/elia","html_url":"https://github.com/elia","followers_url":"https://api.github.com/users/elia/followers","following_url":"https://api.github.com/users/elia/following{/other_user}","gists_url":"https://api.github.com/users/elia/gists{/gist_id}","starred_url":"https://api.github.com/users/elia/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/elia/subscriptions","organizations_url":"https://api.github.com/users/elia/orgs","repos_url":"https://api.github.com/users/elia/repos","events_url":"https://api.github.com/users/elia/events{/privacy}","received_events_url":"https://api.github.com/users/elia/received_events","type":"User","site_admin":false},"created_at":"2015-08-07T13:13:45Z","updated_at":"2015-08-07T13:13:45Z","body":"I'll try to demonstrate the value through benchmarks, but that could take a while (on my part 😕).\r\n\r\nOf course this change is not going to make an app 2x faster and would probably account only for a tiny improvement. Nonetheless I think for a library (vs. application) the numbers shown should be a reason good enough. I found an example of such changes in commit a6bf1f97236283e89769f6af59f3b48a297b738b by @tenderlove.\r\n\r\nWould it be better if I reduce the scope of the change to just the three cases I shown the numbers for?\r\n"}}