Remove unnecessary uses of collect (#1132)

* Remove unnecessary uses of 'collect'

* Fix code

* Remove comments

Author: eliascarv

src/projecting.jl

@@ -14,7 +14,7 @@ proj2D(r::Rope) = Ring(proj2D(vertices(r)))
 
 proj2D(r::Ring) = Ring(proj2D(vertices(r)))
 
-proj2D(p::Ngon) = Ngon(proj2D(collect(vertices(p)))...)
+proj2D(p::Ngon) = Ngon(proj2D(vertices(p)))
 
 proj2D(p::PolyArea) = PolyArea(proj2D.(rings(p)))
 

src/simplification/douglaspeucker.jl

@@ -24,7 +24,7 @@ end
 DouglasPeuckerSimplification(τ) = DouglasPeuckerSimplification(addunit(τ, u"m"))
 
 function simplify(chain::Chain, method::DouglasPeuckerSimplification)
-  verts = _douglaspeucker(vertices(chain), method.τ) |> collect
+  verts = _douglaspeucker(vertices(chain), method.τ)
   isclosed(chain) ? Ring(verts) : Rope(verts)
 end
 

src/simplification/selinger.jl

@@ -69,7 +69,7 @@ function simplify(chain::Chain, method::SelingerSimplification)
     end
   end
 
-  Ring(collect(v[bestpath[begin:(end - 1)]]))
+  Ring(v[bestpath[begin:(end - 1)]])
 end
 
 function dijkstra(I, s, t)

src/transforms/repair.jl

@@ -98,8 +98,6 @@ function apply(::Repair{8}, ring::Ring)
   Ring(v), nothing
 end
 
-repair8(v) = repair8(collect(v))
-
 repair8(v::AbstractVector) = repair8(CircularVector(v))
 
 function repair8(v::CircularVector{<:Point})