Patch-set looks great Eric, thanks! I'm hoping to test this in production later this week or next, but we're a year behind on Unicorn (ugh), so will need to carefully roll that out.
On Tue, Mar 7, 2017 at 7:26 PM, Eric Wong <e...@80x24.org> wrote: > Eric Wong <e...@80x24.org> wrote: >> Simon Eskildsen <simon.eskild...@shopify.com> wrote: >> > @@ -28,6 +29,7 @@ class Unicorn::HttpParser >> > # Drop these frozen strings when Ruby 2.2 becomes more prevalent, >> > # 2.2+ optimizes hash assignments when used with literal string keys >> > HTTP_RESPONSE_START = [ 'HTTP', '/1.1 '] >> > + EMPTY_ARRAY = [].freeze >> >> (minor) this was made before commit e06b699683738f22 >> ("http_request: freeze constant strings passed IO#write") >> but I've trivially fixed it up, locally. > > And I actually screwed it up, pushed out ccc-tcp-v2 branch > with that fix squashed in :x > > Writing tests, now...