Get rid of anonymous eval calls

Things declared in anonymous eval are always annoying to locate. (e.g. profling etc).
sparklemotion · Jan 11, 2023 · 50f78dc · 50f78dc
1 parent 6ea1449
commit 50f78dc
Show file tree

Hide file tree

Showing 7 changed files with 17 additions and 15 deletions.
diff --git a/lib/nokogiri/css/xpath_visitor.rb b/lib/nokogiri/css/xpath_visitor.rb
@@ -230,11 +230,11 @@ def visit_combinator(node)
  "descendant_selector" => "//",
  "child_selector" => "/",
  }.each do |k, v|
- class_eval %{
+ class_eval <<~RUBY, __FILE__, __LINE__ + 1
  def visit_#{k} node
  "\#{node.value.first.accept(self) if node.value.first}#{v}\#{node.value.last.accept(self)}"
  end
- }
+ RUBY
  end
 
  def visit_conditional_selector(node)

diff --git a/lib/nokogiri/xml/node/save_options.rb b/lib/nokogiri/xml/node/save_options.rb
@@ -49,7 +49,7 @@ def initialize(options = 0)
  end
 
  constants.each do |constant|
- class_eval %{
+ class_eval <<~RUBY, __FILE__, __LINE__ + 1
  def #{constant.downcase}
  @options |= #{constant}
  self
@@ -58,7 +58,7 @@ def #{constant.downcase}
  def #{constant.downcase}?
  #{constant} & @options == #{constant}
  end
- }
+ RUBY
  end
 
  alias_method :to_i, :options

diff --git a/lib/nokogiri/xml/parse_options.rb b/lib/nokogiri/xml/parse_options.rb
@@ -169,7 +169,7 @@ def initialize(options = STRICT)
  constants.each do |constant|
  next if constant.to_sym == :STRICT
 
- class_eval %{
+ class_eval <<~RUBY, __FILE__, __LINE__ + 1
  def #{constant.downcase}
  @options |= #{constant}
  self
@@ -183,7 +183,7 @@ def no#{constant.downcase}
  def #{constant.downcase}?
  #{constant} & @options == #{constant}
  end
- }
+ RUBY
  end
 
  def strict

diff --git a/lib/xsd/xmlparser/nokogiri.rb b/lib/xsd/xmlparser/nokogiri.rb
@@ -95,7 +95,9 @@ def end_element_namespace(name, prefix = nil, uri = nil)
  end
 
  ["xmldecl", "start_document", "end_document", "comment"].each do |name|
- class_eval %{ def #{name}(*args); end }
+ class_eval <<~RUBY, __FILE__, __LINE__ + 1
+ def #{name}(*args); end
+ RUBY
  end
 
  add_factory(self)

diff --git a/test/xml/node/test_save_options.rb b/test/xml/node/test_save_options.rb
@@ -7,14 +7,14 @@ module XML
  class Node
  class TestSaveOptions < Nokogiri::TestCase
  SaveOptions.constants.each do |constant|
- class_eval <<-EOEVAL
+ class_eval <<~RUBY, __FILE__, __LINE__ + 1
  def test_predicate_#{constant.downcase}
  options = SaveOptions.new(SaveOptions::#{constant})
  assert options.#{constant.downcase}?
 
  assert SaveOptions.new.#{constant.downcase}.#{constant.downcase}?
  end
- EOEVAL
+ RUBY
  end
 
  def test_default_xml_save_options

diff --git a/test/xml/node/test_subclass.rb b/test/xml/node/test_subclass.rb
@@ -16,16 +16,16 @@ class TestSubclass < Nokogiri::TestCase
  Nokogiri::XML::Node => '"foo", doc',
  Nokogiri::XML::Text => '"foo", doc',
  }.each do |klass, constructor|
- class_eval %{
+ class_eval <<~RUBY, __FILE__, __LINE__ + 1
  def test_subclass_#{klass.name.gsub("::", "_")}
  doc = Nokogiri::XML::Document.new
  klass = Class.new(#{klass.name})
  node = klass.new(#{constructor})
  assert_instance_of klass, node
  end
- }
+ RUBY
 
- class_eval <<-eocode, __FILE__, __LINE__ + 1
+ class_eval <<~RUBY, __FILE__, __LINE__ + 1
  def test_subclass_initialize_#{klass.name.gsub("::", "_")}
  doc = Nokogiri::XML::Document.new
  klass = Class.new(#{klass.name}) do
@@ -38,7 +38,7 @@ def initialize *args
  node = klass.new(#{constructor}, 1)
  assert_equal [#{constructor}, 1], node.initialized_with
  end
- eocode
+ RUBY
  end
  end
  end

diff --git a/test/xml/test_parse_options.rb b/test/xml/test_parse_options.rb
@@ -18,14 +18,14 @@ def test_to_i
  ParseOptions.constants.each do |constant|
  next if constant == "STRICT"
 
- class_eval %{
+ class_eval <<~RUBY, __FILE__, __LINE__ + 1
  def test_predicate_#{constant.downcase}
  options = ParseOptions.new(ParseOptions::#{constant})
  assert options.#{constant.downcase}?
 
  assert ParseOptions.new.#{constant.downcase}.#{constant.downcase}?
  end
- }
+ RUBY
  end
 
  def test_strict_noent