Opened 11 years ago

Closed 11 years ago

#383 closed defect (fixed)

Liblwgeom not compile with POSTGIS_DEBUG_LEVEL 4

Reported by: colivier Owned by: colivier
Priority: medium Milestone:
Component: postgis Version: master
Keywords: measure.c POSTGIS_DEBUG_LEVEL Cc:

Description

./configure
sed -i "s/POSTGIS_DEBUG_LEVEL 0/POSTGIS_DEBUG_LEVEL 4/" postgis_config.h 
cd liblwgeom/
make
measures.c: In function 'lw_dist2d_ptarray_poly':
measures.c:712: error: 'dist' undeclared (first use in this function)
measures.c:712: error: (Each undeclared identifier is reported only once
measures.c:712: error: for each function it appears in.)
measures.c:712: error: 'mindist' undeclared (first use in this function)
measures.c: In function 'lw_dist2d_ptarray_ptarray':
measures.c:783: error: 'dist' undeclared (first use in this function)
measures.c:784: error: 'result' undeclared (first use in this function)
make: *** [measures.o] Error 1

Change History (4)

comment:1 Changed 11 years ago by nicklas

woops That looks like my fault

comment:2 Changed 11 years ago by colivier

Commited a fix as r5124 Could you check it Nicklas ?

comment:3 Changed 11 years ago by nicklas

Yes, I think it looks good. At least it should work.
How relevant the tolerance/mindist is I don't know. But of course it might be good to analyze so the function stops as it should when reaching tolerance. I guess that's why it is there in the first place.

I don't have possibility to try compiling it now, but I guess you have :-)

Thanks Nicklas

comment:4 Changed 11 years ago by colivier

Resolution: fixed
Status: newclosed

Ok,

And yes here on a linux box make/make check are fine.

I close this 30' ticket ;)

Note: See TracTickets for help on using tickets.