[postgis-devel] [postgis-tickets] r15406 - Fix topology PostgreSQL 10 regress errors

[Sandro Santilli]

On Fri, May 26, 2017 at 12:35:19PM -0700, Regina Obe wrote:
> Author: robe
> Date: 2017-05-26 12:35:19 -0700 (Fri, 26 May 2017)
> New Revision: 15406
...

> -    FOR rec2 IN SELECT CASE
> -       WHEN rec.dims = 0 THEN
> +  	-- NOTE: storing this as a constant 
> +  	-- because the CASE WHEN in PostgreSQL 10 refuses to short-circuit 
> +  	-- when CASE contains set returning functions and the CASE boolean is not a constant
> +  	-- TODO: this feels a little dirty to assume so may need a revisit in near future
> +  	var_dims = rec.dims;
> +    FOR rec2 IN SELECT CASE var_dims

Regina, the indentation for this file is wrong.
If you're finally using git (the mirroring scripts should allow you to
setup such a workflow) you can see the issues introduced using:

  git show --check 0d939f5c54eb14313952d7f9db6e8cb4fd09c4a8

Which tells you (with colors, that you won't see here):

  topology/sql/topogeometry/totopogeom.sql.in:243: trailing whitespace, space before tab in indent.
  +       -- NOTE: storing this as a constant
  topology/sql/topogeometry/totopogeom.sql.in:244: trailing whitespace, space before tab in indent.
  +       -- because the CASE WHEN in PostgreSQL 10 refuses to short-circuit
  topology/sql/topogeometry/totopogeom.sql.in:245: space before tab in indent.
  +       -- when CASE contains set returning functions and the CASE boolean is not a constant
  topology/sql/topogeometry/totopogeom.sql.in:246: space before tab in indent.
  +       -- TODO: this feels a little dirty to assume so may need a revisit in near future
  topology/sql/topogeometry/totopogeom.sql.in:247: space before tab in indent.
  +       var_dims = rec.dims;

That file uses spaces for tabs, btw. I see editorconfig doesn't
mention what kind of tabs should be used for SQL files so this is
not something currently easy to deal with (maybe this could be
ticketed, to get some help from editorconfig).

--strk;