ST_AsMVTGeom buffer default changed from 0 to 256

git-svn-id: http://svn.osgeo.org/postgis/trunk@15677 b70326c6-7e19-0410-871a-916f4a2858ee

diff --git a/doc/reference_output.xml b/doc/reference_output.xml
index 6294acec19486e96d6f7358038f8666d87011011..2b2ee8bb3766f7c11337ec1da230e23dd1104372 100644 (file)
--- a/doc/reference_output.xml
+++ b/doc/reference_output.xml
@@ -1361,7 +1361,7 @@ SELECT ST_GeoHash(ST_SetSRID(ST_MakePoint(-126,48),4326),5);
                                <paramdef><type>geometry </type> <parameter>geom</parameter></paramdef>
                                <paramdef><type>box2d </type> <parameter>bounds</parameter></paramdef>
                                <paramdef choice="opt"><type>int4 </type> <parameter>extent=4096</parameter></paramdef>
-                               <paramdef choice="opt"><type>int4 </type> <parameter>buffer=0</parameter></paramdef>
+                               <paramdef choice="opt"><type>int4 </type> <parameter>buffer=256</parameter></paramdef>
                                <paramdef choice="opt"><type>bool </type> <parameter>clip_geom=true</parameter></paramdef>
                        </funcprototype>
                </funcsynopsis>
@@ -1377,7 +1377,7 @@ SELECT ST_GeoHash(ST_SetSRID(ST_MakePoint(-126,48),4326),5);
                <para><varname>geom</varname> is the geometry to transform.</para>
                <para><varname>bounds</varname> is the geometric bounds of the tile contents without buffer.</para>
                <para><varname>extent</varname> is the tile extent in tile coordinate space as defined by the <ulink url="https://www.mapbox.com/vector-tiles/specification/">specification</ulink>. If NULL it will default to 4096.</para>
-               <para><varname>buffer</varname> is the buffer distance in tile coordinate space to optionally clip geometries. If NULL it will default to 0.</para>
+               <para><varname>buffer</varname> is the buffer distance in tile coordinate space to optionally clip geometries. If NULL it will default to 256.</para>
                <para><varname>clip_geom</varname> is a boolean to control if geometries should be clipped or encoded as is. If NULL it will default to true.</para>
 
                <para>Availability: 2.4.0</para>

diff --git a/postgis/lwgeom_out_mvt.c b/postgis/lwgeom_out_mvt.c
index 321e80966f901ff09753cf09abad5826f2589bf3..bc609c9a6ae6f0c2076b7a1686d0a4689b0ef628 100644 (file)
--- a/postgis/lwgeom_out_mvt.c
+++ b/postgis/lwgeom_out_mvt.c
@@ -58,7 +58,7 @@ Datum ST_AsMVTGeom(PG_FUNCTION_ARGS)
                elog(ERROR, "ST_AsMVTGeom: parameter bounds cannot be null");
        bounds = (GBOX *) PG_GETARG_POINTER(1);
        extent = PG_ARGISNULL(2) ? 4096 : PG_GETARG_INT32(2);
-       buffer = PG_ARGISNULL(3) ? 0 : PG_GETARG_INT32(3);
+       buffer = PG_ARGISNULL(3) ? 256 : PG_GETARG_INT32(3);
        clip_geom = PG_ARGISNULL(4) ? true : PG_GETARG_BOOL(4);
        lwgeom_out = mvt_geom(lwgeom_in, bounds, extent, buffer, clip_geom);
        lwgeom_free(lwgeom_in);

diff --git a/postgis/postgis.sql.in b/postgis/postgis.sql.in
index 5a6bdcad15b8f081506789bc07cd31c8f974fa19..d76b88daf29cceaf74d8d813e8804f36bf9fb0ef 100644 (file)
--- a/postgis/postgis.sql.in
+++ b/postgis/postgis.sql.in
@@ -4442,7 +4442,7 @@ CREATE AGGREGATE ST_AsMVT(anyelement, text, int4, text)
 );
 
 -- Availability: 2.4.0
-CREATE OR REPLACE FUNCTION ST_AsMVTGeom(geom geometry, bounds box2d, extent int4 default 4096, buffer int4 default 0, clip_geom bool default true)
+CREATE OR REPLACE FUNCTION ST_AsMVTGeom(geom geometry, bounds box2d, extent int4 default 4096, buffer int4 default 256, clip_geom bool default true)
        RETURNS geometry
        AS 'MODULE_PATHNAME','ST_AsMVTGeom'
        LANGUAGE 'c' IMMUTABLE  _PARALLEL;