mark.cave-ayland at siriusit.co.uk
Fri Dec 19 03:52:42 PST 2008
Paul Ramsey wrote:
> This is not meant to be the last word, it's just a draft, please add
> your changes as you see fit (you in particular, Mark).
> On a related note, as I work with liblwgeom, I wish we had more API
> documentation than the header file... I think doxygen will work fine
> w/ C. should we commit to doxygen as our in-code standard?
This looks entirely sensible, nice work Paul :) I think at some point
we should just add a script to the repo that will indent all the .c/.h
files (excluding those used to generate .sql.in files) and run it as we
hit RC. Doxygen is something I've not personally used, although I've
used equivalents for PHP etc. Have you got any example files in trunk I
could run it on to take a look?
BTW I noticed your r3450 commit, and this got me thinking. One of things
that really trips me up at the moment is LWCURVE - why oh why was it not
called LWCIRCSTRING??! This tends to trip me up a lot, since SQL-MM
defines a curve as a LINESTRING or a CIRCULARSTRING and so makes the
logic a lot harder to follow when trying to map SQL types to C types.
So would anyone really object if I performed another massively invasive
substitution to get this out of my system for once and for all on trunk?
I promise not to break the build ;)
Sirius Corporation - The Open Source Experts
T: +44 870 608 0063
More information about the postgis-devel