[breaking] remove unused Containers.generate_container (#2744)

Author: odow

src/Containers/Containers.jl

@@ -49,7 +49,6 @@ function Base.collect_to_with_first!(
     return dest
 end
 
-include("generate_container.jl")
 include("vectorized_product_iterator.jl")
 include("nested_iterator.jl")
 include("no_duplicate_dict.jl")

src/Containers/generate_container.jl

@@ -143,6 +143,24 @@ function _parse_ref_sets(_error::Function, expr::Expr)
 end
 _parse_ref_sets(_error::Function, expr) = (Any[], Any[], :())
 
+_depends_on(ex::Expr, s::Symbol) = any(a -> _depends_on(a, s), ex.args)
+
+_depends_on(ex::Symbol, s::Symbol) = ex == s
+
+_depends_on(ex, s::Symbol) = false
+
+function _has_dependent_sets(idxvars, idxsets)
+    # check if any index set depends on a previous index var
+    for i in 2:length(idxsets)
+        for j in 1:(i-1)
+            if _depends_on(idxsets[i], idxvars[j])
+                return true
+            end
+        end
+    end
+    return false
+end
+
 """
     _build_ref_sets(_error::Function, expr)
 
@@ -161,7 +179,7 @@ function _build_ref_sets(_error::Function, expr)
             "cannot use splatting operator `...` in the definition of an index set.",
         )
     end
-    has_dependent = has_dependent_sets(idxvars, idxsets)
+    has_dependent = _has_dependent_sets(idxvars, idxsets)
     if has_dependent || condition != :()
         esc_idxvars = esc.(idxvars)
         idxfuns = [