Enforcing frozen string literals with pragma comment.

Gemfile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 source 'https://rubygems.org'
 
 # Specify your gem's dependencies in parser.gemspec

Rakefile

@@ -1,4 +1,5 @@
-# encoding:utf-8
+# encoding: utf-8
+# frozen_string_literal: true
 
 require 'bundler/gem_tasks'
 require 'rake/testtask'

bin/ruby-parse

@@ -1,4 +1,5 @@
-#!/usr/bin/env ruby
+#! /usr/bin/env ruby
+# frozen_string_literal: true
 
 $LOAD_PATH.unshift(File.expand_path('../../lib', __FILE__))
 require 'parser/runner/ruby_parse'

bin/ruby-rewrite

@@ -1,4 +1,5 @@
-#!/usr/bin/env ruby
+#! /usr/bin/env ruby
+# frozen_string_literal: true
 
 $LOAD_PATH.unshift(File.expand_path('../../lib', __FILE__))
 require 'parser/runner/ruby_rewrite'

lib/gauntlet_parser.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'gauntlet'
 require 'parser/all'
 require 'shellwords'

lib/parser.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 if RUBY_VERSION =~ /^1\.[89]\./
   require 'parser/version'
   raise LoadError, <<-UNSUPPORTED_VERSION_MSG

lib/parser/all.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'parser/ruby18'
 require 'parser/ruby19'
 require 'parser/ruby20'

lib/parser/ast/node.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Parser
   module AST
 

lib/parser/ast/processor.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Parser
   module AST
 

lib/parser/base.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Parser
 
   ##

lib/parser/builders/default.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Parser
 
   ##

lib/parser/clobbering_error.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Parser
   ##
   # {Parser::ClobberingError} is raised when {Parser::Source::Rewriter}

lib/parser/context.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Parser
   # Context of parsing that is represented by a stack of scopes.
   #

lib/parser/current.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Parser
   class << self
     def warn_syntax_deviation(feature, version)

lib/parser/deprecation.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Parser
   ##
   # @api private

lib/parser/diagnostic.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Parser
 
   ##

lib/parser/diagnostic/engine.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Parser
 
   ##

lib/parser/lexer/dedenter.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Parser
 
   class Lexer::Dedenter

lib/parser/lexer/explanation.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Parser
 
   module Lexer::Explanation

lib/parser/lexer/literal.rb

@@ -1,4 +1,5 @@
 # encoding: binary
+# frozen_string_literal: true
 
 module Parser
 
@@ -233,7 +234,7 @@ def coerce_encoding(string)
     end
 
     def clear_buffer
-      @buffer = ''
+      @buffer = ''.dup
 
       # Prime the buffer with lexer encoding; otherwise,
       # concatenation will produce varying results.

lib/parser/lexer/stack_state.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Parser
 
   class Lexer::StackState

lib/parser/messages.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Parser
   ##
   # Diagnostic messages (errors, warnings and notices) that can be generated.

lib/parser/meta.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Parser
   # Parser metadata
   module Meta

lib/parser/rewriter.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Parser
 
   ##

lib/parser/runner.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'benchmark'
 require 'find'
 require 'optparse'

lib/parser/runner/ruby_parse.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'parser/runner'
 require 'parser/lexer/explanation'
 

lib/parser/runner/ruby_rewrite.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'parser/runner'
 require 'tempfile'
 

lib/parser/source/buffer.rb

@@ -1,4 +1,5 @@
 # encoding: ascii-8bit
+# frozen_string_literal: true
 
 module Parser
   module Source
@@ -243,7 +244,7 @@ def column_for_position(position)
       def source_lines
         @lines ||= begin
           lines = @source.lines.to_a
-          lines << '' if @source.end_with?("\n".freeze)
+          lines << ''.dup if @source.end_with?("\n".freeze)
 
           lines.each do |line|
             line.chomp!("\n".freeze)

lib/parser/source/comment.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Parser
   module Source
 

lib/parser/source/comment/associator.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Parser
   module Source
 

lib/parser/source/map.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Parser
   module Source
 

lib/parser/source/map/collection.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Parser
   module Source
 

lib/parser/source/map/condition.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Parser
   module Source
 

lib/parser/source/map/constant.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Parser
   module Source
 

lib/parser/source/map/definition.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Parser
   module Source
 

lib/parser/source/map/for.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Parser
   module Source
 

lib/parser/source/map/heredoc.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Parser
   module Source
 

lib/parser/source/map/keyword.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Parser
   module Source
 

lib/parser/source/map/objc_kwarg.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Parser
   module Source
 

lib/parser/source/map/operator.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Parser
   module Source
 

lib/parser/source/map/rescue_body.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Parser
   module Source
 

lib/parser/source/map/send.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Parser
   module Source
 

lib/parser/source/map/ternary.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Parser
   module Source
 

lib/parser/source/map/variable.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Parser
   module Source
 

lib/parser/source/range.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Parser
   module Source
 

lib/parser/source/rewriter.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Parser
   module Source
 
@@ -421,7 +423,7 @@ def merge_actions!(action, existing)
       end
 
       def merge_replacements(actions)
-        result    = ''
+        result    = ''.dup
         prev_act  = nil
 
         actions.each do |act|

lib/parser/source/rewriter/action.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Parser
   module Source
 

lib/parser/source/tree_rewriter.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Parser
   module Source
 

lib/parser/source/tree_rewriter/action.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Parser
   module Source
     ##

lib/parser/static_environment.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Parser
 
   class StaticEnvironment

lib/parser/syntax_error.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Parser
   ##
   # {Parser::SyntaxError} is raised whenever parser detects a syntax error,

lib/parser/tree_rewriter.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Parser
 
   ##

lib/parser/version.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Parser
   VERSION = '2.4.0.2'
 end

parser.gemspec

@@ -1,4 +1,5 @@
 # encoding: utf-8
+# frozen_string_literal: true
 
 require File.expand_path('../lib/parser/version', __FILE__)
 

test/bug_163/fixtures/input.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 if(true)
   puts "Hello, world!"
 end

test/bug_163/fixtures/output.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 if true
   puts "Hello, world!"
 end

test/bug_163/rewriter.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class Rewriter < Parser::Rewriter
   def on_if(node)
     # Crude, totally-not-usable-in-the-real-world code to remove optional

test/helper.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'tempfile'
 require 'minitest/test'
 

test/parse_helper.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ParseHelper
   include AST::Sexp
 

test/racc_coverage_helper.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'racc/grammarfileparser'
 
 # Unfortunately, Ruby's Coverage module ignores module_eval statements,

test/test_base.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'helper'
 require 'parser/current'