Changeset 3824

Timestamp:

03/10/09 07:59:39 (3 years ago)

Author:

colivier

Message:

Add SVG L Command in absolute path according to BNF SVG Path. Add some forgotten geometry release.

Files:

• trunk/postgis/lwgeom_svg.c (modified) (6 diffs)

trunk/postgis/lwgeom_svg.c

@@ -15 +15 @@
  * Refactored by: Olivier Courtin (Camptocamp)
  *
+ * BNF SVG Path: <http://www.w3.org/TR/SVG/paths.html#PathDataBNF>
  **********************************************************************/
 
@@ -286 +287 @@
                 point = lwgeom_getpoint_inspected(insp, i);
                 size += assvg_point_size(point, relative, precision);
-                /* lwpoint_release(point); */
+                if (point) lwpoint_release(point);
         }
         size += sizeof(" ") * --i;  /* Arbitrary comma separator */
@@ -392 +393 @@
                 poly = lwgeom_getpoly_inspected(insp, i);
                 size += assvg_polygon_size(poly, relative, precision);
-                /* lwpoly_release(poly); */
+                if (poly) lwpoly_release(poly);
         }
         size += sizeof(" ") * --i;   /* SVG whitespace Separator */
@@ -411 +412 @@
                 poly = lwgeom_getpoly_inspected(insp, i);
                 ptr += assvg_polygon_buf(poly, ptr, relative, precision);
-                /* lwpoly_release(poly); */
+                if (poly) lwpoly_release(poly);
          }
 
@@ -647 +648 @@
                 sprintf(y, "%.*f", precision, fabs(pt.y) ? pt.y * -1:pt.y);
                 trim_trailing_zeros(y);
-                if (i) ptr += sprintf(ptr, " ");
+                if (i == 1) ptr += sprintf(ptr, " L ");
+                else if (i) ptr += sprintf(ptr, " ");
                 ptr += sprintf(ptr,"%s %s", x, y);
-                /* FIXME Could we really omit SVG L after M start point ? */
         }
 
@@ -663 +664 @@
 {
         return (MAX_DIGS_DOUBLE + precision + sizeof(", "))
-                        * 2 * pa->npoints + sizeof("l ");
-}
+                        * 2 * pa->npoints + sizeof(" L ");
+}