Merge pull request #1649 from puma/schneems/frozen

schneems · web-flow · commit 2668597ec1dd · 2018-10-17T16:16:29.000-05:00
Freeze all the strings!
diff --git a/lib/puma/binder.rb b/lib/puma/binder.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'uri'
 require 'socket'
 
diff --git a/lib/puma/cli.rb b/lib/puma/cli.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'optparse'
 require 'uri'
 
diff --git a/lib/puma/client.rb b/lib/puma/client.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class IO
   # We need to use this for a jruby work around on both 1.8 and 1.9.
   # So this either creates the constant (on 1.8), or harmlessly
diff --git a/lib/puma/cluster.rb b/lib/puma/cluster.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'puma/runner'
 require 'puma/util'
 require 'puma/plugin'
diff --git a/lib/puma/commonlogger.rb b/lib/puma/commonlogger.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Puma
   # Rack::CommonLogger forwards every request to the given +app+, and
   # logs a line in the
diff --git a/lib/puma/configuration.rb b/lib/puma/configuration.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'puma/rack/builder'
 require 'puma/plugin'
 require 'puma/const'
diff --git a/lib/puma/const.rb b/lib/puma/const.rb
@@ -1,4 +1,6 @@
 #encoding: utf-8
+# frozen_string_literal: true
+
 module Puma
   class UnsupportedOption < RuntimeError
   end
diff --git a/lib/puma/control_cli.rb b/lib/puma/control_cli.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'optparse'
 require_relative 'state_file'
 require_relative 'const'
diff --git a/lib/puma/convenient.rb b/lib/puma/convenient.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'puma/launcher'
 require 'puma/configuration'
 
diff --git a/lib/puma/daemon_ext.rb b/lib/puma/daemon_ext.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Process
 
   # This overrides the default version because it is broken if it
diff --git a/lib/puma/delegation.rb b/lib/puma/delegation.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Puma
   module Delegation
     def forward(what, who)
diff --git a/lib/puma/detect.rb b/lib/puma/detect.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Puma
   IS_JRUBY = defined?(JRUBY_VERSION)
 
diff --git a/lib/puma/dsl.rb b/lib/puma/dsl.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Puma
   # The methods that are available for use inside the config file.
   # These same methods are used in Puma cli and the rack handler
diff --git a/lib/puma/events.rb b/lib/puma/events.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'puma/const'
 require "puma/null_io"
 require 'stringio'
diff --git a/lib/puma/io_buffer.rb b/lib/puma/io_buffer.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'puma/detect'
 
 if Puma.jruby?
diff --git a/lib/puma/java_io_buffer.rb b/lib/puma/java_io_buffer.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'java'
 
 # Conservative native JRuby/Java implementation of IOBuffer
diff --git a/lib/puma/jruby_restart.rb b/lib/puma/jruby_restart.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'ffi'
 
 module Puma
diff --git a/lib/puma/launcher.rb b/lib/puma/launcher.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'puma/events'
 require 'puma/detect'
 
diff --git a/lib/puma/minissl.rb b/lib/puma/minissl.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 begin
   require 'io/wait'
   rescue LoadError
diff --git a/lib/puma/null_io.rb b/lib/puma/null_io.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Puma
   # Provides an IO-like object that always appears to contain no data.
   # Used as the value for rack.input when the request has no body.
diff --git a/lib/puma/plugin.rb b/lib/puma/plugin.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Puma
   class UnknownPlugin < RuntimeError; end
 
diff --git a/lib/puma/reactor.rb b/lib/puma/reactor.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'puma/util'
 require 'puma/minissl'
 
diff --git a/lib/puma/runner.rb b/lib/puma/runner.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'puma/server'
 require 'puma/const'
 
diff --git a/lib/puma/server.rb b/lib/puma/server.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'stringio'
 
 require 'puma/thread_pool'
diff --git a/lib/puma/single.rb b/lib/puma/single.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'puma/runner'
 require 'puma/detect'
 require 'puma/plugin'
diff --git a/lib/puma/state_file.rb b/lib/puma/state_file.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'yaml'
 
 module Puma
diff --git a/lib/puma/tcp_logger.rb b/lib/puma/tcp_logger.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Puma
   class TCPLogger
     def initialize(logger, app, quiet=false)
diff --git a/lib/puma/thread_pool.rb b/lib/puma/thread_pool.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'thread'
 
 module Puma
diff --git a/lib/puma/util.rb b/lib/puma/util.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 major, minor, patch = RUBY_VERSION.split('.').map { |v| v.to_i }
 
 if major == 1 && minor == 9 && patch == 3 && RUBY_PATCHLEVEL < 125

Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,5 @@`
	`1`	`+# frozen_string_literal: true`
	`2`	`+`
`1`	`3`	`require 'uri'`
`2`	`4`	`require 'socket'`
`3`	`5`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,5 @@`
	`1`	`+# frozen_string_literal: true`
	`2`	`+`
`1`	`3`	`require 'optparse'`
`2`	`4`	`require 'uri'`
`3`	`5`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,5 @@`
	`1`	`+# frozen_string_literal: true`
	`2`	`+`
`1`	`3`	`class IO`
`2`	`4`	`# We need to use this for a jruby work around on both 1.8 and 1.9.`
`3`	`5`	`# So this either creates the constant (on 1.8), or harmlessly`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,5 @@`
	`1`	`+# frozen_string_literal: true`
	`2`	`+`
`1`	`3`	`require 'puma/runner'`
`2`	`4`	`require 'puma/util'`
`3`	`5`	`require 'puma/plugin'`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,5 @@`
	`1`	`+# frozen_string_literal: true`
	`2`	`+`
`1`	`3`	`module Puma`
`2`	`4`	`# Rack::CommonLogger forwards every request to the given +app+, and`
`3`	`5`	`# logs a line in the`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,5 @@`
	`1`	`+# frozen_string_literal: true`
	`2`	`+`
`1`	`3`	`require 'puma/rack/builder'`
`2`	`4`	`require 'puma/plugin'`
`3`	`5`	`require 'puma/const'`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,5 @@`
	`1`	`+# frozen_string_literal: true`
	`2`	`+`
`1`	`3`	`require 'puma/launcher'`
`2`	`4`	`require 'puma/configuration'`
`3`	`5`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,5 @@`
	`1`	`+# frozen_string_literal: true`
	`2`	`+`
`1`	`3`	`module Process`
`2`	`4`
`3`	`5`	`# This overrides the default version because it is broken if it`