Revert "Add frozen_string_literal to all files and some static code cleanup"

This reverts commit 62dd7b3.

Author: Geophilus Durairaj

Gemfile

@@ -1,14 +1,12 @@
-# frozen_string_literal: true
-
 source 'https://rubygems.org'
 
 gem 'rake', require: false
 gem 'rspec', '~> 3.0'
 
 group :docs do
-  gem 'rdiscount', require: false
   gem 'yard'
   gem 'yard-sitemap', '~> 1.0'
+  gem 'rdiscount', require: false
 end
 
 group :release do

Rakefile

@@ -1,8 +1,6 @@
-# frozen_string_literal: true
-
 $GEM_ROOT = File.dirname(__FILE__)
 
-$LOAD_PATH << File.join($GEM_ROOT, 'lib')
+$: << File.join($GEM_ROOT, 'lib')
 
 $VERSION = ENV['VERSION'] || File.read(File.join($GEM_ROOT, 'VERSION'))
 $GITHUB_ACCESS_TOKEN = ENV['JMESPATH_GITHUB_ACCESS_TOKEN']

bin/jmespath.rb

@@ -1,12 +1,11 @@
 #!/usr/bin/env ruby
-# frozen_string_literal: true
 
-$LOAD_PATH.unshift(File.join(File.dirname(__FILE__), '..', 'lib'))
+$:.unshift(File.join(File.dirname(__FILE__), '..', 'lib'))
 
 require 'jmespath'
 require 'json'
 
 expression = ARGV[0]
-json = JSON.parse($stdin.read)
+json = JSON.parse(STDIN.read)
 
 $stdout.puts(JSON.dump(JMESPath.search(expression, json)))

gemfiles/json-latest.gemfile

@@ -1,5 +1,3 @@
-# frozen_string_literal: true
-
-eval_gemfile(File.expand_path('../Gemfile', __dir__))
+eval_gemfile(File.expand_path('../../Gemfile', __FILE__))
 
 gem 'json'

gemfiles/json-old.gemfile

@@ -1,5 +1,3 @@
-# frozen_string_literal: true
-
-eval_gemfile(File.expand_path('../Gemfile', __dir__))
+eval_gemfile(File.expand_path('../../Gemfile', __FILE__))
 
 gem 'json', '< 2.0'

gemfiles/json-pure.gemfile

@@ -1,5 +1,3 @@
-# frozen_string_literal: true
-
-eval_gemfile(File.expand_path('../Gemfile', __dir__))
+eval_gemfile(File.expand_path('../../Gemfile', __FILE__))
 
 gem 'json_pure', require: 'json/pure'

gemfiles/no-deps.gemfile

@@ -1,5 +1,3 @@
-# frozen_string_literal: true
-
-eval_gemfile(File.expand_path('../Gemfile', __dir__))
+eval_gemfile(File.expand_path('../../Gemfile', __FILE__))
 
 # no dependency on `json` gem

jmespath.gemspec

@@ -1,8 +1,6 @@
-# frozen_string_literal: true
-
 Gem::Specification.new do |spec|
   spec.name          = 'jmespath'
-  spec.version       = File.read(File.expand_path('VERSION', __dir__)).strip
+  spec.version       = File.read(File.expand_path('../VERSION', __FILE__)).strip
   spec.summary       = 'JMESPath - Ruby Edition'
   spec.description   = 'Implements JMESPath for Ruby'
   spec.author        = 'Trevor Rowe'

lib/jmespath.rb

@@ -1,10 +1,9 @@
-# frozen_string_literal: true
-
 require 'json'
 require 'stringio'
 require 'pathname'
 
 module JMESPath
+
   require 'jmespath/caching_parser'
   require 'jmespath/errors'
   require 'jmespath/lexer'
@@ -17,24 +16,26 @@ module JMESPath
   require 'jmespath/version'
 
   class << self
+
     # @param [String] expression A valid
     #   [JMESPath](https://github.com/boto/jmespath) expression.
     # @param [Hash] data
     # @return [Mixed,nil] Returns the matched values. Returns `nil` if the
     #   expression does not resolve inside `data`.
     def search(expression, data, runtime_options = {})
       data = case data
-             when Hash, Struct then data # check for most common case first
-             when Pathname then load_json(data)
-             when IO, StringIO then JSON.parse(data.read)
-             else data
-             end
+        when Hash, Struct then data # check for most common case first
+        when Pathname then load_json(data)
+        when IO, StringIO then JSON.parse(data.read)
+        else data
+        end
       Runtime.new(runtime_options).search(expression, data)
     end
 
     # @api private
     def load_json(path)
-      JSON.parse(File.open(path, 'r', encoding: 'UTF-8', &:read))
+      JSON.parse(File.open(path, 'r', encoding: 'UTF-8') { |f| f.read })
     end
+
   end
 end

lib/jmespath/caching_parser.rb

@@ -1,7 +1,8 @@
-# frozen_string_literal: true
+require 'thread'
 
 module JMESPath
   class CachingParser
+
     def initialize(options = {})
       @parser = options[:parser] || Parser.new(options)
       @mutex = Mutex.new
@@ -24,5 +25,6 @@ def cache_expression(expression)
         @cache[expression] = @parser.parse(expression)
       end
     end
+
   end
 end