Opened 13 years ago

Closed 12 years ago

Last modified 12 years ago

#1162 closed defect (fixed)

chuck old junk about older versions of PostgreSQL

Reported by: robe Owned by: pramsey
Priority: low Milestone: PostGIS 2.1.0
Component: postgis Version: master
Keywords: Cc:

Description

Given all the problems I am having, I have been forced to look at some of the code we have.

We've got ifdefs that are just dead weight.

Case in point: libpgcommon\pgsql_compat.h

We don't support anything lower than 8.4 so none of these conditions really makes sense to have. The file should either be blanked out or be gone.

Same goes for that PG_MODULE_MAGIC ifdef stuff. All versions we have now require PG_MODULE_MAGIC so why do we bother with ifdef.

Same goes for postgis/lwgeom_accum.c

We've got code that has:

#if POSTGIS_PGSQL_VERSION < 84

	result = makeMdArrayResult(state, 1, dims, lbs, mctx);
#else

	result = makeMdArrayResult(state, 1, dims, lbs, mctx, false);
#endif

We don't support lower than 8.4 — this should be just:

result = makeMdArrayResult(state, 1, dims, lbs, mctx, false);

I'd be willing to cleanup some of this if only I could compile again.

Change History (4)

comment:1 by robe, 13 years ago

Milestone: PostGIS 2.0.0PostGIS Future

comment:2 by pramsey, 12 years ago

Resolution: fixed
Status: newclosed

These seem to be almost all gone, perhaps they got cleaned out earlier. The *only* ones I see left are in libpgcommon/pgsql_compat.h and look pretty harmless… I actually want to leave them there as examples of previous compatibility since they'll all just get ignored in the preproc. r10662

comment:3 by robe, 12 years ago

Milestone: PostGIS FuturePostGIS 2.1.0

and he doesn't bother to change the milestone. Barbarian.

comment:4 by pramsey, 12 years ago

Always nice to find someone even more OCD than I…

Note: See TracTickets for help on using tickets.