<div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr">Looks like it was Andres, about two months ago, during the switch to "modern" C99<div><br></div><div><a href="https://github.com/postgres/postgres/blame/696b0c5fd0a8765fe6dfd075a30be06b448fd615/configure.in#L488">https://github.com/postgres/postgres/blame/696b0c5fd0a8765fe6dfd075a30be06b448fd615/configure.in#L488</a><br></div><div><br></div><div>Wikipedia commentary :) </div><div><br></div><div><a href="https://en.wikipedia.org/wiki/Variable-length_array#C99">https://en.wikipedia.org/wiki/Variable-length_array#C99</a><br></div><div><br></div><div>Our failure:</div><div><br></div><div><div>geography_centroid.c:108:24: error: variable length array used [-Werror,-Wvla]</div><div>        POINT3DM points[size];</div><div>                       ^</div><div>geography_centroid.c:261:20: error: variable length array used [-Werror,-Wvla]</div><div>    POINT3DM points[size];</div><div>                   ^</div><div>geography_centroid.c:316:20: error: variable length array used [-Werror,-Wvla]</div><div>    POINT3DM points[size];</div><div>                   ^</div></div><div><br></div><div>Fun! Let the bikeshedding begin!</div><div><br></div><div>P</div></div></div></div></div><br><div class="gmail_quote"><div dir="ltr">On Wed, Oct 31, 2018 at 2:14 PM Paul Ramsey <<a href="mailto:pramsey@cleverelephant.ca">pramsey@cleverelephant.ca</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div dir="ltr">Interesting note: build against Pg12 just failed for me because my build off of trunk master included this CFLAG<div><br></div><div>    -Werror=vla<br></div><div><br></div><div>Funny, hey? It's ./configured exactly the same as my Pg11 version, but it's got that error on variable length array. I'll see if I can figure out why PgSQL is now added that guard.</div><div><br></div><div>P.</div></div></div>
</blockquote></div>