Update core files containing code in the Truffle module to use frozen String literals.

Author: nirvdrum

src/main/ruby/core/truffle/boot.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Copyright (c) 2016, 2017 Oracle and/or its affiliates. All rights reserved. This
 # code is released under a tri EPL/GPL/LGPL license. You can use it,
 # redistribute it and/or modify it under the terms of the:

src/main/ruby/core/truffle/exception_operations.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Copyright (c) 2017 Oracle and/or its affiliates. All rights reserved. This
 # code is released under a tri EPL/GPL/LGPL license. You can use it,
 # redistribute it and/or modify it under the terms of the:

src/main/ruby/core/truffle/ffi/pointer.rb

@@ -1,4 +1,6 @@
-# Copyright (c) 2016, 2017 Oracle and/or its affiliates. All rights reserved. This
+# frozen_string_literal: true
+
+# Copyright (c) 2016, 2018 Oracle and/or its affiliates. All rights reserved. This
 # code is released under a tri EPL/GPL/LGPL license. You can use it,
 # redistribute it and/or modify it under the terms of the:
 #
@@ -93,7 +95,7 @@ def inspect
     # Return the address pointed to as an Integer
     def address
       Truffle.primitive :pointer_address
-      raise PrimitiveFailure, 'FFI::Pointer#address primitive failed'
+      raise PrimitiveFailure, +'FFI::Pointer#address primitive failed'
     end
 
     alias_method :to_i, :address
@@ -110,7 +112,7 @@ def to_native
     # Set the address pointed to from an Integer
     def address=(address)
       Truffle.primitive :pointer_set_address
-      raise PrimitiveFailure, 'FFI::Pointer#address= primitive failed'
+      raise PrimitiveFailure, +'FFI::Pointer#address= primitive failed'
     end
 
     def null?
@@ -120,7 +122,7 @@ def null?
     # Add +value+ to the address pointed to and return a new Pointer
     def +(value)
       Truffle.primitive :pointer_add
-      raise PrimitiveFailure, 'FFI::Pointer#+ primitive failed'
+      raise PrimitiveFailure, +'FFI::Pointer#+ primitive failed'
     end
 
     # Indicates if +self+ and +other+ point to the same address
@@ -131,22 +133,22 @@ def ==(other)
 
     def network_order(start, size)
       Truffle.primitive :pointer_network_order
-      raise PrimitiveFailure, 'FFI::Pointer#network_order primitive failed'
+      raise PrimitiveFailure, +'FFI::Pointer#network_order primitive failed'
     end
 
     # Read +len+ bytes from the memory pointed to and return them as
     # a String
     def read_string_length(len)
       Truffle.primitive :pointer_read_string
-      raise PrimitiveFailure, 'FFI::Pointer#read_string_length primitive failed'
+      raise PrimitiveFailure, +'FFI::Pointer#read_string_length primitive failed'
     end
     alias :read_bytes :read_string_length
 
     # Read bytes from the memory pointed to until a NULL is seen, return
     # the bytes as a String
     def read_string_to_null
       Truffle.primitive :pointer_read_string_to_null
-      raise PrimitiveFailure, 'FFI::Pointer#read_string_to_null primitive failed'
+      raise PrimitiveFailure, +'FFI::Pointer#read_string_to_null primitive failed'
     end
 
     # Read bytes as a String from the memory pointed to
@@ -167,7 +169,7 @@ def get_bytes(offset, length)
     # write up to +len+ bytes.
     def write_string_length(str, len)
       Truffle.primitive :pointer_write_string
-      raise PrimitiveFailure, 'FFI::Pointer#write_string_length primitive failed'
+      raise PrimitiveFailure, +'FFI::Pointer#write_string_length primitive failed'
     end
 
     # Write a String +str+ as bytes to the memory pointed to.
@@ -180,13 +182,13 @@ def write_string(str, len=nil)
     # Read bytes from +offset+ from the memory pointed to as type +type+
     def get_at_offset(offset, type)
       Truffle.primitive :pointer_get_at_offset
-      raise PrimitiveFailure, 'FFI::Pointer#get_at_offset primitive failed'
+      raise PrimitiveFailure, +'FFI::Pointer#get_at_offset primitive failed'
     end
 
     # Write +val+ as type +type+ to bytes from +offset+
     def set_at_offset(offset, type, val)
       Truffle.primitive :pointer_set_at_offset
-      raise PrimitiveFailure, 'FFI::Pointer#set_at_offset primitive failed'
+      raise PrimitiveFailure, +'FFI::Pointer#set_at_offset primitive failed'
     end
 
     # Number of bytes taken up by a pointer.
@@ -199,14 +201,14 @@ def self.size
     # free() on it's address when it is garbage collected.
     def autorelease=(val)
       Truffle.primitive :pointer_set_autorelease
-      raise PrimitiveFailure, 'FFI::Pointer#autorelease= primitive failed'
+      raise PrimitiveFailure, +'FFI::Pointer#autorelease= primitive failed'
     end
 
     ##
     # Returns true if autorelease is enabled, otherwise false.
     def autorelease?
       Truffle.primitive :pointer_autorelease_p
-      raise PrimitiveFailure, 'FFI::Pointer#pointer_autorelease_p primitive failed'
+      raise PrimitiveFailure, +'FFI::Pointer#pointer_autorelease_p primitive failed'
     end
 
     NULL = Pointer.new(0x0)
@@ -270,7 +272,7 @@ def self.new(type, count=nil, clear=true)
 
     def self.malloc(total)
       Truffle.primitive :pointer_malloc
-      raise PrimitiveFailure, 'FFI::MemoryPointer.malloc primitive failed'
+      raise PrimitiveFailure, +'FFI::MemoryPointer.malloc primitive failed'
     end
 
     def self.from_string(str)
@@ -314,14 +316,14 @@ def copy
     #   new_ptr = &ptr[9];
     #
     def [](which)
-      raise ArgumentError, 'unknown type size' unless @type_size
+      raise ArgumentError, +'unknown type size' unless @type_size
       self + (which * @type_size)
     end
 
     # Release the memory pointed to back to the OS.
     def free
       Truffle.primitive :pointer_free
-      raise PrimitiveFailure, 'FFI::MemoryPointer#free primitive failed'
+      raise PrimitiveFailure, +'FFI::MemoryPointer#free primitive failed'
     end
   end
 end

src/main/ruby/core/truffle/internal.rb

@@ -1,4 +1,6 @@
-# Copyright (c) 2014, 2017 Oracle and/or its affiliates. All rights reserved. This
+# frozen_string_literal: true
+
+# Copyright (c) 2014, 2018 Oracle and/or its affiliates. All rights reserved. This
 # code is released under a tri EPL/GPL/LGPL license. You can use it,
 # redistribute it and/or modify it under the terms of the:
 #
@@ -39,7 +41,7 @@ def self.set_class(obj, cls)
       Truffle.primitive :vm_set_class
 
       if obj.kind_of? ImmediateValue
-        raise TypeError, 'Can not change the class of an immediate'
+        raise TypeError, +'Can not change the class of an immediate'
       end
 
       raise ArgumentError, "Class #{cls} is not compatible with #{obj.inspect}"

src/main/ruby/core/truffle/interop.rb

@@ -1,4 +1,6 @@
-# Copyright (c) 2015, 2017 Oracle and/or its affiliates. All rights reserved. This
+# frozen_string_literal: true
+
+# Copyright (c) 2015, 2018 Oracle and/or its affiliates. All rights reserved. This
 # code is released under a tri EPL/GPL/LGPL license. You can use it,
 # redistribute it and/or modify it under the terms of the:
 #
@@ -185,7 +187,7 @@ def self.special_form(receiver, name, *args)
         end
       when :to_str
         receiver = Truffle::Interop.unbox_if_needed(receiver)
-        raise NameError, 'no method to_str' unless receiver.is_a?(String)
+        raise NameError, +'no method to_str' unless receiver.is_a?(String)
         receiver
       when :is_a?, :kind_of?
         receiver = Truffle::Interop.unbox_if_needed(receiver)
@@ -196,7 +198,7 @@ def self.special_form(receiver, name, *args)
             Truffle::Interop.java_instanceof?(receiver, check_class)
           elsif Truffle::Interop.foreign?(check_class)
             # Checking a foreign (not Java) object against a foreign (not Java) class
-            raise TypeError, 'cannot check if a foreign object is an instance of a foreign class'
+            raise TypeError, +'cannot check if a foreign object is an instance of a foreign class'
           else
             # Checking a foreign or Java object against a Ruby class
             false
@@ -216,7 +218,7 @@ def self.inspect_foreign(object)
       if object.is_a?(String)
         object.inspect
       elsif Truffle::Interop.java?(object) && object.nil?
-        '#<Java null>'
+        +'#<Java null>'
       elsif Truffle::Interop.java?(object) && object.respond_to?(:size)
         "#<Java:#{hash_code} #{to_array(object).inspect}>"
       elsif Truffle::Interop.java?(object) && object.is_a?(::Java.type('java.util.Map'))
@@ -226,7 +228,7 @@ def self.inspect_foreign(object)
       elsif Truffle::Interop.java?(object)
         "#<Java:#{hash_code} object #{object.getClass.getName}>"
       elsif Truffle::Interop.null?(object)
-        '#<Foreign null>'
+        +'#<Foreign null>'
       elsif Truffle::Interop.pointer?(object)
         "#<Foreign pointer 0x#{Truffle::Interop.as_pointer(object).to_s(16)}>"
       elsif Truffle::Interop.size?(object)
@@ -266,7 +268,7 @@ def self.respond_to?(object, name)
 
     def self.to_array(object)
       unless Truffle::Interop.size?(object)
-        raise 'foreign object does not have a size to turn it into an array'
+        raise(+'foreign object does not have a size to turn it into an array')
       end
 
       Array.new(Truffle::Interop.size(object)) do |n|

src/main/ruby/core/truffle/io_operations.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Copyright (c) 2018 Oracle and/or its affiliates. All rights reserved. This
 # code is released under a tri EPL/GPL/LGPL license. You can use it,
 # redistribute it and/or modify it under the terms of the:
@@ -18,7 +20,7 @@ def self.set_last_line(value, a_binding)
 
     def self.print(io, args, last_line_binding)
       if args.empty?
-        raise 'last_line_binding is required' if last_line_binding.nil?
+        raise(+'last_line_binding is required') if last_line_binding.nil?
         io.write Truffle::IOOperations.last_line(last_line_binding).to_s
       else
         args.each { |o| io.write o.to_s }

src/main/ruby/core/truffle/kernel_operations.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Copyright (c) 2018 Oracle and/or its affiliates. All rights reserved. This
 # code is released under a tri EPL/GPL/LGPL license. You can use it,
 # redistribute it and/or modify it under the terms of the:
@@ -48,7 +50,7 @@ def self.define_read_only_global(name, getter)
       :$/,
       -> { global_variable_get(:$/) },
       -> v { if v && !Truffle::Type.object_kind_of?(v, String)
-               raise TypeError, '$/ must be a String'
+               raise TypeError, +'$/ must be a String'
              end
              global_variable_set(:$/, v) })
 
@@ -60,7 +62,7 @@ def self.define_read_only_global(name, getter)
       :'$,',
       -> { global_variable_get(:$,) },
       -> v { if v && !Truffle::Type.object_kind_of?(v, String)
-               raise TypeError, '$, must be a String'
+               raise TypeError, +'$, must be a String'
              end
              global_variable_set(:$,, v) })
 
@@ -115,19 +117,19 @@ def self.internal_raise(exc, msg, ctx, internal)
         if exc
           skip = true
         else
-          exc = RuntimeError.new('No current exception')
+          exc = RuntimeError.new(+'No current exception')
         end
       elsif exc.respond_to? :exception
         if undefined.equal? msg
           exc = exc.exception
         else
           exc = exc.exception msg
         end
-        raise TypeError, 'exception class/object expected' unless exc.kind_of?(Exception)
+        raise TypeError, +'exception class/object expected' unless exc.kind_of?(Exception)
       elsif exc.kind_of? String
         exc = RuntimeError.exception exc
       else
-        raise TypeError, 'exception class/object expected'
+        raise TypeError, +'exception class/object expected'
       end
 
       unless skip

src/main/ruby/core/truffle/numeric_operations.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Copyright (c) 2018 Oracle and/or its affiliates. All rights reserved. This
 # code is released under a tri EPL/GPL/LGPL license. You can use it,
 # redistribute it and/or modify it under the terms of the:
@@ -85,7 +87,7 @@ def self.stepping_forever?(limit, step, asc)
     end
 
     def self.step_fetch_args(value, limit, step, by)
-      raise ArgumentError, 'step cannot be 0' if undefined.equal?(by) && step == 0
+      raise ArgumentError, +'step cannot be 0' if undefined.equal?(by) && step == 0
 
       asc = step > 0
       if value.kind_of? Float or limit.kind_of? Float or step.kind_of? Float

src/main/ruby/core/truffle/platform.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Copyright (c) 2014, 2017 Oracle and/or its affiliates. All rights reserved. This
 # code is released under a tri EPL/GPL/LGPL license. You can use it,
 # redistribute it and/or modify it under the terms of the:

src/main/ruby/core/truffle/polyglot.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Copyright (c) 2018 Oracle and/or its affiliates. All rights reserved. This
 # code is released under a tri EPL/GPL/LGPL license. You can use it,
 # redistribute it and/or modify it under the terms of the: