> Thank you for the revised version. This seems fine for me so > marked this as "Ready for Committer".
Thank you for bearing with me for longer than year. > By the way I think that line_perp is back patched, could you > propose it for the older versions? (I already marked my entry as > Rejected) Do you mean back-patching? > Brief summary follows (almost same to the header of patch files): > > - 0001 looks fine. > > * Eliminate SQL level function calls > > * Re-use more functions to implement others > > * Unify internal function names and signatures > > * Remove private functions from geo_decls.h > > * Replace not-possible checks with Assert() > > * Add comments to describe some functions > > * Remove some unreachable code > > * Define delimiter symbols of line datatype like the other ones > > * Remove debugging print()s from the refactored functions > > And one bug fix. What is that one? Should I incorporate it into the commit message?