<div dir="ltr">+1.  High time for this fix.<div><br></div><div>FWIW JTS 1.11 made this change back in 2010.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Apr 11, 2022 at 8:59 AM Paul Ramsey <<a href="mailto:pramsey@cleverelephant.ca">pramsey@cleverelephant.ca</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><a href="https://github.com/postgis/postgis/pull/682/" rel="noreferrer" target="_blank">https://github.com/postgis/postgis/pull/682/</a><br>
<br>
Just a point of discussion, this PR to change the default MULTIPOINT output. <br>
- Pro: it's correct, the ISO spec mandates extra parens, so MULTIPOINT((1 1), (2 2), (3 3))<br>
- Also: It's not new, this was true of the ISO spec from the start, I just mis-read it<br>
- Con: It's a change, to a core output format, after a good decade<br>
<br>
Some side notes<br>
- The input parser already accepts the correct ISO form, so this won't break *our* consumption or ability to round-trip WKT<br>
<br>
I'm inclined to accept, and make sure we have good change / news notes about it, as it's potentially breaking.<br>
<br>
P<br>
_______________________________________________<br>
postgis-devel mailing list<br>
<a href="mailto:postgis-devel@lists.osgeo.org" target="_blank">postgis-devel@lists.osgeo.org</a><br>
<a href="https://lists.osgeo.org/mailman/listinfo/postgis-devel" rel="noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/postgis-devel</a><br>
</blockquote></div>