Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#3628 closed defect (fixed)

lwgeom_functions_basic.c:2637]: (style) Same expression on both sides of '||'.

Reported by: dcb Owned by: pramsey
Priority: medium Milestone: PostGIS 2.3.0
Component: postgis Version: 2.2.x
Keywords: Cc:

Description

Source code is

if ( n == 'y'
n == 'y' ) return LWORD_Y;

Maybe better code

if ( n == 'y'
n == 'Y' ) return LWORD_Y;

Change History (4)

comment:1 by strk, 6 years ago

Resolution: fixed
Status: newclosed

In 15087:

Fix lack of support for capitalized-Y in SwapOrdinates

Closes #3628

comment:2 by strk, 6 years ago

In 15088:

Fix lack of support for capitalized-Y in SwapOrdinates

Closes #3628

comment:3 by strk, 6 years ago

In 15090:

Add test for ST_SwapOrdinate fix (#3628)

comment:4 by strk, 6 years ago

In 15092:

Add test for ST_SwapOrdinate fix (#3628)

Note: See TracTickets for help on using tickets.