Skip to content

Commit 61b73c1

Browse files
committed
xpath: remove needless nil check
It must not be nil.
1 parent 70310a0 commit 61b73c1

File tree

1 file changed

+2
-6
lines changed

1 file changed

+2
-6
lines changed

lib/rexml/xpath_parser.rb

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -474,9 +474,7 @@ def node_test(path_stack, nodesets, any_type: :element)
474474
raw_node = node.raw_node
475475
case raw_node.node_type
476476
when :element
477-
if prefix.nil?
478-
raw_node.name == name
479-
elsif prefix.empty?
477+
if prefix.empty?
480478
if strict?
481479
raw_node.name == name and raw_node.namespace == ""
482480
else
@@ -490,9 +488,7 @@ def node_test(path_stack, nodesets, any_type: :element)
490488
raw_node.name == name and raw_node.namespace == ns
491489
end
492490
when :attribute
493-
if prefix.nil?
494-
raw_node.name == name
495-
elsif prefix.empty?
491+
if prefix.empty?
496492
# FIXME: This DOUBLES the time XPath searches take
497493
raw_node.name == name and
498494
raw_node.namespace == raw_node.element.namespace

0 commit comments

Comments
 (0)