if the closing line segment has zero length, don't include it in the nodepath!

diff --git a/src/nodepath.cpp b/src/nodepath.cpp
index aec5ff058b83417c50c9b3c8fb01b51d22fab4f1..89d1d3e8bc6a22fb4e3159ae8ae9549d8aa876cc 100644 (file)
--- a/src/nodepath.cpp
+++ b/src/nodepath.cpp
@@ -525,9 +525,14 @@ static void subpaths_from_pathvector(Inkscape::NodePath::Path *np, Geom::PathVec
 
         if (pit->closed()) {
             // Add last knot (because sp_nodepath_subpath_close kills the last knot)
-            // Remember that last closing segment is always a lineto, and that its endpoint is the path's initialPoint because the path is closed
-            NR::Point pos = from_2geom(pit->initialPoint()) * np->i2d;
-            sp_nodepath_node_new(sp, NULL, t[i++], NR_LINETO, &pos, &pos, &pos);
+            /* Remember that last closing segment is always a lineto, but its length can be zero if the path is visually closed already
+             * If the length is zero, don't add it to the nodepath. */
+            Geom::Curve const &back = pit->back_closed();
+            if ( back.initialPoint() != back.finalPoint() ) {
+                NR::Point pos = from_2geom(back.finalPoint()) * np->i2d;
+                sp_nodepath_node_new(sp, NULL, t[i++], NR_LINETO, &pos, &pos, &pos);
+            }
+
             sp_nodepath_subpath_close(sp);
         }
     }