Merge pull request #5325 from rmosolgo/fix-vis-for-large-lists

Visibility: better duplicate name lookup

Author: rmosolgo

Diff for: lib/graphql/schema/visibility/profile.rb

@@ -281,9 +281,11 @@ def visible_enum_value?(enum_value, _ctx = nil)
 
         def non_duplicate_items(definitions, visibility_cache)
           non_dups = []
+          names = Set.new
           definitions.each do |defn|
             if visibility_cache[defn]
-              if (dup_defn = non_dups.find { |d| d.graphql_name == defn.graphql_name })
+              if !names.add?(defn.graphql_name)
+                dup_defn = non_dups.find { |d| d.graphql_name == defn.graphql_name }
                 raise_duplicate_definition(dup_defn, defn)
               end
               non_dups << defn