Replace Array#| with Array#+

This is primarily for performance reasons. It yields around a 23% improvement on a file with 300 violations.
yorickpeterse · Jan 9, 2015 · 58feada · 58feada
1 parent b6a1248
commit 58feada
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/lib/ruby-lint/definition/ruby_object.rb b/lib/ruby-lint/definition/ruby_object.rb
@@ -697,7 +697,9 @@ def determine_parent(parent, type, name, exclude = [])
         if parent.type == type and parent.name == name
           parent_definition = parent
         else
-          parent_definition = parent.lookup(type, name, true, exclude | [self])
+          exclude = exclude + [self] unless exclude.include?(self)
+
+          parent_definition = parent.lookup(type, name, true, exclude)
         end
 
         return parent_definition