Merge pull request #106 from naitoh/frozen_string_update

Update frozen string literals

Author: naitoh

lib/rbpdf.rb

@@ -2648,7 +2648,7 @@ def AddFont(family, style='', fontfile='', subset=nil)
       family = 'helvetica';
     end
     if (family == "symbol") or (family == "zapfdingbats")
-      style = +''
+      style = ''
     end
 
     tempstyle = style.upcase
@@ -5695,7 +5695,7 @@ def destroy(destroyall=false, preserve_objcopy=false)
     self.instance_variables.each { |val|
       if destroyall or ((val != '@internal_encoding') and (val != '@state') and (val != '@bufferlen') and (val != '@buffer') and (val != '@diskcache') and (val != '@sign') and (val != '@signature_data') and (val != '@signature_max_length') and (val != '@byterange_string'))
         if (!preserve_objcopy or (val.to_s != '@objcopy')) and !val.nil?
-          eval("#{val} = nil")
+          instance_variable_set(val, nil)
         end
       end
     }
@@ -12046,7 +12046,7 @@ def putshaders()
           out = +'<<'
           out << ' /FunctionType 3'
           out << ' /Domain [0 1]'
-          functions = ''
+          functions = +''
           i = 1
           num_cols = grad['colors'].size
           1.upto(num_cols - 1) do |i|
@@ -12522,7 +12522,6 @@ def isValidCSSSelectorForTag(dom, key, selector)
     if dom[key]['attribute'] and dom[key]['attribute']['class'] and !dom[key]['attribute']['class'].empty?
       selector_class = dom[key]['attribute']['class'].downcase
     end
-    id = ''
     if dom[key]['attribute'] and dom[key]['attribute']['id'] and !dom[key]['attribute']['id'].empty?
       selector_id = dom[key]['attribute']['id'].downcase
     end
@@ -12709,7 +12708,7 @@ def getHtmlDomArray(html)
             uri.split('/').each {|path|
               uri_path = uri_path.join(path)
             }
-            cssdata = ''
+            cssdata = +''
             next unless File.exist?(uri_path)
 
             open(uri_path) do |f|
@@ -13080,7 +13079,7 @@ def getHtmlDomArray(html)
               end
             end
             # font style
-            dom[key]['fontstyle'] ||= ""
+            dom[key]['fontstyle'] ||= +""
             if !dom[key]['style']['font-weight'].nil? and (dom[key]['style']['font-weight'][0,1].downcase == 'b')
               dom[key]['fontstyle'] << 'B'
             end
@@ -13618,8 +13617,8 @@ def writeHTML(html, ln=true, fill=0, reseth=false, cell=false, align='')
             # restore previous object
             rollbackTransaction(true)
             # restore previous values
-            this_method_vars.each {|vkey , vval|
-              eval("#{vkey} = vval")
+            this_method_vars.each {|vkey, vval|
+              binding.local_variable_set(vkey, vval)
             }
             # add a page (or trig AcceptPageBreak() for multicolumn mode)
             pre_y = @y
@@ -14482,7 +14481,7 @@ def writeHTML(html, ln=true, fill=0, reseth=false, cell=false, align='')
           rollbackTransaction(true)
           # restore previous values
           this_method_vars.each {|vkey , vval|
-            eval("#{vkey} = vval")
+            binding.local_variable_set(vkey, vval)
           }
           # add a page (or trig AcceptPageBreak() for multicolumn mode)
           pre_y = @y
@@ -16788,7 +16787,7 @@ def rollbackTransaction(this_self=false)
       if this_self
         objvars = @objcopy.instance_variables
         objvars.each {|key|
-          eval("#{key} = @objcopy.instance_variable_get(key)") if key.to_s != '@objcopy'
+          instance_variable_set(key, @objcopy.instance_variable_get(key)) if key.to_s != '@objcopy'
         }
       end
       return @objcopy