Apply suggestions from code review

Co-authored-by: Lukas <[email protected]>
Jutho · Jan 4, 2024 · 00d1d23 · 00d1d23
1 parent fff1008
commit 00d1d23
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 3 deletions.
diff --git a/src/planar/planaroperations.jl b/src/planar/planaroperations.jl
@@ -61,7 +61,6 @@ function planarcontract!(C::AbstractTensorMap{S,N₁,N₂},
     codB, domB = codomainind(B), domainind(B)
     oindA, cindA = pA
     cindB, oindB = pB
-    # @show codA, domA, codB, domB, oindA, cindA, oindB, cindB, pAB
     oindA, cindA, oindB, cindB = reorder_indices(codA, domA, codB, domB, oindA, cindA,
                                                  oindB, cindB, pAB...)
 
@@ -92,7 +91,6 @@ _cyclicpermute(t::Tuple{}) = ()
 function reorder_indices(codA, domA, codB, domB, oindA, oindB, p1, p2)
     N₁ = length(oindA)
     N₂ = length(oindB)
-    # @show codA, domA, codB, domB, oindA, oindB, p1, p2
     @assert length(p1) == N₁ && all(in(p1), 1:N₁)
     @assert length(p2) == N₂ && all(in(p2), N₁ .+ (1:N₂))
     oindA2 = TupleTools.getindices(oindA, p1)

diff --git a/src/planar/preprocessors.jl b/src/planar/preprocessors.jl
@@ -375,7 +375,7 @@ function _remove_braidingtensors!(ex, indexmap) # ex is guaranteed to be a singl
         newarg, unchanged = _remove_braidingtensors!(ex.args[3], indexmap)
         return Expr(:call, :\, ex.args[2], newarg), unchanged
     else
-        error("unexpected expression $ex")
+        throw(ArgumentError("unexpected expression $ex"))
     end
 end