]> granicus.if.org Git - postgresql/commitdiff
Convert all remaining geometric operators to new fmgr style. This
authorTom Lane <tgl@sss.pgh.pa.us>
Sun, 30 Jul 2000 20:44:02 +0000 (20:44 +0000)
committerTom Lane <tgl@sss.pgh.pa.us>
Sun, 30 Jul 2000 20:44:02 +0000 (20:44 +0000)
allows fixing problems with operators that expected to be able to
return a NULL, such as the '#' line-segment-intersection operator
that tried to return NULL when the two segments don't intersect.
(See, eg, bug report from 1-Nov-99 on pghackers.)  Fix some other
bugs in passing, such as backwards comparison in path_distance().

src/backend/access/rtree/rtproc.c
src/backend/access/rtree/rtree.c
src/backend/utils/adt/geo_ops.c
src/include/catalog/pg_proc.h
src/include/utils/geo_decls.h
src/test/regress/input/create_function_2.source
src/test/regress/output/create_function_2.source
src/test/regress/regress.c

index 3cb7efc64a06530217b3d2230ef6e7d913bd9acc..dfe5805a6b0471b1df1b42bf0adabe5c67ff4702 100644 (file)
@@ -3,12 +3,19 @@
  * rtproc.c
  *       pg_amproc entries for rtrees.
  *
+ * NOTE: for largely-historical reasons, the intersection functions should
+ * return a NULL pointer (*not* an SQL null value) to indicate "no
+ * intersection".  The size functions must be prepared to accept such
+ * a pointer and return 0.  This convention means that only pass-by-reference
+ * data types can be used as the output of the union and intersection
+ * routines, but that's not a big problem.
+ *
+ *
  * Portions Copyright (c) 1996-2000, PostgreSQL, Inc
  * Portions Copyright (c) 1994, Regents of the University of California
  *
- *
  * IDENTIFICATION
- *       $Header: /cvsroot/pgsql/src/backend/access/rtree/Attic/rtproc.c,v 1.28 2000/07/29 18:45:52 tgl Exp $
+ *       $Header: /cvsroot/pgsql/src/backend/access/rtree/Attic/rtproc.c,v 1.29 2000/07/30 20:43:40 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
 #include "utils/geo_decls.h"
 
 
-BOX *
-rt_box_union(BOX *a, BOX *b)
+Datum
+rt_box_union(PG_FUNCTION_ARGS)
 {
+       BOX                *a = PG_GETARG_BOX_P(0);
+       BOX                *b = PG_GETARG_BOX_P(1);
        BOX                *n;
 
-       if ((n = (BOX *) palloc(sizeof(*n))) == (BOX *) NULL)
-               elog(ERROR, "Cannot allocate box for union");
+       n = (BOX *) palloc(sizeof(BOX));
 
        n->high.x = Max(a->high.x, b->high.x);
        n->high.y = Max(a->high.y, b->high.y);
        n->low.x = Min(a->low.x, b->low.x);
        n->low.y = Min(a->low.y, b->low.y);
 
-       return n;
+       PG_RETURN_BOX_P(n);
 }
 
-BOX *
-rt_box_inter(BOX *a, BOX *b)
+Datum
+rt_box_inter(PG_FUNCTION_ARGS)
 {
+       BOX                *a = PG_GETARG_BOX_P(0);
+       BOX                *b = PG_GETARG_BOX_P(1);
        BOX                *n;
 
-       if ((n = (BOX *) palloc(sizeof(*n))) == (BOX *) NULL)
-               elog(ERROR, "Cannot allocate box for union");
+       n = (BOX *) palloc(sizeof(BOX));
 
        n->high.x = Min(a->high.x, b->high.x);
        n->high.y = Min(a->high.y, b->high.y);
@@ -50,21 +59,26 @@ rt_box_inter(BOX *a, BOX *b)
        if (n->high.x < n->low.x || n->high.y < n->low.y)
        {
                pfree(n);
-               return (BOX *) NULL;
+               /* Indicate "no intersection" by returning NULL pointer */
+               n = NULL;
        }
 
-       return n;
+       PG_RETURN_BOX_P(n);
 }
 
-void
-rt_box_size(BOX *a, float *size)
+Datum
+rt_box_size(PG_FUNCTION_ARGS)
 {
+       BOX                *a = PG_GETARG_BOX_P(0);
+       /* NB: size is an output argument */
+       float      *size = (float *) PG_GETARG_POINTER(1);
+
        if (a == (BOX *) NULL || a->high.x <= a->low.x || a->high.y <= a->low.y)
                *size = 0.0;
        else
                *size = (float) ((a->high.x - a->low.x) * (a->high.y - a->low.y));
 
-       return;
+       PG_RETURN_VOID();
 }
 
 /*
@@ -75,10 +89,10 @@ rt_box_size(BOX *a, float *size)
  *             as the return type for the size routine, so we no longer need to
  *             have a special return type for big boxes.
  */
-void
-rt_bigbox_size(BOX *a, float *size)
+Datum
+rt_bigbox_size(PG_FUNCTION_ARGS)
 {
-       rt_box_size(a, size);
+       return rt_box_size(fcinfo);
 }
 
 Datum
@@ -105,30 +119,6 @@ rt_poly_union(PG_FUNCTION_ARGS)
        PG_RETURN_POLYGON_P(p);
 }
 
-Datum
-rt_poly_size(PG_FUNCTION_ARGS)
-{
-       POLYGON    *a = PG_GETARG_POLYGON_P(0);
-       /* NB: size is an output argument */
-       float      *size = (float *) PG_GETARG_POINTER(1);
-       double          xdim,
-                               ydim;
-
-       if (a == (POLYGON *) NULL ||
-               a->boundbox.high.x <= a->boundbox.low.x ||
-               a->boundbox.high.y <= a->boundbox.low.y)
-               *size = 0.0;
-       else
-       {
-               xdim = (a->boundbox.high.x - a->boundbox.low.x);
-               ydim = (a->boundbox.high.y - a->boundbox.low.y);
-
-               *size = (float) (xdim * ydim);
-       }
-
-       PG_RETURN_VOID();
-}
-
 Datum
 rt_poly_inter(PG_FUNCTION_ARGS)
 {
@@ -146,16 +136,52 @@ rt_poly_inter(PG_FUNCTION_ARGS)
        p->boundbox.low.x = Max(a->boundbox.low.x, b->boundbox.low.x);
        p->boundbox.low.y = Max(a->boundbox.low.y, b->boundbox.low.y);
 
-       /* Avoid leaking memory when handed toasted input. */
-       PG_FREE_IF_COPY(a, 0);
-       PG_FREE_IF_COPY(b, 1);
-
        if (p->boundbox.high.x < p->boundbox.low.x ||
                p->boundbox.high.y < p->boundbox.low.y)
        {
                pfree(p);
-               PG_RETURN_NULL();
+               /* Indicate "no intersection" by returning NULL pointer */
+               p = NULL;
        }
 
+       /* Avoid leaking memory when handed toasted input. */
+       PG_FREE_IF_COPY(a, 0);
+       PG_FREE_IF_COPY(b, 1);
+
        PG_RETURN_POLYGON_P(p);
 }
+
+Datum
+rt_poly_size(PG_FUNCTION_ARGS)
+{
+       Pointer         aptr = PG_GETARG_POINTER(0);
+       /* NB: size is an output argument */
+       float      *size = (float *) PG_GETARG_POINTER(1);
+       POLYGON    *a;
+       double          xdim,
+                               ydim;
+
+       /* Can't just use GETARG because of possibility that input is NULL;
+        * since POLYGON is toastable, GETARG will try to inspect its value
+        */
+       if (aptr == NULL)
+       {
+               *size = 0.0;
+               PG_RETURN_VOID();
+       }
+       /* Now safe to apply GETARG */
+       a = PG_GETARG_POLYGON_P(0);
+
+       if (a->boundbox.high.x <= a->boundbox.low.x ||
+               a->boundbox.high.y <= a->boundbox.low.y)
+               *size = 0.0;
+       else
+       {
+               xdim = (a->boundbox.high.x - a->boundbox.low.x);
+               ydim = (a->boundbox.high.y - a->boundbox.low.y);
+
+               *size = (float) (xdim * ydim);
+       }
+
+       PG_RETURN_VOID();
+}
index 583baa534a369e2c8a6957fbf4f22a4ff03ad63a..d36bf79a8d5803e6a12468e4adcc1a4cf83fce9e 100644 (file)
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *       $Header: /cvsroot/pgsql/src/backend/access/rtree/Attic/rtree.c,v 1.52 2000/07/14 22:17:36 tgl Exp $
+ *       $Header: /cvsroot/pgsql/src/backend/access/rtree/Attic/rtree.c,v 1.53 2000/07/30 20:43:40 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -745,13 +745,16 @@ picksplit(Relation r,
                                DatumGetPointer(FunctionCall2(&rtstate->interFn,
                                                                                          PointerGetDatum(datum_alpha),
                                                                                          PointerGetDatum(datum_beta)));
+                       /* The interFn may return a NULL pointer (not an SQL null!)
+                        * to indicate no intersection.  sizeFn must cope with this.
+                        */
                        FunctionCall2(&rtstate->sizeFn,
                                                  PointerGetDatum(inter_d),
                                                  PointerGetDatum(&size_inter));
                        size_waste = size_union - size_inter;
 
-                       pfree(union_d);
-
+                       if (union_d != (char *) NULL)
+                               pfree(union_d);
                        if (inter_d != (char *) NULL)
                                pfree(inter_d);
 
@@ -1051,7 +1054,8 @@ _rtdump(Relation r)
                        itoffno = ItemPointerGetOffsetNumber(&(itup->t_tid));
                        datum = ((char *) itup);
                        datum += sizeof(IndexTupleData);
-                       itkey = (char *) box_out((BOX *) datum);
+                       itkey = DatumGetCString(DirectFunctionCall1(box_out,
+                                                                                               PointerGetDatum(datum)));
                        printf("\t[%d] size %d heap <%d,%d> key:%s\n",
                                   offnum, IndexTupleSize(itup), itblkno, itoffno, itkey);
                        pfree(itkey);
index 5eef1458aa2dac1157b115c2c87878ef70fdd43c..09a2da7158b883605c82ec6e515eeb626ff23253 100644 (file)
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *       $Header: /cvsroot/pgsql/src/backend/utils/adt/geo_ops.c,v 1.53 2000/07/29 18:45:53 tgl Exp $
+ *       $Header: /cvsroot/pgsql/src/backend/utils/adt/geo_ops.c,v 1.54 2000/07/30 20:43:41 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -34,12 +34,16 @@ static int  lseg_crossing(double x, double y, double px, double py);
 static BOX *box_construct(double x1, double x2, double y1, double y2);
 static BOX *box_copy(BOX *box);
 static BOX *box_fill(BOX *result, double x1, double x2, double y1, double y2);
+static bool box_ov(BOX *box1, BOX *box2);
 static double box_ht(BOX *box);
 static double box_wd(BOX *box);
 static double circle_ar(CIRCLE *circle);
 static CIRCLE *circle_copy(CIRCLE *circle);
 static LINE *line_construct_pm(Point *pt, double m);
+static void line_construct_pts(LINE *line, Point *pt1, Point *pt2);
+static bool lseg_intersect_internal(LSEG *l1, LSEG *l2);
 static double lseg_dt(LSEG *l1, LSEG *l2);
+static bool on_ps_internal(Point *pt, LSEG *lseg);
 static void make_bound_box(POLYGON *poly);
 static bool plist_same(int npts, Point *p1, Point *p2);
 static Point *point_construct(double x, double y);
@@ -53,7 +57,12 @@ static int   path_decode(int opentype, int npts, char *str, int *isopen, char **ss
 static char *path_encode(bool closed, int npts, Point *pt);
 static void statlseg_construct(LSEG *lseg, Point *pt1, Point *pt2);
 static double box_ar(BOX *box);
+static void box_cn(Point *center, BOX *box);
 static Point *interpt_sl(LSEG *lseg, LINE *line);
+static bool has_interpt_sl(LSEG *lseg, LINE *line);
+static double dist_pl_internal(Point *pt, LINE *line);
+static double dist_ps_internal(Point *pt, LSEG *lseg);
+static Point *line_interpt_internal(LINE *l1, LINE *l2);
 
 
 /*
@@ -114,7 +123,7 @@ single_decode(char *str, float8 *x, char **s)
                str++;
        *x = strtod(str, &cp);
 #ifdef GEODEBUG
-       fprintf(stderr, "single_decode- (%x) try decoding %s to %g\n", (cp - str), str, *x);
+       printf("single_decode- (%x) try decoding %s to %g\n", (cp - str), str, *x);
 #endif
        if (cp <= str)
                return FALSE;
@@ -325,6 +334,7 @@ pair_count(char *s, char delim)
        return (ndelim % 2) ? ((ndelim + 1) / 2) : -1;
 }
 
+
 /***********************************************************************
  **
  **            Routines for two-dimensional boxes.
@@ -341,19 +351,16 @@ pair_count(char *s, char delim)
  *                             "(f8, f8), (f8, f8)"
  *                             also supports the older style "(f8, f8, f8, f8)"
  */
-BOX *
-box_in(char *str)
+Datum
+box_in(PG_FUNCTION_ARGS)
 {
-       BOX                *box = palloc(sizeof(BOX));
-
+       char       *str = PG_GETARG_CSTRING(0);
+       BOX                *box = (BOX *) palloc(sizeof(BOX));
        int                     isopen;
        char       *s;
        double          x,
                                y;
 
-       if (!PointerIsValid(str))
-               elog(ERROR, " Bad (null) box external representation");
-
        if ((!path_decode(FALSE, 2, str, &isopen, &s, &(box->high)))
                || (*s != '\0'))
                elog(ERROR, "Bad box external representation '%s'", str);
@@ -372,19 +379,18 @@ box_in(char *str)
                box->low.y = y;
        }
 
-       return box;
-}      /* box_in() */
+       PG_RETURN_BOX_P(box);
+}
 
 /*             box_out -               convert a box to external form.
  */
-char *
-box_out(BOX *box)
+Datum
+box_out(PG_FUNCTION_ARGS)
 {
-       if (!PointerIsValid(box))
-               return NULL;
+       BOX                *box = PG_GETARG_BOX_P(0);
 
-       return path_encode(-1, 2, (Point *) &(box->high));
-}      /* box_out() */
+       PG_RETURN_CSTRING(path_encode(-1, 2, &(box->high)));
+}
 
 
 /*             box_construct   -               fill in a new box.
@@ -392,13 +398,13 @@ box_out(BOX *box)
 static BOX *
 box_construct(double x1, double x2, double y1, double y2)
 {
-       BOX                *result = palloc(sizeof(BOX));
+       BOX                *result = (BOX *) palloc(sizeof(BOX));
 
        return box_fill(result, x1, x2, y1, y2);
 }
 
 
-/*             box_fill                -               fill in a static box
+/*             box_fill                -               fill in a given box struct
  */
 static BOX *
 box_fill(BOX *result, double x1, double x2, double y1, double y2)
@@ -448,22 +454,41 @@ box_copy(BOX *box)
 
 /*             box_same                -               are two boxes identical?
  */
-bool
-box_same(BOX *box1, BOX *box2)
+Datum
+box_same(PG_FUNCTION_ARGS)
 {
-       return ((FPeq(box1->high.x, box2->high.x) && FPeq(box1->low.x, box2->low.x)) &&
-       (FPeq(box1->high.y, box2->high.y) && FPeq(box1->low.y, box2->low.y)));
+       BOX                *box1 = PG_GETARG_BOX_P(0);
+       BOX                *box2 = PG_GETARG_BOX_P(1);
+
+       PG_RETURN_BOOL(FPeq(box1->high.x, box2->high.x) &&
+                                  FPeq(box1->low.x, box2->low.x) &&
+                                  FPeq(box1->high.y, box2->high.y) &&
+                                  FPeq(box1->low.y, box2->low.y));
 }
 
 /*             box_overlap             -               does box1 overlap box2?
  */
-bool
-box_overlap(BOX *box1, BOX *box2)
+Datum
+box_overlap(PG_FUNCTION_ARGS)
 {
-       return (((FPge(box1->high.x, box2->high.x) && FPle(box1->low.x, box2->high.x)) ||
-                        (FPge(box2->high.x, box1->high.x) && FPle(box2->low.x, box1->high.x))) &&
-                       ((FPge(box1->high.y, box2->high.y) && FPle(box1->low.y, box2->high.y)) ||
-                        (FPge(box2->high.y, box1->high.y) && FPle(box2->low.y, box1->high.y))));
+       BOX                *box1 = PG_GETARG_BOX_P(0);
+       BOX                *box2 = PG_GETARG_BOX_P(1);
+
+       PG_RETURN_BOOL(box_ov(box1, box2));
+}
+
+static bool
+box_ov(BOX *box1, BOX *box2)
+{
+       return ((FPge(box1->high.x, box2->high.x) &&
+                        FPle(box1->low.x, box2->high.x)) ||
+                       (FPge(box2->high.x, box1->high.x) &&
+                        FPle(box2->low.x, box1->high.x)))
+               &&
+               ((FPge(box1->high.y, box2->high.y) &&
+                 FPle(box1->low.y, box2->high.y)) ||
+                (FPge(box2->high.y, box1->high.y) &&
+                 FPle(box2->low.y, box1->high.y)));
 }
 
 /*             box_overleft    -               is the right edge of box1 to the left of
@@ -472,26 +497,35 @@ box_overlap(BOX *box1, BOX *box2)
  *             This is "less than or equal" for the end of a time range,
  *             when time ranges are stored as rectangles.
  */
-bool
-box_overleft(BOX *box1, BOX *box2)
+Datum
+box_overleft(PG_FUNCTION_ARGS)
 {
-       return FPle(box1->high.x, box2->high.x);
+       BOX                *box1 = PG_GETARG_BOX_P(0);
+       BOX                *box2 = PG_GETARG_BOX_P(1);
+
+       PG_RETURN_BOOL(FPle(box1->high.x, box2->high.x));
 }
 
 /*             box_left                -               is box1 strictly left of box2?
  */
-bool
-box_left(BOX *box1, BOX *box2)
+Datum
+box_left(PG_FUNCTION_ARGS)
 {
-       return FPlt(box1->high.x, box2->low.x);
+       BOX                *box1 = PG_GETARG_BOX_P(0);
+       BOX                *box2 = PG_GETARG_BOX_P(1);
+
+       PG_RETURN_BOOL(FPlt(box1->high.x, box2->low.x));
 }
 
 /*             box_right               -               is box1 strictly right of box2?
  */
-bool
-box_right(BOX *box1, BOX *box2)
+Datum
+box_right(PG_FUNCTION_ARGS)
 {
-       return FPgt(box1->low.x, box2->high.x);
+       BOX                *box1 = PG_GETARG_BOX_P(0);
+       BOX                *box2 = PG_GETARG_BOX_P(1);
+
+       PG_RETURN_BOOL(FPgt(box1->low.x, box2->high.x));
 }
 
 /*             box_overright   -               is the left edge of box1 to the right of
@@ -500,162 +534,183 @@ box_right(BOX *box1, BOX *box2)
  *             This is "greater than or equal" for time ranges, when time ranges
  *             are stored as rectangles.
  */
-bool
-box_overright(BOX *box1, BOX *box2)
+Datum
+box_overright(PG_FUNCTION_ARGS)
 {
-       return box1->low.x >= box2->low.x;
+       BOX                *box1 = PG_GETARG_BOX_P(0);
+       BOX                *box2 = PG_GETARG_BOX_P(1);
+
+       PG_RETURN_BOOL(FPge(box1->low.x, box2->low.x));
 }
 
 /*             box_contained   -               is box1 contained by box2?
  */
-bool
-box_contained(BOX *box1, BOX *box2)
+Datum
+box_contained(PG_FUNCTION_ARGS)
 {
-       return ((FPle(box1->high.x, box2->high.x) && FPge(box1->low.x, box2->low.x)) &&
-       (FPle(box1->high.y, box2->high.y) && FPge(box1->low.y, box2->low.y)));
+       BOX                *box1 = PG_GETARG_BOX_P(0);
+       BOX                *box2 = PG_GETARG_BOX_P(1);
+
+       PG_RETURN_BOOL(FPle(box1->high.x, box2->high.x) &&
+                                  FPge(box1->low.x, box2->low.x) &&
+                                  FPle(box1->high.y, box2->high.y) &&
+                                  FPge(box1->low.y, box2->low.y));
 }
 
 /*             box_contain             -               does box1 contain box2?
  */
-bool
-box_contain(BOX *box1, BOX *box2)
+Datum
+box_contain(PG_FUNCTION_ARGS)
 {
-       return ((FPge(box1->high.x, box2->high.x) && FPle(box1->low.x, box2->low.x) &&
-       FPge(box1->high.y, box2->high.y) && FPle(box1->low.y, box2->low.y)));
+       BOX                *box1 = PG_GETARG_BOX_P(0);
+       BOX                *box2 = PG_GETARG_BOX_P(1);
+
+       PG_RETURN_BOOL(FPge(box1->high.x, box2->high.x) &&
+                                  FPle(box1->low.x, box2->low.x) &&
+                                  FPge(box1->high.y, box2->high.y) &&
+                                  FPle(box1->low.y, box2->low.y));
 }
 
 
 /*             box_positionop  -
  *                             is box1 entirely {above,below} box2?
  */
-bool
-box_below(BOX *box1, BOX *box2)
+Datum
+box_below(PG_FUNCTION_ARGS)
 {
-       return FPle(box1->high.y, box2->low.y);
+       BOX                *box1 = PG_GETARG_BOX_P(0);
+       BOX                *box2 = PG_GETARG_BOX_P(1);
+
+       PG_RETURN_BOOL(FPle(box1->high.y, box2->low.y));
 }
 
-bool
-box_above(BOX *box1, BOX *box2)
+Datum
+box_above(PG_FUNCTION_ARGS)
 {
-       return FPge(box1->low.y, box2->high.y);
+       BOX                *box1 = PG_GETARG_BOX_P(0);
+       BOX                *box2 = PG_GETARG_BOX_P(1);
+
+       PG_RETURN_BOOL(FPge(box1->low.y, box2->high.y));
 }
 
 
 /*             box_relop               -               is area(box1) relop area(box2), within
  *                                                             our accuracy constraint?
  */
-bool
-box_lt(BOX *box1, BOX *box2)
+Datum
+box_lt(PG_FUNCTION_ARGS)
 {
-       return FPlt(box_ar(box1), box_ar(box2));
+       BOX                *box1 = PG_GETARG_BOX_P(0);
+       BOX                *box2 = PG_GETARG_BOX_P(1);
+
+       PG_RETURN_BOOL(FPlt(box_ar(box1), box_ar(box2)));
 }
 
-bool
-box_gt(BOX *box1, BOX *box2)
+Datum
+box_gt(PG_FUNCTION_ARGS)
 {
-       return FPgt(box_ar(box1), box_ar(box2));
+       BOX                *box1 = PG_GETARG_BOX_P(0);
+       BOX                *box2 = PG_GETARG_BOX_P(1);
+
+       PG_RETURN_BOOL(FPgt(box_ar(box1), box_ar(box2)));
 }
 
-bool
-box_eq(BOX *box1, BOX *box2)
+Datum
+box_eq(PG_FUNCTION_ARGS)
 {
-       return FPeq(box_ar(box1), box_ar(box2));
+       BOX                *box1 = PG_GETARG_BOX_P(0);
+       BOX                *box2 = PG_GETARG_BOX_P(1);
+
+       PG_RETURN_BOOL(FPeq(box_ar(box1), box_ar(box2)));
 }
 
-bool
-box_le(BOX *box1, BOX *box2)
+Datum
+box_le(PG_FUNCTION_ARGS)
 {
-       return FPle(box_ar(box1), box_ar(box2));
+       BOX                *box1 = PG_GETARG_BOX_P(0);
+       BOX                *box2 = PG_GETARG_BOX_P(1);
+
+       PG_RETURN_BOOL(FPle(box_ar(box1), box_ar(box2)));
 }
 
-bool
-box_ge(BOX *box1, BOX *box2)
+Datum
+box_ge(PG_FUNCTION_ARGS)
 {
-       return FPge(box_ar(box1), box_ar(box2));
+       BOX                *box1 = PG_GETARG_BOX_P(0);
+       BOX                *box2 = PG_GETARG_BOX_P(1);
+
+       PG_RETURN_BOOL(FPge(box_ar(box1), box_ar(box2)));
 }
 
 
 /*----------------------------------------------------------
  *     "Arithmetic" operators on boxes.
- *             box_foo returns foo as an object (pointer) that
- can be passed between languages.
- *             box_xx  is an internal routine which returns the
- *                             actual value (and cannot be handed back to
- *                             LISP).
  *---------------------------------------------------------*/
 
 /*             box_area                -               returns the area of the box.
  */
-double *
-box_area(BOX *box)
+Datum
+box_area(PG_FUNCTION_ARGS)
 {
-       double     *result = palloc(sizeof(double));
-
-       *result = box_wd(box) * box_ht(box);
+       BOX                *box = PG_GETARG_BOX_P(0);
 
-       return result;
+       PG_RETURN_FLOAT8(box_ar(box));
 }
 
 
 /*             box_width               -               returns the width of the box
  *                                                               (horizontal magnitude).
  */
-double *
-box_width(BOX *box)
+Datum
+box_width(PG_FUNCTION_ARGS)
 {
-       double     *result = palloc(sizeof(double));
-
-       *result = box->high.x - box->low.x;
+       BOX                *box = PG_GETARG_BOX_P(0);
 
-       return result;
-}      /* box_width() */
+       PG_RETURN_FLOAT8(box->high.x - box->low.x);
+}
 
 
 /*             box_height              -               returns the height of the box
  *                                                               (vertical magnitude).
  */
-double *
-box_height(BOX *box)
+Datum
+box_height(PG_FUNCTION_ARGS)
 {
-       double     *result = palloc(sizeof(double));
-
-       *result = box->high.y - box->low.y;
+       BOX                *box = PG_GETARG_BOX_P(0);
 
-       return result;
+       PG_RETURN_FLOAT8(box->high.y - box->low.y);
 }
 
 
 /*             box_distance    -               returns the distance between the
  *                                                               center points of two boxes.
  */
-double *
-box_distance(BOX *box1, BOX *box2)
+Datum
+box_distance(PG_FUNCTION_ARGS)
 {
-       double     *result = palloc(sizeof(double));
-       Point      *a,
-                          *b;
+       BOX                *box1 = PG_GETARG_BOX_P(0);
+       BOX                *box2 = PG_GETARG_BOX_P(1);
+       Point           a,
+                               b;
 
-       a = box_center(box1);
-       b = box_center(box2);
-       *result = HYPOT(a->x - b->x, a->y - b->y);
+       box_cn(&a, box1);
+       box_cn(&b, box2);
 
-       pfree(a);
-       pfree(b);
-       return result;
+       PG_RETURN_FLOAT8(HYPOT(a.x - b.x, a.y - b.y));
 }
 
 
 /*             box_center              -               returns the center point of the box.
  */
-Point *
-box_center(BOX *box)
+Datum
+box_center(PG_FUNCTION_ARGS)
 {
-       Point      *result = palloc(sizeof(Point));
+       BOX                *box = PG_GETARG_BOX_P(0);
+       Point      *result = (Point *) palloc(sizeof(Point));
 
-       result->x = (box->high.x + box->low.x) / 2.0;
-       result->y = (box->high.y + box->low.y) / 2.0;
+       box_cn(result, box);
 
-       return result;
+       PG_RETURN_POINT_P(result);
 }
 
 
@@ -668,6 +723,16 @@ box_ar(BOX *box)
 }
 
 
+/*             box_cn  -               stores the centerpoint of the box into *center.
+ */
+static void
+box_cn(Point *center, BOX *box)
+{
+       center->x = (box->high.x + box->low.x) / 2.0;
+       center->y = (box->high.y + box->low.y) / 2.0;
+}
+
+
 /*             box_wd  -               returns the width (length) of the box
  *                                                               (horizontal magnitude).
  */
@@ -688,28 +753,6 @@ box_ht(BOX *box)
 }
 
 
-/*             box_dt  -               returns the distance between the
- *                                               center points of two boxes.
- */
-#ifdef NOT_USED
-static double
-box_dt(BOX *box1, BOX *box2)
-{
-       double          result;
-       Point      *a,
-                          *b;
-
-       a = box_center(box1);
-       b = box_center(box2);
-       result = HYPOT(a->x - b->x, a->y - b->y);
-
-       pfree(a);
-       pfree(b);
-       return result;
-}
-
-#endif
-
 /*----------------------------------------------------------
  *     Funky operations.
  *---------------------------------------------------------*/
@@ -718,41 +761,40 @@ box_dt(BOX *box1, BOX *box2)
  *                             returns the overlapping portion of two boxes,
  *                               or NULL if they do not intersect.
  */
-BOX *
-box_intersect(BOX *box1, BOX *box2)
+Datum
+box_intersect(PG_FUNCTION_ARGS)
 {
+       BOX                *box1 = PG_GETARG_BOX_P(0);
+       BOX                *box2 = PG_GETARG_BOX_P(1);
        BOX                *result;
 
-       if (!box_overlap(box1, box2))
-               return NULL;
+       if (!box_ov(box1, box2))
+               PG_RETURN_NULL();
 
-       result = palloc(sizeof(BOX));
+       result = (BOX *) palloc(sizeof(BOX));
 
        result->high.x = Min(box1->high.x, box2->high.x);
        result->low.x = Max(box1->low.x, box2->low.x);
        result->high.y = Min(box1->high.y, box2->high.y);
        result->low.y = Max(box1->low.y, box2->low.y);
 
-       return result;
+       PG_RETURN_BOX_P(result);
 }
 
 
 /*             box_diagonal    -
  *                             returns a line segment which happens to be the
  *                               positive-slope diagonal of "box".
- *                             provided, of course, we have LSEGs.
  */
-LSEG *
-box_diagonal(BOX *box)
+Datum
+box_diagonal(PG_FUNCTION_ARGS)
 {
-       Point           p1,
-                               p2;
+       BOX                *box = PG_GETARG_BOX_P(0);
+       LSEG       *result = (LSEG *) palloc(sizeof(LSEG));
 
-       p1.x = box->high.x;
-       p1.y = box->high.y;
-       p2.x = box->low.x;
-       p2.y = box->low.y;
-       return lseg_construct(&p1, &p2);
+       statlseg_construct(result, &box->high, &box->low);
+
+       PG_RETURN_LSEG_P(result);
 }
 
 /***********************************************************************
@@ -764,50 +806,43 @@ box_diagonal(BOX *box)
  **
  ***********************************************************************/
 
-LINE *
-line_in(char *str)
+Datum
+line_in(PG_FUNCTION_ARGS)
 {
+#ifdef ENABLE_LINE_TYPE
+       char       *str = PG_GETARG_CSTRING(0);
+#endif
        LINE       *line;
-
 #ifdef ENABLE_LINE_TYPE
        LSEG            lseg;
        int                     isopen;
        char       *s;
 
-#endif
-
-       if (!PointerIsValid(str))
-               elog(ERROR, " Bad (null) line external representation");
-
-#ifdef ENABLE_LINE_TYPE
        if ((!path_decode(TRUE, 2, str, &isopen, &s, &(lseg.p[0])))
                || (*s != '\0'))
                elog(ERROR, "Bad line external representation '%s'", str);
 
-       line = line_construct_pp(&(lseg.p[0]), &(lseg.p[1]));
+       line = (LINE *) palloc(sizeof(LINE));
+       line_construct_pts(line, &lseg.p[0], &lseg.p[1]);
 #else
        elog(ERROR, "line not yet implemented");
        line = NULL;
 #endif
 
-       return line;
-}      /* line_in() */
+       PG_RETURN_LINE_P(line);
+}
 
 
-char *
-line_out(LINE *line)
+Datum
+line_out(PG_FUNCTION_ARGS)
 {
+#ifdef ENABLE_LINE_TYPE
+       LINE       *line = PG_GETARG_LINE_P(0);
+#endif
        char       *result;
-
 #ifdef ENABLE_LINE_TYPE
        LSEG            lseg;
 
-#endif
-
-       if (!PointerIsValid(line))
-               return NULL;
-
-#ifdef ENABLE_LINE_TYPE
        if (FPzero(line->B))
        {                                                       /* vertical */
                /* use "x = C" */
@@ -815,7 +850,7 @@ line_out(LINE *line)
                result->B = 0;
                result->C = pt1->x;
 #ifdef GEODEBUG
-               printf("line_construct_pp- line is vertical\n");
+               printf("line_out- line is vertical\n");
 #endif
 #ifdef NOT_USED
                result->m = DBL_MAX;
@@ -829,7 +864,7 @@ line_out(LINE *line)
                result->B = -1;
                result->C = pt1->y;
 #ifdef GEODEBUG
-               printf("line_construct_pp- line is horizontal\n");
+               printf("line_out- line is horizontal\n");
 #endif
 #ifdef NOT_USED
                result->m = 0.0;
@@ -840,10 +875,10 @@ line_out(LINE *line)
        {
        }
 
-       if (line_horizontal(line))
+       if (FPzero(line->A))            /* horizontal? */
        {
        }
-       else if (line_vertical(line))
+       else if (FPzero(line->B))       /* vertical? */
        {
        }
        else
@@ -856,8 +891,8 @@ line_out(LINE *line)
        result = NULL;
 #endif
 
-       return result;
-}      /* line_out() */
+       PG_RETURN_CSTRING(result);
+}
 
 
 /*----------------------------------------------------------
@@ -871,7 +906,7 @@ line_out(LINE *line)
 static LINE *
 line_construct_pm(Point *pt, double m)
 {
-       LINE       *result = palloc(sizeof(LINE));
+       LINE       *result = (LINE *) palloc(sizeof(LINE));
 
        /* use "mx - y + yinter = 0" */
        result->A = m;
@@ -886,126 +921,142 @@ line_construct_pm(Point *pt, double m)
 #endif
 
        return result;
-}      /* line_construct_pm() */
-
+}
 
-/* line_construct_pp()
- * two points
+/*
+ * Fill already-allocated LINE struct from two points on the line
  */
-LINE *
-line_construct_pp(Point *pt1, Point *pt2)
+static void
+line_construct_pts(LINE *line, Point *pt1, Point *pt2)
 {
-       LINE       *result = palloc(sizeof(LINE));
-
        if (FPeq(pt1->x, pt2->x))
        {                                                       /* vertical */
                /* use "x = C" */
-               result->A = -1;
-               result->B = 0;
-               result->C = pt1->x;
-#ifdef GEODEBUG
-               printf("line_construct_pp- line is vertical\n");
-#endif
+               line->A = -1;
+               line->B = 0;
+               line->C = pt1->x;
 #ifdef NOT_USED
-               result->m = DBL_MAX;
+               line->m = DBL_MAX;
+#endif
+#ifdef GEODEBUG
+               printf("line_construct_pts- line is vertical\n");
 #endif
-
        }
        else if (FPeq(pt1->y, pt2->y))
        {                                                       /* horizontal */
                /* use "x = C" */
-               result->A = 0;
-               result->B = -1;
-               result->C = pt1->y;
-#ifdef GEODEBUG
-               printf("line_construct_pp- line is horizontal\n");
-#endif
+               line->A = 0;
+               line->B = -1;
+               line->C = pt1->y;
 #ifdef NOT_USED
-               result->m = 0.0;
+               line->m = 0.0;
+#endif
+#ifdef GEODEBUG
+               printf("line_construct_pts- line is horizontal\n");
 #endif
-
        }
        else
        {
                /* use "mx - y + yinter = 0" */
+               line->A = (pt2->y - pt1->y) / (pt2->x - pt1->x);
+               line->B = -1.0;
+               line->C = pt1->y - line->A * pt1->x;
 #ifdef NOT_USED
-               result->A = (pt1->y - pt2->y) / (pt1->x - pt2->x);
+               line->m = line->A;
 #endif
-               result->A = (pt2->y - pt1->y) / (pt2->x - pt1->x);
-               result->B = -1.0;
-               result->C = pt1->y - result->A * pt1->x;
 #ifdef GEODEBUG
-               printf("line_construct_pp- line is neither vertical nor horizontal (diffs x=%.*g, y=%.*g\n",
+               printf("line_construct_pts- line is neither vertical nor horizontal (diffs x=%.*g, y=%.*g\n",
                           digits8, (pt2->x - pt1->x), digits8, (pt2->y - pt1->y));
-#endif
-#ifdef NOT_USED
-               result->m = result->A;
 #endif
        }
-       return result;
-}      /* line_construct_pp() */
+}
+
+/* line_construct_pp()
+ * two points
+ */
+Datum
+line_construct_pp(PG_FUNCTION_ARGS)
+{
+       Point      *pt1 = PG_GETARG_POINT_P(0);
+       Point      *pt2 = PG_GETARG_POINT_P(1);
+       LINE       *result = (LINE *) palloc(sizeof(LINE));
+
+       line_construct_pts(result, pt1, pt2);
+       PG_RETURN_LINE_P(result);
+}
 
 
 /*----------------------------------------------------------
  *     Relative position routines.
  *---------------------------------------------------------*/
 
-bool
-line_intersect(LINE *l1, LINE *l2)
+Datum
+line_intersect(PG_FUNCTION_ARGS)
 {
-       return !line_parallel(l1, l2);
+       LINE       *l1 = PG_GETARG_LINE_P(0);
+       LINE       *l2 = PG_GETARG_LINE_P(1);
+
+       PG_RETURN_BOOL(! DatumGetBool(DirectFunctionCall2(line_parallel,
+                                                                                                         LinePGetDatum(l1),
+                                                                                                         LinePGetDatum(l2))));
 }
 
-bool
-line_parallel(LINE *l1, LINE *l2)
+Datum
+line_parallel(PG_FUNCTION_ARGS)
 {
+       LINE       *l1 = PG_GETARG_LINE_P(0);
+       LINE       *l2 = PG_GETARG_LINE_P(1);
+
 #ifdef NOT_USED
-       return FPeq(l1->m, l2->m);
+       PG_RETURN_BOOL(FPeq(l1->m, l2->m));
 #endif
        if (FPzero(l1->B))
-               return FPzero(l2->B);
+               PG_RETURN_BOOL(FPzero(l2->B));
 
-       return FPeq(l2->A, l1->A * (l2->B / l1->B));
-}      /* line_parallel() */
+       PG_RETURN_BOOL(FPeq(l2->A, l1->A * (l2->B / l1->B)));
+}
 
-bool
-line_perp(LINE *l1, LINE *l2)
+Datum
+line_perp(PG_FUNCTION_ARGS)
 {
+       LINE       *l1 = PG_GETARG_LINE_P(0);
+       LINE       *l2 = PG_GETARG_LINE_P(1);
+
 #ifdef NOT_USED
        if (l1->m)
-               return FPeq(l2->m / l1->m, -1.0);
+               PG_RETURN_BOOL(FPeq(l2->m / l1->m, -1.0));
        else if (l2->m)
-               return FPeq(l1->m / l2->m, -1.0);
+               PG_RETURN_BOOL(FPeq(l1->m / l2->m, -1.0));
 #endif
        if (FPzero(l1->A))
-               return FPzero(l2->B);
+               PG_RETURN_BOOL(FPzero(l2->B));
        else if (FPzero(l1->B))
-               return FPzero(l2->A);
+               PG_RETURN_BOOL(FPzero(l2->A));
 
-       return FPeq(((l1->A * l2->B) / (l1->B * l2->A)), -1.0);
-}      /* line_perp() */
+       PG_RETURN_BOOL(FPeq(((l1->A * l2->B) / (l1->B * l2->A)), -1.0));
+}
 
-bool
-line_vertical(LINE *line)
+Datum
+line_vertical(PG_FUNCTION_ARGS)
 {
-#ifdef NOT_USED
-       return FPeq(line->A, -1.0) && FPzero(line->B);
-#endif
-       return FPzero(line->B);
-}      /* line_vertical() */
+       LINE       *line = PG_GETARG_LINE_P(0);
+
+       PG_RETURN_BOOL(FPzero(line->B));
+}
 
-bool
-line_horizontal(LINE *line)
+Datum
+line_horizontal(PG_FUNCTION_ARGS)
 {
-#ifdef NOT_USED
-       return FPzero(line->m);
-#endif
-       return FPzero(line->A);
-}      /* line_horizontal() */
+       LINE       *line = PG_GETARG_LINE_P(0);
 
-bool
-line_eq(LINE *l1, LINE *l2)
+       PG_RETURN_BOOL(FPzero(line->A));
+}
+
+Datum
+line_eq(PG_FUNCTION_ARGS)
 {
+       LINE       *l1 = PG_GETARG_LINE_P(0);
+       LINE       *l2 = PG_GETARG_LINE_P(1);
        double          k;
 
        if (!FPzero(l2->A))
@@ -1017,9 +1068,9 @@ line_eq(LINE *l1, LINE *l2)
        else
                k = 1.0;
 
-       return (FPeq(l1->A, k * l2->A) &&
-                       FPeq(l1->B, k * l2->B) &&
-                       FPeq(l1->C, k * l2->C));
+       PG_RETURN_BOOL(FPeq(l1->A, k * l2->A) &&
+                                  FPeq(l1->B, k * l2->B) &&
+                                  FPeq(l1->C, k * l2->C));
 }
 
 
@@ -1030,44 +1081,69 @@ line_eq(LINE *l1, LINE *l2)
 /* line_distance()
  * Distance between two lines.
  */
-double *
-line_distance(LINE *l1, LINE *l2)
+Datum
+line_distance(PG_FUNCTION_ARGS)
 {
-       double     *result = palloc(sizeof(double));
+       LINE       *l1 = PG_GETARG_LINE_P(0);
+       LINE       *l2 = PG_GETARG_LINE_P(1);
+       float8          result;
        Point      *tmp;
 
-       if (line_intersect(l1, l2))
-       {
-               *result = 0.0;
-               return result;
-       }
-       if (line_vertical(l1))
-               *result = fabs(l1->C - l2->C);
-       else
-       {
-               tmp = point_construct(0.0, l1->C);
-               result = dist_pl(tmp, l2);
-               pfree(tmp);
-       }
-       return result;
+       if (! DatumGetBool(DirectFunctionCall2(line_parallel,
+                                                                                  LinePGetDatum(l1),
+                                                                                  LinePGetDatum(l2))))
+               PG_RETURN_FLOAT8(0.0);
+       if (FPzero(l1->B))                      /* vertical? */
+               PG_RETURN_FLOAT8(fabs(l1->C - l2->C));
+       tmp = point_construct(0.0, l1->C);
+       result = dist_pl_internal(tmp, l2);
+       pfree(tmp);
+       PG_RETURN_FLOAT8(result);
 }
 
 /* line_interpt()
  * Point where two lines l1, l2 intersect (if any)
  */
-Point *
-line_interpt(LINE *l1, LINE *l2)
+Datum
+line_interpt(PG_FUNCTION_ARGS)
+{
+       LINE       *l1 = PG_GETARG_LINE_P(0);
+       LINE       *l2 = PG_GETARG_LINE_P(1);
+       Point      *result;
+
+       result = line_interpt_internal(l1, l2);
+
+       if (result == NULL)
+               PG_RETURN_NULL();
+       PG_RETURN_POINT_P(result);
+}
+
+/*
+ * Internal version of line_interpt
+ *
+ * returns a NULL pointer if no intersection point
+ */
+static Point *
+line_interpt_internal(LINE *l1, LINE *l2)
 {
        Point      *result;
        double          x,
                                y;
 
-       if (line_parallel(l1, l2))
+       /*
+        * NOTE: if the lines are identical then we will find they are parallel
+        * and report "no intersection".  This is a little weird, but since
+        * there's no *unique* intersection, maybe it's appropriate behavior.
+        */
+       if (DatumGetBool(DirectFunctionCall2(line_parallel,
+                                                                                LinePGetDatum(l1),
+                                                                                LinePGetDatum(l2))))
                return NULL;
+
 #ifdef NOT_USED
-       if (line_vertical(l1))
+       if (FPzero(l1->B))                      /* l1 vertical? */
                result = point_construct(l2->m * l1->C + l2->C, l1->C);
-       else if (line_vertical(l2))
+       else if (FPzero(l2->B))         /* l2 vertical? */
                result = point_construct(l1->m * l2->C + l1->C, l2->C);
        else
        {
@@ -1076,32 +1152,18 @@ line_interpt(LINE *l1, LINE *l2)
        }
 #endif
 
-       if (line_vertical(l1))
+       if (FPzero(l1->B))                      /* l1 vertical? */
        {
-#ifdef NOT_USED
-               x = l1->C;
-               y = -((l2->A * x + l2->C) / l2->B);
-#endif
                x = l1->C;
                y = (l2->A * x + l2->C);
-
        }
-       else if (line_vertical(l2))
+       else if (FPzero(l2->B))         /* l2 vertical? */
        {
-#ifdef NOT_USED
-               x = l2->C;
-               y = -((l1->A * x + l1->C) / l1->B);
-#endif
                x = l2->C;
                y = (l1->A * x + l1->C);
-
        }
        else
        {
-#ifdef NOT_USED
-               x = (l2->B * l1->C - l1->B * l2->C) / (l2->A * l1->B - l1->A * l2->B);
-               y = -((l1->A * x + l1->C) / l1->B);
-#endif
                x = (l1->C - l2->C) / (l2->A - l1->A);
                y = (l1->A * x + l1->C);
        }
@@ -1112,8 +1174,9 @@ line_interpt(LINE *l1, LINE *l2)
                   digits8, l1->A, digits8, l1->B, digits8, l1->C, digits8, l2->A, digits8, l2->B, digits8, l2->C);
        printf("line_interpt- lines intersect at (%.*g,%.*g)\n", digits8, x, digits8, y);
 #endif
+
        return result;
-}      /* line_interpt() */
+}
 
 
 /***********************************************************************
@@ -1164,7 +1227,7 @@ path_in(PG_FUNCTION_ARGS)
        }
 
        size = offsetof(PATH, p[0]) + sizeof(path->p[0]) * npts;
-       path = palloc(size);
+       path = (PATH *) palloc(size);
 
        path->size = size;
        path->npts = npts;
@@ -1326,7 +1389,7 @@ path_inter(PG_FUNCTION_ARGS)
                b2.low.x = Min(p2->p[i].x, b2.low.x);
                b2.low.y = Min(p2->p[i].y, b2.low.y);
        }
-       if (!box_overlap(&b1, &b2))
+       if (!box_ov(&b1, &b2))
                PG_RETURN_BOOL(false);
 
        /* pairwise check lseg intersections */
@@ -1336,7 +1399,7 @@ path_inter(PG_FUNCTION_ARGS)
                {
                        statlseg_construct(&seg1, &p1->p[i], &p1->p[i + 1]);
                        statlseg_construct(&seg2, &p2->p[j], &p2->p[j + 1]);
-                       if (lseg_intersect(&seg1, &seg2))
+                       if (lseg_intersect_internal(&seg1, &seg2))
                                PG_RETURN_BOOL(true);
                }
        }
@@ -1347,15 +1410,15 @@ path_inter(PG_FUNCTION_ARGS)
 
 /* path_distance()
  * This essentially does a cartesian product of the lsegs in the
- *     two paths, and finds the max distance between any two lsegs
+ *     two paths, and finds the min distance between any two lsegs
  */
 Datum
 path_distance(PG_FUNCTION_ARGS)
 {
        PATH       *p1 = PG_GETARG_PATH_P(0);
        PATH       *p2 = PG_GETARG_PATH_P(1);
-       bool            have_max = false;
-       float8          max = 0.0;              /* initialize to keep compiler quiet */
+       bool            have_min = false;
+       float8          min = 0.0;              /* initialize to keep compiler quiet */
        float8          tmp;
        int                     i,
                                j;
@@ -1369,19 +1432,21 @@ path_distance(PG_FUNCTION_ARGS)
                        statlseg_construct(&seg1, &p1->p[i], &p1->p[i + 1]);
                        statlseg_construct(&seg2, &p2->p[j], &p2->p[j + 1]);
 
-                       tmp = *lseg_distance(&seg1, &seg2);
-                       if (!have_max || max < tmp)
+                       tmp = DatumGetFloat8(DirectFunctionCall2(lseg_distance,
+                                                                                                        LsegPGetDatum(&seg1),
+                                                                                                        LsegPGetDatum(&seg2)));
+                       if (!have_min || tmp < min)
                        {
-                               max = tmp;
-                               have_max = true;
+                               min = tmp;
+                               have_min = true;
                        }
                }
        }
 
-       if (! have_max)
+       if (! have_min)
                PG_RETURN_NULL();
 
-       PG_RETURN_FLOAT8(max);
+       PG_RETURN_FLOAT8(min);
 }
 
 
@@ -1416,43 +1481,39 @@ path_length(PG_FUNCTION_ARGS)
  *                             "x,y"
  *---------------------------------------------------------*/
 
-Point *
-point_in(char *str)
+Datum
+point_in(PG_FUNCTION_ARGS)
 {
+       char       *str = PG_GETARG_CSTRING(0);
        Point      *point;
-
        double          x,
                                y;
        char       *s;
 
-       if (!PointerIsValid(str))
-               elog(ERROR, "Bad (null) point external representation");
-
-       if (!pair_decode(str, &x, &y, &s) || (strlen(s) > 0))
+       if (!pair_decode(str, &x, &y, &s) || (*s != '\0'))
                elog(ERROR, "Bad point external representation '%s'", str);
 
-       point = palloc(sizeof(Point));
+       point = (Point *) palloc(sizeof(Point));
 
        point->x = x;
        point->y = y;
 
-       return point;
-}      /* point_in() */
+       PG_RETURN_POINT_P(point);
+}
 
-char *
-point_out(Point *pt)
+Datum
+point_out(PG_FUNCTION_ARGS)
 {
-       if (!PointerIsValid(pt))
-               return NULL;
+       Point      *pt = PG_GETARG_POINT_P(0);
 
-       return path_encode(-1, 1, pt);
-}      /* point_out() */
+       PG_RETURN_CSTRING(path_encode(-1, 1, pt));
+}
 
 
 static Point *
 point_construct(double x, double y)
 {
-       Point      *result = palloc(sizeof(Point));
+       Point      *result = (Point *) palloc(sizeof(Point));
 
        result->x = x;
        result->y = y;
@@ -1468,7 +1529,7 @@ point_copy(Point *pt)
        if (!PointerIsValid(pt))
                return NULL;
 
-       result = palloc(sizeof(Point));
+       result = (Point *) palloc(sizeof(Point));
 
        result->x = pt->x;
        result->y = pt->y;
@@ -1485,77 +1546,91 @@ point_copy(Point *pt)
  *             EPSILON = 0.0).
  *---------------------------------------------------------*/
 
-bool
-point_left(Point *pt1, Point *pt2)
+Datum
+point_left(PG_FUNCTION_ARGS)
 {
-       return FPlt(pt1->x, pt2->x);
+       Point      *pt1 = PG_GETARG_POINT_P(0);
+       Point      *pt2 = PG_GETARG_POINT_P(1);
+
+       PG_RETURN_BOOL(FPlt(pt1->x, pt2->x));
 }
 
-bool
-point_right(Point *pt1, Point *pt2)
+Datum
+point_right(PG_FUNCTION_ARGS)
 {
-       return FPgt(pt1->x, pt2->x);
+       Point      *pt1 = PG_GETARG_POINT_P(0);
+       Point      *pt2 = PG_GETARG_POINT_P(1);
+
+       PG_RETURN_BOOL(FPgt(pt1->x, pt2->x));
 }
 
-bool
-point_above(Point *pt1, Point *pt2)
+Datum
+point_above(PG_FUNCTION_ARGS)
 {
-       return FPgt(pt1->y, pt2->y);
+       Point      *pt1 = PG_GETARG_POINT_P(0);
+       Point      *pt2 = PG_GETARG_POINT_P(1);
+
+       PG_RETURN_BOOL(FPgt(pt1->y, pt2->y));
 }
 
-bool
-point_below(Point *pt1, Point *pt2)
+Datum
+point_below(PG_FUNCTION_ARGS)
 {
-       return FPlt(pt1->y, pt2->y);
+       Point      *pt1 = PG_GETARG_POINT_P(0);
+       Point      *pt2 = PG_GETARG_POINT_P(1);
+
+       PG_RETURN_BOOL(FPlt(pt1->y, pt2->y));
 }
 
-bool
-point_vert(Point *pt1, Point *pt2)
+Datum
+point_vert(PG_FUNCTION_ARGS)
 {
-       return FPeq(pt1->x, pt2->x);
+       Point      *pt1 = PG_GETARG_POINT_P(0);
+       Point      *pt2 = PG_GETARG_POINT_P(1);
+
+       PG_RETURN_BOOL(FPeq(pt1->x, pt2->x));
 }
 
-bool
-point_horiz(Point *pt1, Point *pt2)
+Datum
+point_horiz(PG_FUNCTION_ARGS)
 {
-       return FPeq(pt1->y, pt2->y);
+       Point      *pt1 = PG_GETARG_POINT_P(0);
+       Point      *pt2 = PG_GETARG_POINT_P(1);
+
+       PG_RETURN_BOOL(FPeq(pt1->y, pt2->y));
 }
 
-bool
-point_eq(Point *pt1, Point *pt2)
+Datum
+point_eq(PG_FUNCTION_ARGS)
 {
-       return point_horiz(pt1, pt2) && point_vert(pt1, pt2);
+       Point      *pt1 = PG_GETARG_POINT_P(0);
+       Point      *pt2 = PG_GETARG_POINT_P(1);
+
+       PG_RETURN_BOOL(FPeq(pt1->x, pt2->x) && FPeq(pt1->y, pt2->y));
 }
 
-bool
-point_ne(Point *pt1, Point *pt2)
+Datum
+point_ne(PG_FUNCTION_ARGS)
 {
-       return !point_eq(pt1, pt2);
+       Point      *pt1 = PG_GETARG_POINT_P(0);
+       Point      *pt2 = PG_GETARG_POINT_P(1);
+
+       PG_RETURN_BOOL(FPne(pt1->x, pt2->x) || FPne(pt1->y, pt2->y));
 }
 
 /*----------------------------------------------------------
  *     "Arithmetic" operators on points.
  *---------------------------------------------------------*/
 
-int32
-pointdist(Point *p1, Point *p2)
-{
-       int32           result;
-
-       result = point_dt(p1, p2);
-       return result;
-}
-
-double *
-point_distance(Point *pt1, Point *pt2)
+Datum
+point_distance(PG_FUNCTION_ARGS)
 {
-       double     *result = palloc(sizeof(double));
+       Point      *pt1 = PG_GETARG_POINT_P(0);
+       Point      *pt2 = PG_GETARG_POINT_P(1);
 
-       *result = HYPOT(pt1->x - pt2->x, pt1->y - pt2->y);
-       return result;
+       PG_RETURN_FLOAT8(HYPOT(pt1->x - pt2->x, pt1->y - pt2->y));
 }
 
-
 double
 point_dt(Point *pt1, Point *pt2)
 {
@@ -1566,23 +1641,20 @@ point_dt(Point *pt1, Point *pt2)
        return HYPOT(pt1->x - pt2->x, pt1->y - pt2->y);
 }
 
-double *
-point_slope(Point *pt1, Point *pt2)
+Datum
+point_slope(PG_FUNCTION_ARGS)
 {
-       double     *result = palloc(sizeof(double));
+       Point      *pt1 = PG_GETARG_POINT_P(0);
+       Point      *pt2 = PG_GETARG_POINT_P(1);
 
-       if (point_vert(pt1, pt2))
-               *result = (double) DBL_MAX;
-       else
-               *result = (pt1->y - pt2->y) / (pt1->x - pt1->x);
-       return result;
+       PG_RETURN_FLOAT8(point_sl(pt1, pt2));
 }
 
 
 double
 point_sl(Point *pt1, Point *pt2)
 {
-       return (point_vert(pt1, pt2)
+       return (FPeq(pt1->x, pt2->x)
                        ? (double) DBL_MAX
                        : (pt1->y - pt2->y) / (pt1->x - pt2->x));
 }
@@ -1603,18 +1675,15 @@ point_sl(Point *pt1, Point *pt2)
  *             (old form)              "(x1, y1, x2, y2)"
  *---------------------------------------------------------*/
 
-LSEG *
-lseg_in(char *str)
+Datum
+lseg_in(PG_FUNCTION_ARGS)
 {
+       char       *str = PG_GETARG_CSTRING(0);
        LSEG       *lseg;
-
        int                     isopen;
        char       *s;
 
-       if (!PointerIsValid(str))
-               elog(ERROR, " Bad (null) lseg external representation");
-
-       lseg = palloc(sizeof(LSEG));
+       lseg = (LSEG *) palloc(sizeof(LSEG));
 
        if ((!path_decode(TRUE, 2, str, &isopen, &s, &(lseg->p[0])))
                || (*s != '\0'))
@@ -1624,27 +1693,28 @@ lseg_in(char *str)
        lseg->m = point_sl(&lseg->p[0], &lseg->p[1]);
 #endif
 
-       return lseg;
-}      /* lseg_in() */
+       PG_RETURN_LSEG_P(lseg);
+}
 
 
-char *
-lseg_out(LSEG *ls)
+Datum
+lseg_out(PG_FUNCTION_ARGS)
 {
-       if (!PointerIsValid(ls))
-               return NULL;
+       LSEG       *ls = PG_GETARG_LSEG_P(0);
 
-       return path_encode(FALSE, 2, (Point *) &(ls->p[0]));
-}      /* lseg_out() */
+       PG_RETURN_CSTRING(path_encode(FALSE, 2, (Point *) &(ls->p[0])));
+}
 
 
 /* lseg_construct -
  *             form a LSEG from two Points.
  */
-LSEG *
-lseg_construct(Point *pt1, Point *pt2)
+Datum
+lseg_construct(PG_FUNCTION_ARGS)
 {
-       LSEG       *result = palloc(sizeof(LSEG));
+       Point      *pt1 = PG_GETARG_POINT_P(0);
+       Point      *pt2 = PG_GETARG_POINT_P(1);
+       LSEG       *result = (LSEG *) palloc(sizeof(LSEG));
 
        result->p[0].x = pt1->x;
        result->p[0].y = pt1->y;
@@ -1655,7 +1725,7 @@ lseg_construct(Point *pt1, Point *pt2)
        result->m = point_sl(pt1, pt2);
 #endif
 
-       return result;
+       PG_RETURN_LSEG_P(result);
 }
 
 /* like lseg_construct, but assume space already allocated */
@@ -1672,18 +1742,13 @@ statlseg_construct(LSEG *lseg, Point *pt1, Point *pt2)
 #endif
 }
 
-double *
-lseg_length(LSEG *lseg)
+Datum
+lseg_length(PG_FUNCTION_ARGS)
 {
-       double     *result;
-
-       if (!PointerIsValid(lseg))
-               return NULL;
+       LSEG       *lseg = PG_GETARG_LSEG_P(0);
 
-       result = point_distance(&lseg->p[0], &lseg->p[1]);
-
-       return result;
-}      /* lseg_length() */
+       PG_RETURN_FLOAT8(point_dt(&lseg->p[0], &lseg->p[1]));
+}
 
 /*----------------------------------------------------------
  *     Relative position routines.
@@ -1693,35 +1758,46 @@ lseg_length(LSEG *lseg)
  **  find intersection of the two lines, and see if it falls on
  **  both segments.
  */
-bool
-lseg_intersect(LSEG *l1, LSEG *l2)
+Datum
+lseg_intersect(PG_FUNCTION_ARGS)
 {
-       LINE       *ln;
+       LSEG       *l1 = PG_GETARG_LSEG_P(0);
+       LSEG       *l2 = PG_GETARG_LSEG_P(1);
+
+       PG_RETURN_BOOL(lseg_intersect_internal(l1, l2));
+}
+
+static bool
+lseg_intersect_internal(LSEG *l1, LSEG *l2)
+{
+       LINE            ln;
        Point      *interpt;
        bool            retval;
 
-       ln = line_construct_pp(&l2->p[0], &l2->p[1]);
-       interpt = interpt_sl(l1, ln);
+       line_construct_pts(&ln, &l2->p[0], &l2->p[1]);
+       interpt = interpt_sl(l1, &ln);
 
-       if (interpt != NULL && on_ps(interpt, l2))      /* interpt on l1 and l2 */
-               retval = TRUE;
+       if (interpt != NULL && on_ps_internal(interpt, l2))
+               retval = true;                  /* interpt on l1 and l2 */
        else
-               retval = FALSE;
+               retval = false;
        if (interpt != NULL)
                pfree(interpt);
-       pfree(ln);
        return retval;
 }
 
-bool
-lseg_parallel(LSEG *l1, LSEG *l2)
+Datum
+lseg_parallel(PG_FUNCTION_ARGS)
 {
+       LSEG       *l1 = PG_GETARG_LSEG_P(0);
+       LSEG       *l2 = PG_GETARG_LSEG_P(1);
+
 #ifdef NOT_USED
-       return FPeq(l1->m, l2->m);
+       PG_RETURN_BOOL(FPeq(l1->m, l2->m));
 #endif
-       return (FPeq(point_sl(&(l1->p[0]), &(l1->p[1])),
-                                point_sl(&(l2->p[0]), &(l2->p[1]))));
-}      /* lseg_parallel() */
+       PG_RETURN_BOOL(FPeq(point_sl(&l1->p[0], &l1->p[1]),
+                                               point_sl(&l2->p[0], &l2->p[1])));
+}
 
 /* lseg_perp()
  * Determine if two line segments are perpendicular.
@@ -1732,9 +1808,11 @@ lseg_parallel(LSEG *l1, LSEG *l2)
  *     returned by point_sl() and the results seem better.
  * - thomas 1998-01-31
  */
-bool
-lseg_perp(LSEG *l1, LSEG *l2)
+Datum
+lseg_perp(PG_FUNCTION_ARGS)
 {
+       LSEG       *l1 = PG_GETARG_LSEG_P(0);
+       LSEG       *l2 = PG_GETARG_LSEG_P(1);
        double          m1,
                                m2;
 
@@ -1745,67 +1823,93 @@ lseg_perp(LSEG *l1, LSEG *l2)
        printf("lseg_perp- slopes are %g and %g\n", m1, m2);
 #endif
        if (FPzero(m1))
-               return FPeq(m2, DBL_MAX);
+               PG_RETURN_BOOL(FPeq(m2, DBL_MAX));
        else if (FPzero(m2))
-               return FPeq(m1, DBL_MAX);
+               PG_RETURN_BOOL(FPeq(m1, DBL_MAX));
 
-       return FPeq(m1 / m2, -1.0);
-}      /* lseg_perp() */
+       PG_RETURN_BOOL(FPeq(m1 / m2, -1.0));
+}
 
-bool
-lseg_vertical(LSEG *lseg)
+Datum
+lseg_vertical(PG_FUNCTION_ARGS)
 {
-       return FPeq(lseg->p[0].x, lseg->p[1].x);
+       LSEG       *lseg = PG_GETARG_LSEG_P(0);
+
+       PG_RETURN_BOOL(FPeq(lseg->p[0].x, lseg->p[1].x));
 }
 
-bool
-lseg_horizontal(LSEG *lseg)
+Datum
+lseg_horizontal(PG_FUNCTION_ARGS)
 {
-       return FPeq(lseg->p[0].y, lseg->p[1].y);
+       LSEG       *lseg = PG_GETARG_LSEG_P(0);
+
+       PG_RETURN_BOOL(FPeq(lseg->p[0].y, lseg->p[1].y));
 }
 
 
-bool
-lseg_eq(LSEG *l1, LSEG *l2)
+Datum
+lseg_eq(PG_FUNCTION_ARGS)
 {
-       return (FPeq(l1->p[0].x, l2->p[0].x) &&
-                       FPeq(l1->p[1].y, l2->p[1].y) &&
-                       FPeq(l1->p[0].x, l2->p[0].x) &&
-                       FPeq(l1->p[1].y, l2->p[1].y));
-}      /* lseg_eq() */
+       LSEG       *l1 = PG_GETARG_LSEG_P(0);
+       LSEG       *l2 = PG_GETARG_LSEG_P(1);
+
+       PG_RETURN_BOOL(FPeq(l1->p[0].x, l2->p[0].x) &&
+                                  FPeq(l1->p[1].y, l2->p[1].y) &&
+                                  FPeq(l1->p[0].x, l2->p[0].x) &&
+                                  FPeq(l1->p[1].y, l2->p[1].y));
+}
 
-bool
-lseg_ne(LSEG *l1, LSEG *l2)
+Datum
+lseg_ne(PG_FUNCTION_ARGS)
 {
-       return (!FPeq(l1->p[0].x, l2->p[0].x) ||
-                       !FPeq(l1->p[1].y, l2->p[1].y) ||
-                       !FPeq(l1->p[0].x, l2->p[0].x) ||
-                       !FPeq(l1->p[1].y, l2->p[1].y));
-}      /* lseg_ne() */
+       LSEG       *l1 = PG_GETARG_LSEG_P(0);
+       LSEG       *l2 = PG_GETARG_LSEG_P(1);
 
-bool
-lseg_lt(LSEG *l1, LSEG *l2)
+       PG_RETURN_BOOL(!FPeq(l1->p[0].x, l2->p[0].x) ||
+                                  !FPeq(l1->p[1].y, l2->p[1].y) ||
+                                  !FPeq(l1->p[0].x, l2->p[0].x) ||
+                                  !FPeq(l1->p[1].y, l2->p[1].y));
+}
+
+Datum
+lseg_lt(PG_FUNCTION_ARGS)
 {
-       return FPlt(point_dt(&l1->p[0], &l1->p[1]), point_dt(&l2->p[0], &l2->p[1]));
-}      /* lseg_lt() */
+       LSEG       *l1 = PG_GETARG_LSEG_P(0);
+       LSEG       *l2 = PG_GETARG_LSEG_P(1);
+
+       PG_RETURN_BOOL(FPlt(point_dt(&l1->p[0], &l1->p[1]),
+                                               point_dt(&l2->p[0], &l2->p[1])));
+}
 
-bool
-lseg_le(LSEG *l1, LSEG *l2)
+Datum
+lseg_le(PG_FUNCTION_ARGS)
 {
-       return FPle(point_dt(&l1->p[0], &l1->p[1]), point_dt(&l2->p[0], &l2->p[1]));
-}      /* lseg_le() */
+       LSEG       *l1 = PG_GETARG_LSEG_P(0);
+       LSEG       *l2 = PG_GETARG_LSEG_P(1);
+
+       PG_RETURN_BOOL(FPle(point_dt(&l1->p[0], &l1->p[1]),
+                                               point_dt(&l2->p[0], &l2->p[1])));
+}
 
-bool
-lseg_gt(LSEG *l1, LSEG *l2)
+Datum
+lseg_gt(PG_FUNCTION_ARGS)
 {
-       return FPgt(point_dt(&l1->p[0], &l1->p[1]), point_dt(&l2->p[0], &l2->p[1]));
-}      /* lseg_gt() */
+       LSEG       *l1 = PG_GETARG_LSEG_P(0);
+       LSEG       *l2 = PG_GETARG_LSEG_P(1);
+
+       PG_RETURN_BOOL(FPgt(point_dt(&l1->p[0], &l1->p[1]),
+                                               point_dt(&l2->p[0], &l2->p[1])));
+}
 
-bool
-lseg_ge(LSEG *l1, LSEG *l2)
+Datum
+lseg_ge(PG_FUNCTION_ARGS)
 {
-       return FPge(point_dt(&l1->p[0], &l1->p[1]), point_dt(&l2->p[0], &l2->p[1]));
-}      /* lseg_ge() */
+       LSEG       *l1 = PG_GETARG_LSEG_P(0);
+       LSEG       *l2 = PG_GETARG_LSEG_P(1);
+
+       PG_RETURN_BOOL(FPge(point_dt(&l1->p[0], &l1->p[1]),
+                                               point_dt(&l2->p[0], &l2->p[1])));
+}
 
 
 /*----------------------------------------------------------
@@ -1817,14 +1921,13 @@ lseg_ge(LSEG *l1, LSEG *l2)
  *             point will be from one of the endpoints to the other
  *             segment.
  */
-double *
-lseg_distance(LSEG *l1, LSEG *l2)
+Datum
+lseg_distance(PG_FUNCTION_ARGS)
 {
-       double     *result = palloc(sizeof(double));
-
-       *result = lseg_dt(l1, l2);
+       LSEG       *l1 = PG_GETARG_LSEG_P(0);
+       LSEG       *l2 = PG_GETARG_LSEG_P(1);
 
-       return result;
+       PG_RETURN_FLOAT8(lseg_dt(l1, l2));
 }
 
 /* lseg_dt()
@@ -1835,97 +1938,90 @@ lseg_distance(LSEG *l1, LSEG *l2)
 static double
 lseg_dt(LSEG *l1, LSEG *l2)
 {
-       double     *d,
-                               result;
+       double          result,
+                               d;
 
-       if (lseg_intersect(l1, l2))
+       if (lseg_intersect_internal(l1, l2))
                return 0.0;
 
-       d = dist_ps(&l1->p[0], l2);
-       result = *d;
-       pfree(d);
-       d = dist_ps(&l1->p[1], l2);
-       result = Min(result, *d);
-       pfree(d);
-       d = dist_ps(&l2->p[0], l1);
-       result = Min(result, *d);
-       pfree(d);
-       d = dist_ps(&l2->p[1], l1);
-       result = Min(result, *d);
-       pfree(d);
+       d = dist_ps_internal(&l1->p[0], l2);
+       result = d;
+       d = dist_ps_internal(&l1->p[1], l2);
+       result = Min(result, d);
+       d = dist_ps_internal(&l2->p[0], l1);
+       result = Min(result, d);
+       d = dist_ps_internal(&l2->p[1], l1);
+       result = Min(result, d);
 
        return result;
-}      /* lseg_dt() */
+}
 
 
-Point *
-lseg_center(LSEG *lseg)
+Datum
+lseg_center(PG_FUNCTION_ARGS)
 {
+       LSEG       *lseg = PG_GETARG_LSEG_P(0);
        Point      *result;
 
-       if (!PointerIsValid(lseg))
-               return NULL;
-
-       result = palloc(sizeof(Point));
+       result = (Point *) palloc(sizeof(Point));
 
-       result->x = (lseg->p[0].x - lseg->p[1].x) / 2;
-       result->y = (lseg->p[0].y - lseg->p[1].y) / 2;
+       result->x = (lseg->p[0].x - lseg->p[1].x) / 2.0;
+       result->y = (lseg->p[0].y - lseg->p[1].y) / 2.0;
 
-       return result;
-}      /* lseg_center() */
+       PG_RETURN_POINT_P(result);
+}
 
 
 /* lseg_interpt -
  *             Find the intersection point of two segments (if any).
- *             Find the intersection of the appropriate lines; if the
- *             point is not on a given segment, there is no valid segment
- *             intersection point at all.
- * If there is an intersection, then check explicitly for matching
- *     endpoints since there may be rounding effects with annoying
- *     lsb residue. - tgl 1997-07-09
  */
-Point *
-lseg_interpt(LSEG *l1, LSEG *l2)
+Datum
+lseg_interpt(PG_FUNCTION_ARGS)
 {
+       LSEG       *l1 = PG_GETARG_LSEG_P(0);
+       LSEG       *l2 = PG_GETARG_LSEG_P(1);
        Point      *result;
-       LINE       *tmp1,
-                          *tmp2;
-
-       if (!PointerIsValid(l1) || !PointerIsValid(l2))
-               return NULL;
+       LINE            tmp1,
+                               tmp2;
 
-       tmp1 = line_construct_pp(&l1->p[0], &l1->p[1]);
-       tmp2 = line_construct_pp(&l2->p[0], &l2->p[1]);
-       result = line_interpt(tmp1, tmp2);
-       if (PointerIsValid(result))
+       /*
+        * Find the intersection of the appropriate lines, if any.
+        */
+       line_construct_pts(&tmp1, &l1->p[0], &l1->p[1]);
+       line_construct_pts(&tmp2, &l2->p[0], &l2->p[1]);
+       result = line_interpt_internal(&tmp1, &tmp2);
+       if (!PointerIsValid(result))
+               PG_RETURN_NULL();
+       /*
+        * If the line intersection point isn't within l1 (or equivalently l2),
+        * there is no valid segment intersection point at all.
+        */
+       if (!on_ps_internal(result, l1) ||
+               !on_ps_internal(result, l2))
        {
-               if (on_ps(result, l1))
-               {
-                       if ((FPeq(l1->p[0].x, l2->p[0].x) && FPeq(l1->p[0].y, l2->p[0].y))
-                               || (FPeq(l1->p[0].x, l2->p[1].x) && FPeq(l1->p[0].y, l2->p[1].y)))
-                       {
-                               result->x = l1->p[0].x;
-                               result->y = l1->p[0].y;
-
-                       }
-                       else if ((FPeq(l1->p[1].x, l2->p[0].x) && FPeq(l1->p[1].y, l2->p[0].y))
-                                        || (FPeq(l1->p[1].x, l2->p[1].x) && FPeq(l1->p[1].y, l2->p[1].y)))
-                       {
-                               result->x = l1->p[1].x;
-                               result->y = l1->p[1].y;
-                       }
-               }
-               else
-               {
-                       pfree(result);
-                       result = NULL;
-               }
+               pfree(result);
+               PG_RETURN_NULL();
+       }
+       /*
+        * If there is an intersection, then check explicitly for matching
+        *      endpoints since there may be rounding effects with annoying
+        *      lsb residue. - tgl 1997-07-09
+        */
+       if ((FPeq(l1->p[0].x, l2->p[0].x) && FPeq(l1->p[0].y, l2->p[0].y)) ||
+               (FPeq(l1->p[0].x, l2->p[1].x) && FPeq(l1->p[0].y, l2->p[1].y)))
+       {
+               result->x = l1->p[0].x;
+               result->y = l1->p[0].y;
+       }
+       else if ((FPeq(l1->p[1].x, l2->p[0].x) && FPeq(l1->p[1].y, l2->p[0].y)) ||
+                        (FPeq(l1->p[1].x, l2->p[1].x) && FPeq(l1->p[1].y, l2->p[1].y)))
+       {
+               result->x = l1->p[1].x;
+               result->y = l1->p[1].y;
        }
-       pfree(tmp1);
-       pfree(tmp2);
 
-       return result;
-}      /* lseg_interpt() */
+       PG_RETURN_POINT_P(result);
+}
 
 /***********************************************************************
  **
@@ -1934,34 +2030,43 @@ lseg_interpt(LSEG *l1, LSEG *l2)
  **
  ***********************************************************************/
 
-#define ABOVE  1
-#define BELOW  0
-#define UNDEF  -1
-
-
 /*---------------------------------------------------------------------
  *             dist_
  *                             Minimum distance from one object to another.
  *-------------------------------------------------------------------*/
 
-double *
-dist_pl(Point *pt, LINE *line)
+Datum
+dist_pl(PG_FUNCTION_ARGS)
 {
-       double     *result = palloc(sizeof(double));
+       Point      *pt = PG_GETARG_POINT_P(0);
+       LINE       *line = PG_GETARG_LINE_P(1);
+
+       PG_RETURN_FLOAT8(dist_pl_internal(pt, line));
+}
 
-       *result = (line->A * pt->x + line->B * pt->y + line->C) /
+static double
+dist_pl_internal(Point *pt, LINE *line)
+{
+       return (line->A * pt->x + line->B * pt->y + line->C) /
                HYPOT(line->A, line->B);
+}
 
-       return result;
+Datum
+dist_ps(PG_FUNCTION_ARGS)
+{
+       Point      *pt = PG_GETARG_POINT_P(0);
+       LSEG       *lseg = PG_GETARG_LSEG_P(1);
+
+       PG_RETURN_FLOAT8(dist_ps_internal(pt, lseg));
 }
 
-double *
-dist_ps(Point *pt, LSEG *lseg)
+static double
+dist_ps_internal(Point *pt, LSEG *lseg)
 {
        double          m;                              /* slope of perp. */
        LINE       *ln;
-       double     *result,
-                          *tmpdist;
+       double          result,
+                               tmpdist;
        Point      *ip;
 
 /*
@@ -1976,10 +2081,6 @@ dist_ps(Point *pt, LSEG *lseg)
        }
        else
        {
-#ifdef NOT_USED
-               m = (-1) * (lseg->p[1].y - lseg->p[0].y) /
-                       (lseg->p[1].x - lseg->p[0].x);
-#endif
                m = ((lseg->p[0].y - lseg->p[1].y) / (lseg->p[1].x - lseg->p[0].x));
        }
        ln = line_construct_pm(pt, m);
@@ -1997,30 +2098,27 @@ dist_ps(Point *pt, LSEG *lseg)
        /* intersection is on the line segment? */
        if ((ip = interpt_sl(lseg, ln)) != NULL)
        {
-               result = point_distance(pt, ip);
+               result = point_dt(pt, ip);
 #ifdef GEODEBUG
                printf("dist_ps- distance is %f to intersection point is (%f,%f)\n",
-                          *result, ip->x, ip->y);
+                          result, ip->x, ip->y);
 #endif
-
-               /* otherwise, intersection is not on line segment */
+               pfree(ip);
        }
        else
        {
-               result = point_distance(pt, &lseg->p[0]);
-               tmpdist = point_distance(pt, &lseg->p[1]);
-               if (*tmpdist < *result)
-                       *result = *tmpdist;
-               pfree(tmpdist);
+               /* intersection is not on line segment */
+               result = point_dt(pt, &lseg->p[0]);
+               tmpdist = point_dt(pt, &lseg->p[1]);
+               if (tmpdist < result)
+                       result = tmpdist;
        }
 
-       if (ip != NULL)
-               pfree(ip);
        pfree(ln);
+
        return result;
 }
 
-
 /*
  ** Distance from a point to a path
  */
@@ -2041,7 +2139,7 @@ dist_ppath(PG_FUNCTION_ARGS)
                        PG_RETURN_NULL();
                        /* one point in path? then get distance between two points... */
                case 1:
-                       result = *point_distance(pt, &path->p[0]);
+                       result = point_dt(pt, &path->p[0]);
                        break;
                default:
                        /* make sure the path makes sense... */
@@ -2054,7 +2152,7 @@ dist_ppath(PG_FUNCTION_ARGS)
                        for (i = 0; i < path->npts - 1; i++)
                        {
                                statlseg_construct(&lseg, &path->p[i], &path->p[i + 1]);
-                               tmp = *dist_ps(pt, &lseg);
+                               tmp = dist_ps_internal(pt, &lseg);
                                if (i == 0 || tmp < result)
                                        result = tmp;
                        }
@@ -2063,90 +2161,81 @@ dist_ppath(PG_FUNCTION_ARGS)
        PG_RETURN_FLOAT8(result);
 }
 
-double *
-dist_pb(Point *pt, BOX *box)
+Datum
+dist_pb(PG_FUNCTION_ARGS)
 {
+       Point      *pt = PG_GETARG_POINT_P(0);
+       BOX                *box = PG_GETARG_BOX_P(1);
+       float8          result;
        Point      *tmp;
-       double     *result;
 
-       tmp = close_pb(pt, box);
-       result = point_distance(tmp, pt);
+       tmp = DatumGetPointP(DirectFunctionCall2(close_pb,
+                                                                                        PointPGetDatum(pt),
+                                                                                        BoxPGetDatum(box)));
+       result = point_dt(tmp, pt);
        pfree(tmp);
 
-       return result;
+       PG_RETURN_FLOAT8(result);
 }
 
 
-double *
-dist_sl(LSEG *lseg, LINE *line)
+Datum
+dist_sl(PG_FUNCTION_ARGS)
 {
-       double     *result,
-                          *d2;
+       LSEG       *lseg = PG_GETARG_LSEG_P(0);
+       LINE       *line = PG_GETARG_LINE_P(1);
+       float8          result,
+                               d2;
 
-       if (inter_sl(lseg, line))
+       if (has_interpt_sl(lseg, line))
        {
-               result = palloc(sizeof(double));
-               *result = 0.0;
-
+               result = 0.0;
        }
        else
        {
-               result = dist_pl(&lseg->p[0], line);
-               d2 = dist_pl(&lseg->p[1], line);
-               if (*d2 > *result)
-               {
-                       pfree(result);
+               result = dist_pl_internal(&lseg->p[0], line);
+               d2 = dist_pl_internal(&lseg->p[1], line);
+               /* XXX shouldn't we take the min not max? */
+               if (d2 > result)
                        result = d2;
-               }
-               else
-                       pfree(d2);
        }
 
-       return result;
+       PG_RETURN_FLOAT8(result);
 }
 
 
-double *
-dist_sb(LSEG *lseg, BOX *box)
+Datum
+dist_sb(PG_FUNCTION_ARGS)
 {
+       LSEG       *lseg = PG_GETARG_LSEG_P(0);
+       BOX                *box = PG_GETARG_BOX_P(1);
        Point      *tmp;
-       double     *result;
-
-       tmp = close_sb(lseg, box);
-       if (tmp == NULL)
-       {
-               result = palloc(sizeof(double));
-               *result = 0.0;
-       }
-       else
-       {
-               result = dist_pb(tmp, box);
-               pfree(tmp);
-       }
+       Datum           result;
+
+       tmp = DatumGetPointP(DirectFunctionCall2(close_sb,
+                                                                                        LsegPGetDatum(lseg),
+                                                                                        BoxPGetDatum(box)));
+       result = DirectFunctionCall2(dist_pb,
+                                                                PointPGetDatum(tmp),
+                                                                BoxPGetDatum(box));
+       pfree(tmp);
 
-       return result;
+       PG_RETURN_DATUM(result);
 }
 
 
-double *
-dist_lb(LINE *line, BOX *box)
+Datum
+dist_lb(PG_FUNCTION_ARGS)
 {
-       Point      *tmp;
-       double     *result;
+#ifdef NOT_USED
+       LINE       *line = PG_GETARG_LINE_P(0);
+       BOX                *box = PG_GETARG_BOX_P(1);
+#endif
 
-       tmp = close_lb(line, box);
-       if (tmp == NULL)
-       {
-               result = palloc(sizeof(double));
-               *result = 0.0;
-       }
-       else
-       {
-               result = dist_pb(tmp, box);
-               pfree(tmp);
-       }
+       /* think about this one for a while */
+       elog(ERROR, "dist_lb not implemented");
 
-       return result;
+       PG_RETURN_NULL();
 }
 
 
@@ -2173,7 +2262,7 @@ dist_cpoly(PG_FUNCTION_ARGS)
        seg.p[0].y = poly->p[0].y;
        seg.p[1].x = poly->p[poly->npts - 1].x;
        seg.p[1].y = poly->p[poly->npts - 1].y;
-       result = *dist_ps(&(circle->center), &seg);
+       result = dist_ps_internal(&circle->center, &seg);
 #ifdef GEODEBUG
        printf("dist_cpoly- segment 0/n distance is %f\n", result);
 #endif
@@ -2185,7 +2274,7 @@ dist_cpoly(PG_FUNCTION_ARGS)
                seg.p[0].y = poly->p[i].y;
                seg.p[1].x = poly->p[i + 1].x;
                seg.p[1].y = poly->p[i + 1].y;
-               d = *dist_ps(&(circle->center), &seg);
+               d = dist_ps_internal(&circle->center, &seg);
 #ifdef GEODEBUG
                printf("dist_cpoly- segment %d distance is %f\n", (i + 1), d);
 #endif
@@ -2208,31 +2297,31 @@ dist_cpoly(PG_FUNCTION_ARGS)
  *                               lines and boxes, since there are typically two.
  *-------------------------------------------------------------------*/
 
+/* Get intersection point of lseg and line; returns NULL if no intersection */
 static Point *
 interpt_sl(LSEG *lseg, LINE *line)
 {
-       LINE       *tmp;
+       LINE            tmp;
        Point      *p;
 
-       tmp = line_construct_pp(&lseg->p[0], &lseg->p[1]);
-       p = line_interpt(tmp, line);
+       line_construct_pts(&tmp, &lseg->p[0], &lseg->p[1]);
+       p = line_interpt_internal(&tmp, line);
 #ifdef GEODEBUG
        printf("interpt_sl- segment is (%.*g %.*g) (%.*g %.*g)\n",
                   digits8, lseg->p[0].x, digits8, lseg->p[0].y, digits8, lseg->p[1].x, digits8, lseg->p[1].y);
        printf("interpt_sl- segment becomes line A=%.*g B=%.*g C=%.*g\n",
-                  digits8, tmp->A, digits8, tmp->B, digits8, tmp->C);
+                  digits8, tmp.A, digits8, tmp.B, digits8, tmp.C);
 #endif
        if (PointerIsValid(p))
        {
 #ifdef GEODEBUG
                printf("interpt_sl- intersection point is (%.*g %.*g)\n", digits8, p->x, digits8, p->y);
 #endif
-               if (on_ps(p, lseg))
+               if (on_ps_internal(p, lseg))
                {
 #ifdef GEODEBUG
                        printf("interpt_sl- intersection point is on segment\n");
 #endif
-
                }
                else
                {
@@ -2241,10 +2330,23 @@ interpt_sl(LSEG *lseg, LINE *line)
                }
        }
 
-       pfree(tmp);
        return p;
 }
 
+/* variant: just indicate if intersection point exists */
+static bool
+has_interpt_sl(LSEG *lseg, LINE *line)
+{
+       Point      *tmp;
+
+       tmp = interpt_sl(lseg, line);
+       if (tmp)
+       {
+               pfree(tmp);
+               return true;
+       }
+       return false;
+}
 
 /*---------------------------------------------------------------------
  *             close_
@@ -2255,36 +2357,33 @@ interpt_sl(LSEG *lseg, LINE *line)
  *             The intersection point of a perpendicular of the line
  *             through the point.
  */
-Point *
-close_pl(Point *pt, LINE *line)
+Datum
+close_pl(PG_FUNCTION_ARGS)
 {
+       Point      *pt = PG_GETARG_POINT_P(0);
+       LINE       *line = PG_GETARG_LINE_P(1);
        Point      *result;
        LINE       *tmp;
        double          invm;
 
-       result = palloc(sizeof(Point));
+       result = (Point *) palloc(sizeof(Point));
+
 #ifdef NOT_USED
        if (FPeq(line->A, -1.0) && FPzero(line->B))
        {                                                       /* vertical */
        }
 #endif
-       if (line_vertical(line))
+       if (FPzero(line->B))            /* vertical? */
        {
                result->x = line->C;
                result->y = pt->y;
-               return result;
-
-#ifdef NOT_USED
-       }
-       else if (FPzero(line->m))
-       {                                                       /* horizontal */
-#endif
+               PG_RETURN_POINT_P(result);
        }
-       else if (line_horizontal(line))
+       if (FPzero(line->A))            /* horizontal? */
        {
                result->x = pt->x;
                result->y = line->C;
-               return result;
+               PG_RETURN_POINT_P(result);
        }
        /* drop a perpendicular and find the intersection point */
 #ifdef NOT_USED
@@ -2293,9 +2392,10 @@ close_pl(Point *pt, LINE *line)
        /* invert and flip the sign on the slope to get a perpendicular */
        invm = line->B / line->A;
        tmp = line_construct_pm(pt, invm);
-       result = line_interpt(tmp, line);
-       return result;
-}      /* close_pl() */
+       result = line_interpt_internal(tmp, line);
+       Assert(result != NULL);
+       PG_RETURN_POINT_P(result);
+}
 
 
 /* close_ps()
@@ -2308,10 +2408,12 @@ close_pl(Point *pt, LINE *line)
  *     evaluating to only zero or one to use as an array index.
  *             bug fixes by gthaker@atl.lmco.com; May 1, 1998
  */
-Point *
-close_ps(Point *pt, LSEG *lseg)
+Datum
+close_ps(PG_FUNCTION_ARGS)
 {
-       Point      *result;
+       Point      *pt = PG_GETARG_POINT_P(0);
+       LSEG       *lseg = PG_GETARG_LSEG_P(1);
+       Point      *result = NULL;
        LINE       *tmp;
        double          invm;
        int                     xh,
@@ -2319,16 +2421,16 @@ close_ps(Point *pt, LSEG *lseg)
 
 #ifdef GEODEBUG
        printf("close_sp:pt->x %f pt->y %f\nlseg(0).x %f lseg(0).y %f  lseg(1).x %f lseg(1).y %f\n",
-       pt->x, pt->y, lseg->p[0].x, lseg->p[0].y, lseg->p[1].x, lseg->p[1].y);
+                  pt->x, pt->y, lseg->p[0].x, lseg->p[0].y,
+                  lseg->p[1].x, lseg->p[1].y);
 #endif
 
-       result = NULL;
+       /* xh (or yh) is the index of upper x( or y) end point of lseg */
+       /* !xh (or !yh) is the index of lower x( or y) end point of lseg */
        xh = lseg->p[0].x < lseg->p[1].x;
        yh = lseg->p[0].y < lseg->p[1].y;
-       /* !xh (or !yh) is the index of lower x( or y) end point of lseg */
 
-       /* vertical segment? */
-       if (lseg_vertical(lseg))
+       if (FPeq(lseg->p[0].x, lseg->p[1].x))   /* vertical? */
        {
 #ifdef GEODEBUG
                printf("close_ps- segment is vertical\n");
@@ -2339,16 +2441,16 @@ close_ps(Point *pt, LSEG *lseg)
                else if (pt->y > lseg->p[yh].y)
                        result = point_copy(&lseg->p[yh]);      /* above the lseg */
                if (result != NULL)
-                       return result;
+                       PG_RETURN_POINT_P(result);
 
                /* point lines along (to left or right) of the vertical lseg. */
 
-               result = palloc(sizeof(*result));
+               result = (Point *) palloc(sizeof(Point));
                result->x = lseg->p[0].x;
                result->y = pt->y;
-               return result;
+               PG_RETURN_POINT_P(result);
        }
-       else if (lseg_horizontal(lseg))
+       else if (FPeq(lseg->p[0].y, lseg->p[1].y)) /* horizontal? */
        {
 #ifdef GEODEBUG
                printf("close_ps- segment is horizontal\n");
@@ -2359,13 +2461,13 @@ close_ps(Point *pt, LSEG *lseg)
                else if (pt->x > lseg->p[xh].x)
                        result = point_copy(&lseg->p[xh]);      /* right of the lseg */
                if (result != NULL)
-                       return result;
+                       PG_RETURN_POINT_P(result);
 
                /* point lines along (at top or below) the horiz. lseg. */
-               result = palloc(sizeof(*result));
+               result = (Point *) palloc(sizeof(Point));
                result->x = pt->x;
                result->y = lseg->p[0].y;
-               return result;
+               PG_RETURN_POINT_P(result);
        }
 
        /*
@@ -2373,8 +2475,6 @@ close_ps(Point *pt, LSEG *lseg)
         * one of the end points or someplace on the lseg.
         */
 
-       /* TODO: Ask if "tmp" should be freed to prevent memory leak */
-
        invm = -1.0 / point_sl(&(lseg->p[0]), &(lseg->p[1]));
        tmp = line_construct_pm(&lseg->p[!yh], invm);           /* lower edge of the
                                                                                                                 * "band" */
@@ -2382,8 +2482,11 @@ close_ps(Point *pt, LSEG *lseg)
        {                                                       /* we are below the lower edge */
                result = point_copy(&lseg->p[!yh]);             /* below the lseg, take
                                                                                                 * lower end pt */
-/*       fprintf(stderr,"below: tmp A %f  B %f   C %f    m %f\n",tmp->A,tmp->B,tmp->C, tmp->m); */
-               return result;
+#ifdef GEODEBUG
+               printf("close_ps below: tmp A %f  B %f   C %f    m %f\n",
+                          tmp->A,tmp->B,tmp->C, tmp->m);
+#endif
+               PG_RETURN_POINT_P(result);
        }
        tmp = line_construct_pm(&lseg->p[yh], invm);            /* upper edge of the
                                                                                                                 * "band" */
@@ -2391,8 +2494,11 @@ close_ps(Point *pt, LSEG *lseg)
        {                                                       /* we are below the lower edge */
                result = point_copy(&lseg->p[yh]);              /* above the lseg, take
                                                                                                 * higher end pt */
-/*       fprintf(stderr,"above: tmp A %f  B %f   C %f    m %f\n",tmp->A,tmp->B,tmp->C, tmp->m); */
-               return result;
+#ifdef GEODEBUG
+               printf("close_ps above: tmp A %f  B %f   C %f    m %f\n",
+                          tmp->A,tmp->B,tmp->C, tmp->m);
+#endif
+               PG_RETURN_POINT_P(result);
        }
 
        /*
@@ -2400,116 +2506,127 @@ close_ps(Point *pt, LSEG *lseg)
         * point will be somewhere on the lseg
         */
        tmp = line_construct_pm(pt, invm);
-/*     fprintf(stderr,"tmp A %f  B %f   C %f    m %f\n",tmp->A,tmp->B,tmp->C, tmp->m); */
+#ifdef GEODEBUG
+       printf("close_ps- tmp A %f  B %f   C %f    m %f\n",
+                  tmp->A,tmp->B,tmp->C, tmp->m);
+#endif
        result = interpt_sl(lseg, tmp);
-/*     fprintf(stderr,"result.x %f  result.y %f\n", result->x, result->y); */
-       return result;
-}      /* close_ps() */
+       Assert(result != NULL);
+#ifdef GEODEBUG
+       printf("close_ps- result.x %f  result.y %f\n", result->x, result->y);
+#endif
+       PG_RETURN_POINT_P(result);
+}
 
 
 /* close_lseg()
  * Closest point to l1 on l2.
  */
-Point *
-close_lseg(LSEG *l1, LSEG *l2)
+Datum
+close_lseg(PG_FUNCTION_ARGS)
 {
+       LSEG       *l1 = PG_GETARG_LSEG_P(0);
+       LSEG       *l2 = PG_GETARG_LSEG_P(1);
        Point      *result = NULL;
        Point           point;
        double          dist;
-       double     *d;
+       double          d;
 
-       d = dist_ps(&l1->p[0], l2);
-       dist = *d;
-       memcpy(&point, &l1->p[0], sizeof(point));
-       pfree(d);
+       d = dist_ps_internal(&l1->p[0], l2);
+       dist = d;
+       memcpy(&point, &l1->p[0], sizeof(Point));
 
-       if (*(d = dist_ps(&l1->p[1], l2)) < dist)
+       if ((d = dist_ps_internal(&l1->p[1], l2)) < dist)
        {
-               dist = *d;
-               memcpy(&point, &l1->p[1], sizeof(point));
+               dist = d;
+               memcpy(&point, &l1->p[1], sizeof(Point));
        }
-       pfree(d);
 
-       if (*(d = dist_ps(&l2->p[0], l1)) < dist)
+       if ((d = dist_ps_internal(&l2->p[0], l1)) < dist)
        {
-               result = close_ps(&l2->p[0], l1);
-               memcpy(&point, result, sizeof(point));
+               result = DatumGetPointP(DirectFunctionCall2(close_ps,
+                                                                                                       PointPGetDatum(&l2->p[0]),
+                                                                                                       LsegPGetDatum(l1)));
+               memcpy(&point, result, sizeof(Point));
                pfree(result);
-               result = close_ps(&point, l2);
+               result = DatumGetPointP(DirectFunctionCall2(close_ps,
+                                                                                                       PointPGetDatum(&point),
+                                                                                                       LsegPGetDatum(l2)));
        }
-       pfree(d);
 
-       if (*(d = dist_ps(&l2->p[1], l1)) < dist)
+       if ((d = dist_ps_internal(&l2->p[1], l1)) < dist)
        {
                if (result != NULL)
                        pfree(result);
 
-               result = close_ps(&l2->p[1], l1);
-               memcpy(&point, result, sizeof(point));
+               result = DatumGetPointP(DirectFunctionCall2(close_ps,
+                                                                                                       PointPGetDatum(&l2->p[1]),
+                                                                                                       LsegPGetDatum(l1)));
+               memcpy(&point, result, sizeof(Point));
                pfree(result);
-               result = close_ps(&point, l2);
+               result = DatumGetPointP(DirectFunctionCall2(close_ps,
+                                                                                                       PointPGetDatum(&point),
+                                                                                                       LsegPGetDatum(l2)));
        }
-       pfree(d);
 
        if (result == NULL)
-       {
-               result = palloc(sizeof(*result));
-               memcpy(result, &point, sizeof(*result));
-       }
+               result = point_copy(&point);
 
-       return result;
-}      /* close_lseg() */
+       PG_RETURN_POINT_P(result);
+}
 
 /* close_pb()
  * Closest point on or in box to specified point.
  */
-Point *
-close_pb(Point *pt, BOX *box)
+Datum
+close_pb(PG_FUNCTION_ARGS)
 {
+       Point      *pt = PG_GETARG_POINT_P(0);
+       BOX                *box = PG_GETARG_BOX_P(1);
        LSEG            lseg,
                                seg;
        Point           point;
        double          dist,
-                          *d;
+                               d;
 
-       if (on_pb(pt, box))
-               return pt;
+       if (DatumGetBool(DirectFunctionCall2(on_pb,
+                                                                                PointPGetDatum(pt),
+                                                                                BoxPGetDatum(box))))
+               PG_RETURN_POINT_P(pt);
 
        /* pairwise check lseg distances */
        point.x = box->low.x;
        point.y = box->high.y;
        statlseg_construct(&lseg, &box->low, &point);
-       dist = *(d = dist_ps(pt, &lseg));
-       pfree(d);
+       dist = d = dist_ps_internal(pt, &lseg);
 
        statlseg_construct(&seg, &box->high, &point);
-       if (*(d = dist_ps(pt, &seg)) < dist)
+       if ((d = dist_ps_internal(pt, &seg)) < dist)
        {
-               dist = *d;
+               dist = d;
                memcpy(&lseg, &seg, sizeof(lseg));
        }
-       pfree(d);
 
        point.x = box->high.x;
        point.y = box->low.y;
        statlseg_construct(&seg, &box->low, &point);
-       if (*(d = dist_ps(pt, &seg)) < dist)
+       if ((d = dist_ps_internal(pt, &seg)) < dist)
        {
-               dist = *d;
+               dist = d;
                memcpy(&lseg, &seg, sizeof(lseg));
        }
-       pfree(d);
 
        statlseg_construct(&seg, &box->high, &point);
-       if (*(d = dist_ps(pt, &seg)) < dist)
+       if ((d = dist_ps_internal(pt, &seg)) < dist)
        {
-               dist = *d;
+               dist = d;
                memcpy(&lseg, &seg, sizeof(lseg));
        }
-       pfree(d);
 
-       return close_ps(pt, &lseg);
-}      /* close_pb() */
+       PG_RETURN_DATUM(DirectFunctionCall2(close_ps,
+                                                                               PointPGetDatum(pt),
+                                                                               LsegPGetDatum(&lseg)));
+}
 
 /* close_sl()
  * Closest point on line to line segment.
@@ -2520,77 +2637,78 @@ close_pb(Point *pt, BOX *box)
  * Copied code to new routine close_ls() but haven't fixed this one yet.
  * - thomas 1998-01-31
  */
-Point *
-close_sl(LSEG *lseg, LINE *line)
+Datum
+close_sl(PG_FUNCTION_ARGS)
 {
+       LSEG       *lseg = PG_GETARG_LSEG_P(0);
+       LINE       *line = PG_GETARG_LINE_P(1);
        Point      *result;
-       double     *d1,
-                          *d2;
+       float8          d1,
+                               d2;
 
        result = interpt_sl(lseg, line);
        if (result)
-               return result;
+               PG_RETURN_POINT_P(result);
 
-       d1 = dist_pl(&lseg->p[0], line);
-       d2 = dist_pl(&lseg->p[1], line);
+       d1 = dist_pl_internal(&lseg->p[0], line);
+       d2 = dist_pl_internal(&lseg->p[1], line);
        if (d1 < d2)
                result = point_copy(&lseg->p[0]);
        else
                result = point_copy(&lseg->p[1]);
 
-       pfree(d1);
-       pfree(d2);
-       return result;
+       PG_RETURN_POINT_P(result);
 }
 
 /* close_ls()
  * Closest point on line segment to line.
  */
-Point *
-close_ls(LINE *line, LSEG *lseg)
+Datum
+close_ls(PG_FUNCTION_ARGS)
 {
+       LINE       *line = PG_GETARG_LINE_P(0);
+       LSEG       *lseg = PG_GETARG_LSEG_P(1);
        Point      *result;
-       double     *d1,
-                          *d2;
+       float8          d1,
+                               d2;
 
        result = interpt_sl(lseg, line);
        if (result)
-               return result;
+               PG_RETURN_POINT_P(result);
 
-       d1 = dist_pl(&lseg->p[0], line);
-       d2 = dist_pl(&lseg->p[1], line);
+       d1 = dist_pl_internal(&lseg->p[0], line);
+       d2 = dist_pl_internal(&lseg->p[1], line);
        if (d1 < d2)
                result = point_copy(&lseg->p[0]);
        else
                result = point_copy(&lseg->p[1]);
 
-       pfree(d1);
-       pfree(d2);
-       return result;
-}      /* close_ls() */
+       PG_RETURN_POINT_P(result);
+}
 
 /* close_sb()
  * Closest point on or in box to line segment.
  */
-Point *
-close_sb(LSEG *lseg, BOX *box)
+Datum
+close_sb(PG_FUNCTION_ARGS)
 {
-       Point      *result;
-       Point      *pt;
+       LSEG       *lseg = PG_GETARG_LSEG_P(0);
+       BOX                *box = PG_GETARG_BOX_P(1);
        Point           point;
-
        LSEG            bseg,
                                seg;
        double          dist,
                                d;
 
        /* segment intersects box? then just return closest point to center */
-       if (inter_sb(lseg, box))
+       if (DatumGetBool(DirectFunctionCall2(inter_sb,
+                                                                                LsegPGetDatum(lseg),
+                                                                                BoxPGetDatum(box))))
        {
-               pt = box_center(box);
-               result = close_ps(pt, lseg);
-               pfree(pt);
-               return result;
+               box_cn(&point, box);
+               PG_RETURN_DATUM(DirectFunctionCall2(close_ps,
+                                                                                       PointPGetDatum(&point),
+                                                                                       LsegPGetDatum(lseg)));
        }
 
        /* pairwise check lseg distances */
@@ -2623,16 +2741,23 @@ close_sb(LSEG *lseg, BOX *box)
        }
 
        /* OK, we now have the closest line segment on the box boundary */
-       return close_lseg(lseg, &bseg);
-}      /* close_sb() */
+       PG_RETURN_DATUM(DirectFunctionCall2(close_lseg,
+                                                                               LsegPGetDatum(lseg),
+                                                                               LsegPGetDatum(&bseg)));
+}
 
-Point *
-close_lb(LINE *line, BOX *box)
+Datum
+close_lb(PG_FUNCTION_ARGS)
 {
+#ifdef NOT_USED
+       LINE       *line = PG_GETARG_LINE_P(0);
+       BOX                *box = PG_GETARG_BOX_P(1);
+#endif
+
        /* think about this one for a while */
        elog(ERROR, "close_lb not implemented");
 
-       return NULL;
+       PG_RETURN_NULL();
 }
 
 /*---------------------------------------------------------------------
@@ -2643,13 +2768,13 @@ close_lb(LINE *line, BOX *box)
 /* on_pl -
  *             Does the point satisfy the equation?
  */
-bool
-on_pl(Point *pt, LINE *line)
+Datum
+on_pl(PG_FUNCTION_ARGS)
 {
-       if (!PointerIsValid(pt) || !PointerIsValid(line))
-               return FALSE;
+       Point      *pt = PG_GETARG_POINT_P(0);
+       LINE       *line = PG_GETARG_LINE_P(1);
 
-       return FPzero(line->A * pt->x + line->B * pt->y + line->C);
+       PG_RETURN_BOOL(FPzero(line->A * pt->x + line->B * pt->y + line->C));
 }
 
 
@@ -2657,24 +2782,30 @@ on_pl(Point *pt, LINE *line)
  *             Determine colinearity by detecting a triangle inequality.
  * This algorithm seems to behave nicely even with lsb residues - tgl 1997-07-09
  */
-bool
-on_ps(Point *pt, LSEG *lseg)
+Datum
+on_ps(PG_FUNCTION_ARGS)
 {
-       if (!PointerIsValid(pt) || !PointerIsValid(lseg))
-               return FALSE;
+       Point      *pt = PG_GETARG_POINT_P(0);
+       LSEG       *lseg = PG_GETARG_LSEG_P(1);
 
-       return (FPeq(point_dt(pt, &lseg->p[0]) + point_dt(pt, &lseg->p[1]),
-                                point_dt(&lseg->p[0], &lseg->p[1])));
+       PG_RETURN_BOOL(on_ps_internal(pt, lseg));
 }
 
-bool
-on_pb(Point *pt, BOX *box)
+static bool
+on_ps_internal(Point *pt, LSEG *lseg)
 {
-       if (!PointerIsValid(pt) || !PointerIsValid(box))
-               return FALSE;
+       return FPeq(point_dt(pt, &lseg->p[0]) + point_dt(pt, &lseg->p[1]),
+                               point_dt(&lseg->p[0], &lseg->p[1]));
+}
+
+Datum
+on_pb(PG_FUNCTION_ARGS)
+{
+       Point      *pt = PG_GETARG_POINT_P(0);
+       BOX                *box = PG_GETARG_BOX_P(1);
 
-       return (pt->x <= box->high.x && pt->x >= box->low.x &&
-                       pt->y <= box->high.y && pt->y >= box->low.y);
+       PG_RETURN_BOOL(pt->x <= box->high.x && pt->x >= box->low.x &&
+                                  pt->y <= box->high.y && pt->y >= box->low.y);
 }
 
 /* on_ppath -
@@ -2720,45 +2851,46 @@ on_ppath(PG_FUNCTION_ARGS)
        PG_RETURN_BOOL(point_inside(pt, path->npts, path->p) != 0);
 }
 
-
-bool
-on_sl(LSEG *lseg, LINE *line)
+Datum
+on_sl(PG_FUNCTION_ARGS)
 {
-       if (!PointerIsValid(lseg) || !PointerIsValid(line))
-               return FALSE;
+       LSEG       *lseg = PG_GETARG_LSEG_P(0);
+       LINE       *line = PG_GETARG_LINE_P(1);
 
-       return on_pl(&lseg->p[0], line) && on_pl(&lseg->p[1], line);
-}      /* on_sl() */
+       PG_RETURN_BOOL(DatumGetBool(DirectFunctionCall2(on_pl,
+                                                                                               PointPGetDatum(&lseg->p[0]),
+                                                                                               LinePGetDatum(line))) &&
+                                  DatumGetBool(DirectFunctionCall2(on_pl,
+                                                                                               PointPGetDatum(&lseg->p[1]),
+                                                                                               LinePGetDatum(line))));
+}
 
-bool
-on_sb(LSEG *lseg, BOX *box)
+Datum
+on_sb(PG_FUNCTION_ARGS)
 {
-       if (!PointerIsValid(lseg) || !PointerIsValid(box))
-               return FALSE;
+       LSEG       *lseg = PG_GETARG_LSEG_P(0);
+       BOX                *box = PG_GETARG_BOX_P(1);
 
-       return on_pb(&lseg->p[0], box) && on_pb(&lseg->p[1], box);
-}      /* on_sb() */
+       PG_RETURN_BOOL(DatumGetBool(DirectFunctionCall2(on_pb,
+                                                                                               PointPGetDatum(&lseg->p[0]),
+                                                                                               BoxPGetDatum(box))) &&
+                                  DatumGetBool(DirectFunctionCall2(on_pb,
+                                                                                               PointPGetDatum(&lseg->p[1]),
+                                                                                               BoxPGetDatum(box))));
+}
 
 /*---------------------------------------------------------------------
  *             inter_
  *                             Whether one object intersects another.
  *-------------------------------------------------------------------*/
 
-bool
-inter_sl(LSEG *lseg, LINE *line)
+Datum
+inter_sl(PG_FUNCTION_ARGS)
 {
-       Point      *tmp;
+       LSEG       *lseg = PG_GETARG_LSEG_P(0);
+       LINE       *line = PG_GETARG_LINE_P(1);
 
-       if (!PointerIsValid(lseg) || !PointerIsValid(line))
-               return FALSE;
-
-       tmp = interpt_sl(lseg, line);
-       if (tmp)
-       {
-               pfree(tmp);
-               return TRUE;
-       }
-       return FALSE;
+       PG_RETURN_BOOL(has_interpt_sl(lseg, line));
 }
 
 /* inter_sb()
@@ -2771,15 +2903,14 @@ inter_sl(LSEG *lseg, LINE *line)
  * Optimize for non-intersection by checking for box intersection first.
  * - thomas 1998-01-30
  */
-bool
-inter_sb(LSEG *lseg, BOX *box)
+Datum
+inter_sb(PG_FUNCTION_ARGS)
 {
+       LSEG       *lseg = PG_GETARG_LSEG_P(0);
+       BOX                *box = PG_GETARG_BOX_P(1);
        BOX                     lbox;
        LSEG            bseg;
-       Point           point;
-
-       if (!PointerIsValid(lseg) || !PointerIsValid(box))
-               return FALSE;
+       Point           point;
 
        lbox.low.x = Min(lseg->p[0].x, lseg->p[1].x);
        lbox.low.y = Min(lseg->p[0].y, lseg->p[1].y);
@@ -2787,77 +2918,81 @@ inter_sb(LSEG *lseg, BOX *box)
        lbox.high.y = Max(lseg->p[0].y, lseg->p[1].y);
 
        /* nothing close to overlap? then not going to intersect */
-       if (!box_overlap(&lbox, box))
-               return FALSE;
+       if (!box_ov(&lbox, box))
+               PG_RETURN_BOOL(false);
 
        /* an endpoint of segment is inside box? then clearly intersects */
-       if (on_pb(&lseg->p[0], box) || on_pb(&lseg->p[1], box))
-               return TRUE;
+       if (DatumGetBool(DirectFunctionCall2(on_pb,
+                                                                                PointPGetDatum(&lseg->p[0]),
+                                                                                BoxPGetDatum(box))) ||
+               DatumGetBool(DirectFunctionCall2(on_pb,
+                                                                                PointPGetDatum(&lseg->p[1]),
+                                                                                BoxPGetDatum(box))))
+               PG_RETURN_BOOL(true);
 
        /* pairwise check lseg intersections */
        point.x = box->low.x;
        point.y = box->high.y;
        statlseg_construct(&bseg, &box->low, &point);
-       if (lseg_intersect(&bseg, lseg))
-               return TRUE;
+       if (lseg_intersect_internal(&bseg, lseg))
+               PG_RETURN_BOOL(true);
 
        statlseg_construct(&bseg, &box->high, &point);
-       if (lseg_intersect(&bseg, lseg))
-               return TRUE;
+       if (lseg_intersect_internal(&bseg, lseg))
+               PG_RETURN_BOOL(true);
 
        point.x = box->high.x;
        point.y = box->low.y;
        statlseg_construct(&bseg, &box->low, &point);
-       if (lseg_intersect(&bseg, lseg))
-               return TRUE;
+       if (lseg_intersect_internal(&bseg, lseg))
+               PG_RETURN_BOOL(true);
 
        statlseg_construct(&bseg, &box->high, &point);
-       if (lseg_intersect(&bseg, lseg))
-               return TRUE;
+       if (lseg_intersect_internal(&bseg, lseg))
+               PG_RETURN_BOOL(true);
 
        /* if we dropped through, no two segs intersected */
-       return FALSE;
-}      /* inter_sb() */
+       PG_RETURN_BOOL(false);
+}
 
 /* inter_lb()
  * Do line and box intersect?
  */
-bool
-inter_lb(LINE *line, BOX *box)
+Datum
+inter_lb(PG_FUNCTION_ARGS)
 {
+       LINE       *line = PG_GETARG_LINE_P(0);
+       BOX                *box = PG_GETARG_BOX_P(1);
        LSEG            bseg;
        Point           p1,
                                p2;
 
-       if (!PointerIsValid(line) || !PointerIsValid(box))
-               return FALSE;
-
        /* pairwise check lseg intersections */
        p1.x = box->low.x;
        p1.y = box->low.y;
        p2.x = box->low.x;
        p2.y = box->high.y;
        statlseg_construct(&bseg, &p1, &p2);
-       if (inter_sl(&bseg, line))
-               return TRUE;
+       if (has_interpt_sl(&bseg, line))
+               PG_RETURN_BOOL(true);
        p1.x = box->high.x;
        p1.y = box->high.y;
        statlseg_construct(&bseg, &p1, &p2);
-       if (inter_sl(&bseg, line))
-               return TRUE;
+       if (has_interpt_sl(&bseg, line))
+               PG_RETURN_BOOL(true);
        p2.x = box->high.x;
        p2.y = box->low.y;
        statlseg_construct(&bseg, &p1, &p2);
-       if (inter_sl(&bseg, line))
-               return TRUE;
+       if (has_interpt_sl(&bseg, line))
+               PG_RETURN_BOOL(true);
        p1.x = box->low.x;
        p1.y = box->low.y;
        statlseg_construct(&bseg, &p1, &p2);
-       if (inter_sl(&bseg, line))
-               return TRUE;
+       if (has_interpt_sl(&bseg, line))
+               PG_RETURN_BOOL(true);
 
        /* if we dropped through, no intersection */
-       return FALSE;
+       PG_RETURN_BOOL(false);
 }
 
 /*------------------------------------------------------------------
@@ -3039,7 +3174,7 @@ poly_overlap(PG_FUNCTION_ARGS)
        POLYGON    *polya = PG_GETARG_POLYGON_P(0);
        POLYGON    *polyb = PG_GETARG_POLYGON_P(1);
 
-       PG_RETURN_BOOL(box_overlap(&(polya->boundbox), &(polyb->boundbox)));
+       PG_RETURN_BOOL(box_ov(&polya->boundbox, &polyb->boundbox));
 }
 
 
@@ -3056,7 +3191,9 @@ poly_contain(PG_FUNCTION_ARGS)
        /*
         * Quick check to see if bounding box is contained.
         */
-       if (box_contain(&(polya->boundbox), &(polyb->boundbox)))
+       if (DatumGetBool(DirectFunctionCall2(box_contain,
+                                                                                BoxPGetDatum(&polya->boundbox),
+                                                                                BoxPGetDatum(&polyb->boundbox))))
        {
                for (i = 0; i < polyb->npts; i++)
                {
@@ -3153,74 +3290,69 @@ poly_distance(PG_FUNCTION_ARGS)
  **
  ***********************************************************************/
 
-Point *
-point(float8 *x, float8 *y)
+Datum
+construct_point(PG_FUNCTION_ARGS)
 {
-       if (!(PointerIsValid(x) && PointerIsValid(y)))
-               return NULL;
-
-       return point_construct(*x, *y);
-}      /* point() */
+       float8          x = PG_GETARG_FLOAT8(0);
+       float8          y = PG_GETARG_FLOAT8(1);
 
+       PG_RETURN_POINT_P(point_construct(x, y));
+}
 
-Point *
-point_add(Point *p1, Point *p2)
+Datum
+point_add(PG_FUNCTION_ARGS)
 {
+       Point      *p1 = PG_GETARG_POINT_P(0);
+       Point      *p2 = PG_GETARG_POINT_P(1);
        Point      *result;
 
-       if (!(PointerIsValid(p1) && PointerIsValid(p2)))
-               return NULL;
-
-       result = palloc(sizeof(Point));
+       result = (Point *) palloc(sizeof(Point));
 
        result->x = (p1->x + p2->x);
        result->y = (p1->y + p2->y);
 
-       return result;
-}      /* point_add() */
+       PG_RETURN_POINT_P(result);
+}
 
-Point *
-point_sub(Point *p1, Point *p2)
+Datum
+point_sub(PG_FUNCTION_ARGS)
 {
+       Point      *p1 = PG_GETARG_POINT_P(0);
+       Point      *p2 = PG_GETARG_POINT_P(1);
        Point      *result;
 
-       if (!(PointerIsValid(p1) && PointerIsValid(p2)))
-               return NULL;
-
-       result = palloc(sizeof(Point));
+       result = (Point *) palloc(sizeof(Point));
 
        result->x = (p1->x - p2->x);
        result->y = (p1->y - p2->y);
 
-       return result;
-}      /* point_sub() */
+       PG_RETURN_POINT_P(result);
+}
 
-Point *
-point_mul(Point *p1, Point *p2)
+Datum
+point_mul(PG_FUNCTION_ARGS)
 {
+       Point      *p1 = PG_GETARG_POINT_P(0);
+       Point      *p2 = PG_GETARG_POINT_P(1);
        Point      *result;
 
-       if (!(PointerIsValid(p1) && PointerIsValid(p2)))
-               return NULL;
-
-       result = palloc(sizeof(Point));
+       result = (Point *) palloc(sizeof(Point));
 
        result->x = (p1->x * p2->x) - (p1->y * p2->y);
        result->y = (p1->x * p2->y) + (p1->y * p2->x);
 
-       return result;
-}      /* point_mul() */
+       PG_RETURN_POINT_P(result);
+}
 
-Point *
-point_div(Point *p1, Point *p2)
+Datum
+point_div(PG_FUNCTION_ARGS)
 {
+       Point      *p1 = PG_GETARG_POINT_P(0);
+       Point      *p2 = PG_GETARG_POINT_P(1);
        Point      *result;
        double          div;
 
-       if (!(PointerIsValid(p1) && PointerIsValid(p2)))
-               return NULL;
-
-       result = palloc(sizeof(Point));
+       result = (Point *) palloc(sizeof(Point));
 
        div = (p2->x * p2->x) + (p2->y * p2->y);
 
@@ -3230,8 +3362,8 @@ point_div(Point *p1, Point *p2)
        result->x = ((p1->x * p2->x) + (p1->y * p2->y)) / div;
        result->y = ((p2->x * p1->y) - (p2->y * p1->x)) / div;
 
-       return result;
-}      /* point_div() */
+       PG_RETURN_POINT_P(result);
+}
 
 
 /***********************************************************************
@@ -3240,86 +3372,86 @@ point_div(Point *p1, Point *p2)
  **
  ***********************************************************************/
 
-BOX *
-box(Point *p1, Point *p2)
+Datum
+points_box(PG_FUNCTION_ARGS)
 {
-       BOX                *result;
-
-       if (!(PointerIsValid(p1) && PointerIsValid(p2)))
-               return NULL;
+       Point      *p1 = PG_GETARG_POINT_P(0);
+       Point      *p2 = PG_GETARG_POINT_P(1);
 
-       result = box_construct(p1->x, p2->x, p1->y, p2->y);
-
-       return result;
-}      /* box() */
+       PG_RETURN_BOX_P(box_construct(p1->x, p2->x, p1->y, p2->y));
+}
 
-BOX *
-box_add(BOX *box, Point *p)
+Datum
+box_add(PG_FUNCTION_ARGS)
 {
-       BOX                *result;
-
-       if (!(PointerIsValid(box) && PointerIsValid(p)))
-               return NULL;
-
-       result = box_construct((box->high.x + p->x), (box->low.x + p->x),
-                                                  (box->high.y + p->y), (box->low.y + p->y));
+       BOX                *box = PG_GETARG_BOX_P(0);
+       Point      *p = PG_GETARG_POINT_P(1);
 
-       return result;
-}      /* box_add() */
+       PG_RETURN_BOX_P(box_construct((box->high.x + p->x),
+                                                                 (box->low.x + p->x),
+                                                                 (box->high.y + p->y),
+                                                                 (box->low.y + p->y)));
+}
 
-BOX *
-box_sub(BOX *box, Point *p)
+Datum
+box_sub(PG_FUNCTION_ARGS)
 {
-       BOX                *result;
-
-       if (!(PointerIsValid(box) && PointerIsValid(p)))
-               return NULL;
-
-       result = box_construct((box->high.x - p->x), (box->low.x - p->x),
-                                                  (box->high.y - p->y), (box->low.y - p->y));
+       BOX                *box = PG_GETARG_BOX_P(0);
+       Point      *p = PG_GETARG_POINT_P(1);
 
-       return result;
-}      /* box_sub() */
+       PG_RETURN_BOX_P(box_construct((box->high.x - p->x),
+                                                                 (box->low.x - p->x),
+                                                                 (box->high.y - p->y),
+                                                                 (box->low.y - p->y)));
+}
 
-BOX *
-box_mul(BOX *box, Point *p)
+Datum
+box_mul(PG_FUNCTION_ARGS)
 {
+       BOX                *box = PG_GETARG_BOX_P(0);
+       Point      *p = PG_GETARG_POINT_P(1);
        BOX                *result;
        Point      *high,
                           *low;
 
-       if (!(PointerIsValid(box) && PointerIsValid(p)))
-               return NULL;
-
-       high = point_mul(&box->high, p);
-       low = point_mul(&box->low, p);
+       high = DatumGetPointP(DirectFunctionCall2(point_mul,
+                                                                                         PointPGetDatum(&box->high),
+                                                                                         PointPGetDatum(p)));
+       low = DatumGetPointP(DirectFunctionCall2(point_mul,
+                                                                                        PointPGetDatum(&box->low),
+                                                                                        PointPGetDatum(p)));
 
        result = box_construct(high->x, low->x, high->y, low->y);
+
        pfree(high);
        pfree(low);
 
-       return result;
-}      /* box_mul() */
+       PG_RETURN_BOX_P(result);
+}
 
-BOX *
-box_div(BOX *box, Point *p)
+Datum
+box_div(PG_FUNCTION_ARGS)
 {
+       BOX                *box = PG_GETARG_BOX_P(0);
+       Point      *p = PG_GETARG_POINT_P(1);
        BOX                *result;
        Point      *high,
                           *low;
 
-       if (!(PointerIsValid(box) && PointerIsValid(p)))
-               return NULL;
-
-       high = point_div(&box->high, p);
-       low = point_div(&box->low, p);
+       high = DatumGetPointP(DirectFunctionCall2(point_div,
+                                                                                         PointPGetDatum(&box->high),
+                                                                                         PointPGetDatum(p)));
+       low = DatumGetPointP(DirectFunctionCall2(point_div,
+                                                                                        PointPGetDatum(&box->low),
+                                                                                        PointPGetDatum(p)));
 
        result = box_construct(high->x, low->x, high->y, low->y);
+
        pfree(high);
        pfree(low);
 
-       return result;
-}      /* box_div() */
+       PG_RETURN_BOX_P(result);
+}
 
 
 /***********************************************************************
@@ -3412,7 +3544,9 @@ path_mul_pt(PG_FUNCTION_ARGS)
 
        for (i = 0; i < path->npts; i++)
        {
-               p = point_mul(&path->p[i], point);
+               p = DatumGetPointP(DirectFunctionCall2(point_mul,
+                                                                                          PointPGetDatum(&path->p[i]),
+                                                                                          PointPGetDatum(point)));
                path->p[i].x = p->x;
                path->p[i].y = p->y;
                pfree(p);
@@ -3431,7 +3565,9 @@ path_div_pt(PG_FUNCTION_ARGS)
 
        for (i = 0; i < path->npts; i++)
        {
-               p = point_div(&path->p[i], point);
+               p = DatumGetPointP(DirectFunctionCall2(point_div,
+                                                                                          PointPGetDatum(&path->p[i]),
+                                                                                          PointPGetDatum(point)));
                path->p[i].x = p->x;
                path->p[i].y = p->y;
                pfree(p);
@@ -3502,20 +3638,16 @@ Datum
 poly_center(PG_FUNCTION_ARGS)
 {
        POLYGON    *poly = PG_GETARG_POLYGON_P(0);
-       Point      *result;
+       Datum           result;
        CIRCLE     *circle;
 
        circle = DatumGetCircleP(DirectFunctionCall1(poly_circle,
                                                                                                 PolygonPGetDatum(poly)));
-       if (PointerIsValid(circle))
-       {
-               result = circle_center(circle);
-               pfree(circle);
-       }
-       else
-               PG_RETURN_NULL();
+       result = DirectFunctionCall1(circle_center,
+                                                                CirclePGetDatum(circle));
+       pfree(circle);
 
-       PG_RETURN_POINT_P(result);
+       PG_RETURN_DATUM(result);
 }
 
 
@@ -3608,19 +3740,16 @@ poly_path(PG_FUNCTION_ARGS)
  *                             "((f8,f8)<f8>)"
  *                             also supports quick entry style "(f8,f8,f8)"
  */
-CIRCLE *
-circle_in(char *str)
+Datum
+circle_in(PG_FUNCTION_ARGS)
 {
+       char       *str = PG_GETARG_CSTRING(0);
        CIRCLE     *circle;
-
        char       *s,
                           *cp;
        int                     depth = 0;
 
-       if (!PointerIsValid(str))
-               elog(ERROR, " Bad (null) circle external representation");
-
-       circle = palloc(sizeof(CIRCLE));
+       circle = (CIRCLE *) palloc(sizeof(CIRCLE));
 
        s = str;
        while (isspace((int) *s))
@@ -3663,20 +3792,18 @@ circle_in(char *str)
        if (*s != '\0')
                elog(ERROR, "Bad circle external representation '%s'", str);
 
-       return circle;
-}      /* circle_in() */
+       PG_RETURN_CIRCLE_P(circle);
+}
 
 /*             circle_out              -               convert a circle to external form.
  */
-char *
-circle_out(CIRCLE *circle)
+Datum
+circle_out(PG_FUNCTION_ARGS)
 {
+       CIRCLE     *circle = PG_GETARG_CIRCLE_P(0);
        char       *result;
        char       *cp;
 
-       if (!PointerIsValid(circle))
-               return NULL;
-
        result = palloc(3 * (P_MAXLEN + 1) + 3);
 
        cp = result;
@@ -3695,8 +3822,8 @@ circle_out(CIRCLE *circle)
        *cp++ = RDELIM_C;
        *cp = '\0';
 
-       return result;
-}      /* circle_out() */
+       PG_RETURN_CSTRING(result);
+}
 
 
 /*----------------------------------------------------------
@@ -3706,135 +3833,186 @@ circle_out(CIRCLE *circle)
 
 /*             circles identical?
  */
-bool
-circle_same(CIRCLE *circle1, CIRCLE *circle2)
+Datum
+circle_same(PG_FUNCTION_ARGS)
 {
-       return (FPeq(circle1->radius, circle2->radius)
-                       && FPeq(circle1->center.x, circle2->center.x)
-                       && FPeq(circle1->center.y, circle2->center.y));
-}      /* circle_same() */
+       CIRCLE     *circle1 = PG_GETARG_CIRCLE_P(0);
+       CIRCLE     *circle2 = PG_GETARG_CIRCLE_P(1);
+
+       PG_RETURN_BOOL(FPeq(circle1->radius, circle2->radius) &&
+                                  FPeq(circle1->center.x, circle2->center.x) &&
+                                  FPeq(circle1->center.y, circle2->center.y));
+}
 
 /*             circle_overlap  -               does circle1 overlap circle2?
  */
-bool
-circle_overlap(CIRCLE *circle1, CIRCLE *circle2)
+Datum
+circle_overlap(PG_FUNCTION_ARGS)
 {
-       return FPle(point_dt(&circle1->center, &circle2->center), (circle1->radius + circle2->radius));
+       CIRCLE     *circle1 = PG_GETARG_CIRCLE_P(0);
+       CIRCLE     *circle2 = PG_GETARG_CIRCLE_P(1);
+
+       PG_RETURN_BOOL(FPle(point_dt(&circle1->center, &circle2->center),
+                                               circle1->radius + circle2->radius));
 }
 
 /*             circle_overleft -               is the right edge of circle1 to the left of
  *                                                             the right edge of circle2?
  */
-bool
-circle_overleft(CIRCLE *circle1, CIRCLE *circle2)
+Datum
+circle_overleft(PG_FUNCTION_ARGS)
 {
-       return FPle((circle1->center.x + circle1->radius), (circle2->center.x + circle2->radius));
+       CIRCLE     *circle1 = PG_GETARG_CIRCLE_P(0);
+       CIRCLE     *circle2 = PG_GETARG_CIRCLE_P(1);
+
+       PG_RETURN_BOOL(FPle((circle1->center.x + circle1->radius),
+                                               (circle2->center.x + circle2->radius)));
 }
 
 /*             circle_left             -               is circle1 strictly left of circle2?
  */
-bool
-circle_left(CIRCLE *circle1, CIRCLE *circle2)
+Datum
+circle_left(PG_FUNCTION_ARGS)
 {
-       return FPle((circle1->center.x + circle1->radius), (circle2->center.x - circle2->radius));
+       CIRCLE     *circle1 = PG_GETARG_CIRCLE_P(0);
+       CIRCLE     *circle2 = PG_GETARG_CIRCLE_P(1);
+
+       PG_RETURN_BOOL(FPle((circle1->center.x + circle1->radius),
+                                               (circle2->center.x - circle2->radius)));
 }
 
 /*             circle_right    -               is circle1 strictly right of circle2?
  */
-bool
-circle_right(CIRCLE *circle1, CIRCLE *circle2)
+Datum
+circle_right(PG_FUNCTION_ARGS)
 {
-       return FPge((circle1->center.x - circle1->radius), (circle2->center.x + circle2->radius));
+       CIRCLE     *circle1 = PG_GETARG_CIRCLE_P(0);
+       CIRCLE     *circle2 = PG_GETARG_CIRCLE_P(1);
+
+       PG_RETURN_BOOL(FPge((circle1->center.x - circle1->radius),
+                                               (circle2->center.x + circle2->radius)));
 }
 
 /*             circle_overright                -               is the left edge of circle1 to the right of
  *                                                             the left edge of circle2?
  */
-bool
-circle_overright(CIRCLE *circle1, CIRCLE *circle2)
+Datum
+circle_overright(PG_FUNCTION_ARGS)
 {
-       return FPge((circle1->center.x - circle1->radius), (circle2->center.x - circle2->radius));
+       CIRCLE     *circle1 = PG_GETARG_CIRCLE_P(0);
+       CIRCLE     *circle2 = PG_GETARG_CIRCLE_P(1);
+
+       PG_RETURN_BOOL(FPge((circle1->center.x - circle1->radius),
+                                               (circle2->center.x - circle2->radius)));
 }
 
 /*             circle_contained                -               is circle1 contained by circle2?
  */
-bool
-circle_contained(CIRCLE *circle1, CIRCLE *circle2)
+Datum
+circle_contained(PG_FUNCTION_ARGS)
 {
-       return FPle((point_dt(&circle1->center, &circle2->center) + circle1->radius), circle2->radius);
+       CIRCLE     *circle1 = PG_GETARG_CIRCLE_P(0);
+       CIRCLE     *circle2 = PG_GETARG_CIRCLE_P(1);
+
+       PG_RETURN_BOOL(FPle((point_dt(&circle1->center, &circle2->center) + circle1->radius), circle2->radius));
 }
 
 /*             circle_contain  -               does circle1 contain circle2?
  */
-bool
-circle_contain(CIRCLE *circle1, CIRCLE *circle2)
+Datum
+circle_contain(PG_FUNCTION_ARGS)
 {
-       return FPle((point_dt(&circle1->center, &circle2->center) + circle2->radius), circle1->radius);
+       CIRCLE     *circle1 = PG_GETARG_CIRCLE_P(0);
+       CIRCLE     *circle2 = PG_GETARG_CIRCLE_P(1);
+
+       PG_RETURN_BOOL(FPle((point_dt(&circle1->center, &circle2->center) + circle2->radius), circle1->radius));
 }
 
 
 /*             circle_positionop               -
  *                             is circle1 entirely {above,below} circle2?
  */
-bool
-circle_below(CIRCLE *circle1, CIRCLE *circle2)
+Datum
+circle_below(PG_FUNCTION_ARGS)
 {
-       return FPle((circle1->center.y + circle1->radius), (circle2->center.y - circle2->radius));
+       CIRCLE     *circle1 = PG_GETARG_CIRCLE_P(0);
+       CIRCLE     *circle2 = PG_GETARG_CIRCLE_P(1);
+
+       PG_RETURN_BOOL(FPle(circle1->center.y + circle1->radius,
+                                               circle2->center.y - circle2->radius));
 }
 
-bool
-circle_above(CIRCLE *circle1, CIRCLE *circle2)
+Datum
+circle_above(PG_FUNCTION_ARGS)
 {
-       return FPge((circle1->center.y - circle1->radius), (circle2->center.y + circle2->radius));
+       CIRCLE     *circle1 = PG_GETARG_CIRCLE_P(0);
+       CIRCLE     *circle2 = PG_GETARG_CIRCLE_P(1);
+
+       PG_RETURN_BOOL(FPge(circle1->center.y - circle1->radius,
+                                               circle2->center.y + circle2->radius));
 }
 
 
 /*             circle_relop    -               is area(circle1) relop area(circle2), within
  *                                                             our accuracy constraint?
  */
-bool
-circle_eq(CIRCLE *circle1, CIRCLE *circle2)
+Datum
+circle_eq(PG_FUNCTION_ARGS)
 {
-       return FPeq(circle_ar(circle1), circle_ar(circle2));
-}      /* circle_eq() */
+       CIRCLE     *circle1 = PG_GETARG_CIRCLE_P(0);
+       CIRCLE     *circle2 = PG_GETARG_CIRCLE_P(1);
 
-bool
-circle_ne(CIRCLE *circle1, CIRCLE *circle2)
+       PG_RETURN_BOOL(FPeq(circle_ar(circle1), circle_ar(circle2)));
+}
+
+Datum
+circle_ne(PG_FUNCTION_ARGS)
 {
-       return !circle_eq(circle1, circle2);
-}      /* circle_ne() */
+       CIRCLE     *circle1 = PG_GETARG_CIRCLE_P(0);
+       CIRCLE     *circle2 = PG_GETARG_CIRCLE_P(1);
+
+       PG_RETURN_BOOL(FPne(circle_ar(circle1), circle_ar(circle2)));
+}
 
-bool
-circle_lt(CIRCLE *circle1, CIRCLE *circle2)
+Datum
+circle_lt(PG_FUNCTION_ARGS)
 {
-       return FPlt(circle_ar(circle1), circle_ar(circle2));
-}      /* circle_lt() */
+       CIRCLE     *circle1 = PG_GETARG_CIRCLE_P(0);
+       CIRCLE     *circle2 = PG_GETARG_CIRCLE_P(1);
+
+       PG_RETURN_BOOL(FPlt(circle_ar(circle1), circle_ar(circle2)));
+}
 
-bool
-circle_gt(CIRCLE *circle1, CIRCLE *circle2)
+Datum
+circle_gt(PG_FUNCTION_ARGS)
 {
-       return FPgt(circle_ar(circle1), circle_ar(circle2));
-}      /* circle_gt() */
+       CIRCLE     *circle1 = PG_GETARG_CIRCLE_P(0);
+       CIRCLE     *circle2 = PG_GETARG_CIRCLE_P(1);
+
+       PG_RETURN_BOOL(FPgt(circle_ar(circle1), circle_ar(circle2)));
+}
 
-bool
-circle_le(CIRCLE *circle1, CIRCLE *circle2)
+Datum
+circle_le(PG_FUNCTION_ARGS)
 {
-       return FPle(circle_ar(circle1), circle_ar(circle2));
-}      /* circle_le() */
+       CIRCLE     *circle1 = PG_GETARG_CIRCLE_P(0);
+       CIRCLE     *circle2 = PG_GETARG_CIRCLE_P(1);
 
-bool
-circle_ge(CIRCLE *circle1, CIRCLE *circle2)
+       PG_RETURN_BOOL(FPle(circle_ar(circle1), circle_ar(circle2)));
+}
+
+Datum
+circle_ge(PG_FUNCTION_ARGS)
 {
-       return FPge(circle_ar(circle1), circle_ar(circle2));
-}      /* circle_ge() */
+       CIRCLE     *circle1 = PG_GETARG_CIRCLE_P(0);
+       CIRCLE     *circle2 = PG_GETARG_CIRCLE_P(1);
+
+       PG_RETURN_BOOL(FPge(circle_ar(circle1), circle_ar(circle2)));
+}
 
 
 /*----------------------------------------------------------
  *     "Arithmetic" operators on circles.
- *             circle_foo              returns foo as an object (pointer) that
- can be passed between languages.
- *             circle_xx               is an internal routine which returns the
- *                                             actual value.
  *---------------------------------------------------------*/
 
 static CIRCLE *
@@ -3845,207 +4023,197 @@ circle_copy(CIRCLE *circle)
        if (!PointerIsValid(circle))
                return NULL;
 
-       result = palloc(sizeof(CIRCLE));
-
-       memmove((char *) result, (char *) circle, sizeof(CIRCLE));
+       result = (CIRCLE *) palloc(sizeof(CIRCLE));
+       memcpy((char *) result, (char *) circle, sizeof(CIRCLE));
        return result;
-}      /* circle_copy() */
+}
 
 
 /* circle_add_pt()
  * Translation operator.
  */
-CIRCLE *
-circle_add_pt(CIRCLE *circle, Point *point)
+Datum
+circle_add_pt(PG_FUNCTION_ARGS)
 {
+       CIRCLE     *circle = PG_GETARG_CIRCLE_P(0);
+       Point      *point = PG_GETARG_POINT_P(1);
        CIRCLE     *result;
 
-       if (!PointerIsValid(circle) || !PointerIsValid(point))
-               return NULL;
-
        result = circle_copy(circle);
 
        result->center.x += point->x;
        result->center.y += point->y;
 
-       return result;
-}      /* circle_add_pt() */
+       PG_RETURN_CIRCLE_P(result);
+}
 
-CIRCLE *
-circle_sub_pt(CIRCLE *circle, Point *point)
+Datum
+circle_sub_pt(PG_FUNCTION_ARGS)
 {
+       CIRCLE     *circle = PG_GETARG_CIRCLE_P(0);
+       Point      *point = PG_GETARG_POINT_P(1);
        CIRCLE     *result;
 
-       if (!PointerIsValid(circle) || !PointerIsValid(point))
-               return NULL;
-
        result = circle_copy(circle);
 
        result->center.x -= point->x;
        result->center.y -= point->y;
 
-       return result;
-}      /* circle_sub_pt() */
+       PG_RETURN_CIRCLE_P(result);
+}
 
 
 /* circle_mul_pt()
  * Rotation and scaling operators.
  */
-CIRCLE *
-circle_mul_pt(CIRCLE *circle, Point *point)
+Datum
+circle_mul_pt(PG_FUNCTION_ARGS)
 {
+       CIRCLE     *circle = PG_GETARG_CIRCLE_P(0);
+       Point      *point = PG_GETARG_POINT_P(1);
        CIRCLE     *result;
        Point      *p;
 
-       if (!PointerIsValid(circle) || !PointerIsValid(point))
-               return NULL;
-
        result = circle_copy(circle);
 
-       p = point_mul(&circle->center, point);
+       p = DatumGetPointP(DirectFunctionCall2(point_mul,
+                                                                                  PointPGetDatum(&circle->center),
+                                                                                  PointPGetDatum(point)));
        result->center.x = p->x;
        result->center.y = p->y;
        pfree(p);
        result->radius *= HYPOT(point->x, point->y);
 
-       return result;
-}      /* circle_mul_pt() */
+       PG_RETURN_CIRCLE_P(result);
+}
 
-CIRCLE *
-circle_div_pt(CIRCLE *circle, Point *point)
+Datum
+circle_div_pt(PG_FUNCTION_ARGS)
 {
+       CIRCLE     *circle = PG_GETARG_CIRCLE_P(0);
+       Point      *point = PG_GETARG_POINT_P(1);
        CIRCLE     *result;
        Point      *p;
 
-       if (!PointerIsValid(circle) || !PointerIsValid(point))
-               return NULL;
-
        result = circle_copy(circle);
 
-       p = point_div(&circle->center, point);
+       p = DatumGetPointP(DirectFunctionCall2(point_div,
+                                                                                  PointPGetDatum(&circle->center),
+                                                                                  PointPGetDatum(point)));
        result->center.x = p->x;
        result->center.y = p->y;
        pfree(p);
        result->radius /= HYPOT(point->x, point->y);
 
-       return result;
-}      /* circle_div_pt() */
+       PG_RETURN_CIRCLE_P(result);
+}
 
 
 /*             circle_area             -               returns the area of the circle.
  */
-double *
-circle_area(CIRCLE *circle)
+Datum
+circle_area(PG_FUNCTION_ARGS)
 {
-       double     *result;
-
-       result = palloc(sizeof(double));
-       *result = circle_ar(circle);
+       CIRCLE     *circle = PG_GETARG_CIRCLE_P(0);
 
-       return result;
+       PG_RETURN_FLOAT8(circle_ar(circle));
 }
 
 
 /*             circle_diameter -               returns the diameter of the circle.
  */
-double *
-circle_diameter(CIRCLE *circle)
+Datum
+circle_diameter(PG_FUNCTION_ARGS)
 {
-       double     *result;
-
-       result = palloc(sizeof(double));
-       *result = (2 * circle->radius);
+       CIRCLE     *circle = PG_GETARG_CIRCLE_P(0);
 
-       return result;
+       PG_RETURN_FLOAT8(2 * circle->radius);
 }
 
 
 /*             circle_radius   -               returns the radius of the circle.
  */
-double *
-circle_radius(CIRCLE *circle)
+Datum
+circle_radius(PG_FUNCTION_ARGS)
 {
-       double     *result;
-
-       result = palloc(sizeof(double));
-       *result = circle->radius;
+       CIRCLE     *circle = PG_GETARG_CIRCLE_P(0);
 
-       return result;
+       PG_RETURN_FLOAT8(circle->radius);
 }
 
 
 /*             circle_distance -               returns the distance between
  *                                                               two circles.
  */
-double *
-circle_distance(CIRCLE *circle1, CIRCLE *circle2)
+Datum
+circle_distance(PG_FUNCTION_ARGS)
 {
-       double     *result;
-
-       result = palloc(sizeof(double));
-       *result = (point_dt(&circle1->center, &circle2->center)
-                          - (circle1->radius + circle2->radius));
-       if (*result < 0)
-               *result = 0;
+       CIRCLE     *circle1 = PG_GETARG_CIRCLE_P(0);
+       CIRCLE     *circle2 = PG_GETARG_CIRCLE_P(1);
+       float8          result;
 
-       return result;
-}      /* circle_distance() */
+       result = point_dt(&circle1->center, &circle2->center)
+               - (circle1->radius + circle2->radius);
+       if (result < 0)
+               result = 0;
+       PG_RETURN_FLOAT8(result);
+}
 
 
-bool
-circle_contain_pt(CIRCLE *circle, Point *point)
+Datum
+circle_contain_pt(PG_FUNCTION_ARGS)
 {
-       bool            within;
-       double     *d;
-
-       if (!PointerIsValid(circle) || !PointerIsValid(point))
-               return FALSE;
-
-       d = point_distance(&(circle->center), point);
-       within = (*d <= circle->radius);
-       pfree(d);
+       CIRCLE     *circle = PG_GETARG_CIRCLE_P(0);
+       Point      *point = PG_GETARG_POINT_P(1);
+       double          d;
 
-       return within;
-}      /* circle_contain_pt() */
+       d = point_dt(&circle->center, point);
+       PG_RETURN_BOOL(d <= circle->radius);
+}
 
 
-bool
-pt_contained_circle(Point *point, CIRCLE *circle)
+Datum
+pt_contained_circle(PG_FUNCTION_ARGS)
 {
-       return circle_contain_pt(circle, point);
-}      /* circle_contain_pt() */
+       Point      *point = PG_GETARG_POINT_P(0);
+       CIRCLE     *circle = PG_GETARG_CIRCLE_P(1);
+       double          d;
+
+       d = point_dt(&circle->center, point);
+       PG_RETURN_BOOL(d <= circle->radius);
+}
 
 
 /*             dist_pc -               returns the distance between
  *                                               a point and a circle.
  */
-double *
-dist_pc(Point *point, CIRCLE *circle)
+Datum
+dist_pc(PG_FUNCTION_ARGS)
 {
-       double     *result;
-
-       result = palloc(sizeof(double));
-
-       *result = (point_dt(point, &circle->center) - circle->radius);
-       if (*result < 0)
-               *result = 0;
+       Point      *point = PG_GETARG_POINT_P(0);
+       CIRCLE     *circle = PG_GETARG_CIRCLE_P(1);
+       float8          result;
 
-       return result;
-}      /* dist_pc() */
+       result = point_dt(point, &circle->center) - circle->radius;
+       if (result < 0)
+               result = 0;
+       PG_RETURN_FLOAT8(result);
+}
 
 
 /*             circle_center   -               returns the center point of the circle.
  */
-Point *
-circle_center(CIRCLE *circle)
+Datum
+circle_center(PG_FUNCTION_ARGS)
 {
+       CIRCLE     *circle = PG_GETARG_CIRCLE_P(0);
        Point      *result;
 
-       result = palloc(sizeof(Point));
+       result = (Point *) palloc(sizeof(Point));
        result->x = circle->center.x;
        result->y = circle->center.y;
 
-       return result;
+       PG_RETURN_POINT_P(result);
 }
 
 
@@ -4058,85 +4226,63 @@ circle_ar(CIRCLE *circle)
 }
 
 
-/*             circle_dt               -               returns the distance between the
- *                                               center points of two circlees.
- */
-#ifdef NOT_USED
-double
-circle_dt(CIRCLE *circle1, CIRCLE *circle2)
-{
-       double          result;
-
-       result = point_dt(&circle1->center, &circle2->center);
-
-       return result;
-}
-
-#endif
-
 /*----------------------------------------------------------
  *     Conversion operators.
  *---------------------------------------------------------*/
 
-CIRCLE *
-circle(Point *center, float8 *radius)
+Datum
+cr_circle(PG_FUNCTION_ARGS)
 {
+       Point      *center = PG_GETARG_POINT_P(0);
+       float8          radius = PG_GETARG_FLOAT8(1);
        CIRCLE     *result;
 
-       if (!(PointerIsValid(center) && PointerIsValid(radius)))
-               return NULL;
-
-       result = palloc(sizeof(CIRCLE));
+       result = (CIRCLE *) palloc(sizeof(CIRCLE));
 
        result->center.x = center->x;
        result->center.y = center->y;
-       result->radius = *radius;
+       result->radius = radius;
 
-       return result;
+       PG_RETURN_CIRCLE_P(result);
 }
 
-
-BOX *
-circle_box(CIRCLE *circle)
+Datum
+circle_box(PG_FUNCTION_ARGS)
 {
+       CIRCLE     *circle = PG_GETARG_CIRCLE_P(0);
        BOX                *box;
        double          delta;
 
-       if (!PointerIsValid(circle))
-               return NULL;
-
-       box = palloc(sizeof(BOX));
+       box = (BOX *) palloc(sizeof(BOX));
 
-       delta = circle->radius / sqrt(2.0e0);
+       delta = circle->radius / sqrt(2.0);
 
        box->high.x = circle->center.x + delta;
        box->low.x = circle->center.x - delta;
        box->high.y = circle->center.y + delta;
        box->low.y = circle->center.y - delta;
 
-       return box;
-}      /* circle_box() */
+       PG_RETURN_BOX_P(box);
+}
 
 /* box_circle()
  * Convert a box to a circle.
  */
-CIRCLE *
-box_circle(BOX *box)
+Datum
+box_circle(PG_FUNCTION_ARGS)
 {
+       BOX                *box = PG_GETARG_BOX_P(0);
        CIRCLE     *circle;
 
-       if (!PointerIsValid(box))
-               return NULL;
-
-       circle = palloc(sizeof(CIRCLE));
+       circle = (CIRCLE *) palloc(sizeof(CIRCLE));
 
        circle->center.x = (box->high.x + box->low.x) / 2;
        circle->center.y = (box->high.y + box->low.y) / 2;
 
        circle->radius = point_dt(&circle->center, &box->high);
 
-       return circle;
-}      /* box_circle() */
+       PG_RETURN_CIRCLE_P(circle);
+}
 
 
 Datum
@@ -4332,7 +4478,7 @@ lseg_crossing(double x, double y, double px, double py)
                        return HIT_IT;
                return FPgt((sgn * z), 0) ? 0 : 2 * sgn;
        }
-}      /* lseg_crossing() */
+}
 
 
 static bool
@@ -4392,4 +4538,4 @@ plist_same(int npts, Point *p1, Point *p2)
        }
 
        return FALSE;
-}      /* plist_same() */
+}
index 8e96586d57511355f9d8da9e70fe4ecc26e5edc8..a75e7880f72356395918ede20d015dbb8ddc2688 100644 (file)
@@ -7,7 +7,7 @@
  * Portions Copyright (c) 1996-2000, PostgreSQL, Inc
  * Portions Copyright (c) 1994, Regents of the University of California
  *
- * $Id: pg_proc.h,v 1.151 2000/07/29 18:45:57 tgl Exp $
+ * $Id: pg_proc.h,v 1.152 2000/07/30 20:43:44 tgl Exp $
  *
  * NOTES
  *       The script catalog/genbki.sh reads this file and generates .bki
@@ -243,54 +243,54 @@ DESCR("convert int2 to text");
 DATA(insert OID = 114 (  text                     PGUID 12 f t t t 1 f  25 "26" 100 0 0 100    oid_text - ));
 DESCR("convert oid to text");
 
-DATA(insert OID = 115 (  box_above                PGUID 11 f t t t 2 f  16 "603 603" 100 1 0 100  box_above - ));
+DATA(insert OID = 115 (  box_above                PGUID 12 f t t t 2 f  16 "603 603" 100 1 0 100  box_above - ));
 DESCR("is above");
-DATA(insert OID = 116 (  box_below                PGUID 11 f t t t 2 f  16 "603 603" 100 1 0 100  box_below - ));
+DATA(insert OID = 116 (  box_below                PGUID 12 f t t t 2 f  16 "603 603" 100 1 0 100  box_below - ));
 DESCR("is below");
 
-DATA(insert OID = 117 (  point_in                 PGUID 11 f t t t 1 f 600 "0" 100 0 0 100  point_in - ));
+DATA(insert OID = 117 (  point_in                 PGUID 12 f t t t 1 f 600 "0" 100 0 0 100  point_in - ));
 DESCR("(internal)");
-DATA(insert OID = 118 (  point_out                PGUID 11 f t t t 1 f 23  "0" 100 0 0 100  point_out - ));
+DATA(insert OID = 118 (  point_out                PGUID 12 f t t t 1 f 23  "600" 100 0 0 100  point_out - ));
 DESCR("(internal)");
-DATA(insert OID = 119 (  lseg_in                  PGUID 11 f t t t 1 f 601 "0" 100 0 0 100  lseg_in - ));
+DATA(insert OID = 119 (  lseg_in                  PGUID 12 f t t t 1 f 601 "0" 100 0 0 100  lseg_in - ));
 DESCR("(internal)");
-DATA(insert OID = 120 (  lseg_out                 PGUID 11 f t t t 1 f 23  "0" 100 0 0 100  lseg_out - ));
+DATA(insert OID = 120 (  lseg_out                 PGUID 12 f t t t 1 f 23  "0" 100 0 0 100  lseg_out - ));
 DESCR("(internal)");
 DATA(insert OID = 121 (  path_in                  PGUID 12 f t t t 1 f 602 "0" 100 0 0 100  path_in - ));
 DESCR("(internal)");
 DATA(insert OID = 122 (  path_out                 PGUID 12 f t t t 1 f 23  "0" 100 0 0 100  path_out - ));
 DESCR("(internal)");
-DATA(insert OID = 123 (  box_in                           PGUID 11 f t t t 1 f 603 "0" 100 0 0 100  box_in - ));
+DATA(insert OID = 123 (  box_in                           PGUID 12 f t t t 1 f 603 "0" 100 0 0 100  box_in - ));
 DESCR("(internal)");
-DATA(insert OID = 124 (  box_out                  PGUID 11 f t t t 1 f 23  "0" 100 0 0 100  box_out - ));
+DATA(insert OID = 124 (  box_out                  PGUID 12 f t t t 1 f 23  "0" 100 0 0 100  box_out - ));
 DESCR("(internal)");
-DATA(insert OID = 125 (  box_overlap      PGUID 11 f t t t 2 f 16 "603 603" 100 1 0 100        box_overlap - ));
+DATA(insert OID = 125 (  box_overlap      PGUID 12 f t t t 2 f 16 "603 603" 100 1 0 100        box_overlap - ));
 DESCR("overlaps");
-DATA(insert OID = 126 (  box_ge                           PGUID 11 f t t t 2 f 16 "603 603" 100 1 0 100        box_ge - ));
-DESCR("greater-than-or-equal");
-DATA(insert OID = 127 (  box_gt                           PGUID 11 f t t t 2 f 16 "603 603" 100 1 0 100        box_gt - ));
-DESCR("greater-than");
-DATA(insert OID = 128 (  box_eq                           PGUID 11 f t t t 2 f 16 "603 603" 100 1 0 100        box_eq - ));
-DESCR("equal");
-DATA(insert OID = 129 (  box_lt                           PGUID 11 f t t t 2 f 16 "603 603" 100 1 0 100        box_lt - ));
-DESCR("less-than");
-DATA(insert OID = 130 (  box_le                           PGUID 11 f t t t 2 f 16 "603 603" 100 1 0 100        box_le - ));
-DESCR("less-than-or-equal");
-DATA(insert OID = 131 (  point_above      PGUID 11 f t t t 2 f 16 "600 600" 100 0 0 100        point_above - ));
+DATA(insert OID = 126 (  box_ge                           PGUID 12 f t t t 2 f 16 "603 603" 100 1 0 100        box_ge - ));
+DESCR("greater-than-or-equal by area");
+DATA(insert OID = 127 (  box_gt                           PGUID 12 f t t t 2 f 16 "603 603" 100 1 0 100        box_gt - ));
+DESCR("greater-than by area");
+DATA(insert OID = 128 (  box_eq                           PGUID 12 f t t t 2 f 16 "603 603" 100 1 0 100        box_eq - ));
+DESCR("equal by area");
+DATA(insert OID = 129 (  box_lt                           PGUID 12 f t t t 2 f 16 "603 603" 100 1 0 100        box_lt - ));
+DESCR("less-than by area");
+DATA(insert OID = 130 (  box_le                           PGUID 12 f t t t 2 f 16 "603 603" 100 1 0 100        box_le - ));
+DESCR("less-than-or-equal by area");
+DATA(insert OID = 131 (  point_above      PGUID 12 f t t t 2 f 16 "600 600" 100 0 0 100        point_above - ));
 DESCR("is above");
-DATA(insert OID = 132 (  point_left               PGUID 11 f t t t 2 f 16 "600 600" 100 0 0 100        point_left - ));
-DESCR("is left of");
-DATA(insert OID = 133 (  point_right      PGUID 11 f t t t 2 f 16 "600 600" 100 0 0 100        point_right - ));
+DATA(insert OID = 132 (  point_left               PGUID 12 f t t t 2 f 16 "600 600" 100 0 0 100        point_left - ));
 DESCR("is left of");
-DATA(insert OID = 134 (  point_below      PGUID 11 f t t t 2 f 16 "600 600" 100 0 0 100        point_below - ));
+DATA(insert OID = 133 (  point_right      PGUID 12 f t t t 2 f 16 "600 600" 100 0 0 100        point_right - ));
+DESCR("is right of");
+DATA(insert OID = 134 (  point_below      PGUID 12 f t t t 2 f 16 "600 600" 100 0 0 100        point_below - ));
 DESCR("is below");
-DATA(insert OID = 135 (  point_eq                 PGUID 11 f t t t 2 f 16 "600 600" 100 0 0 100        point_eq - ));
+DATA(insert OID = 135 (  point_eq                 PGUID 12 f t t t 2 f 16 "600 600" 100 0 0 100        point_eq - ));
 DESCR("same as");
-DATA(insert OID = 136 (  on_pb                    PGUID 11 f t t t 2 f 16 "600 603" 100 0 0 100        on_pb - ));
+DATA(insert OID = 136 (  on_pb                    PGUID 12 f t t t 2 f 16 "600 603" 100 0 0 100        on_pb - ));
 DESCR("point is inside");
 DATA(insert OID = 137 (  on_ppath                 PGUID 12 f t t t 2 f 16 "600 602" 100 0 1 0  on_ppath - ));
 DESCR("contained in");
-DATA(insert OID = 138 (  box_center               PGUID 11 f t t t 1 f 600 "603" 100 1 0 100  box_center - ));
+DATA(insert OID = 138 (  box_center               PGUID 12 f t t t 1 f 600 "603" 100 1 0 100  box_center - ));
 DESCR("center of");
 DATA(insert OID = 139 (  areasel                  PGUID 12 f t f t 5 f 701 "26 26 21 0 23" 100 0 0 100  areasel - ));
 DESCR("restriction selectivity for area-comparison operators");
@@ -300,8 +300,6 @@ DATA(insert OID = 141 (  int4mul               PGUID 12 f t t t 2 f 23 "23 23" 100 0 0 100
 DESCR("multiply");
 DATA(insert OID = 142 (  int4fac                  PGUID 12 f t t t 1 f 23 "23" 100 0 0 100  int4fac - ));
 DESCR("factorial");
-DATA(insert OID = 143 (  pointdist                PGUID 11 f t t t 2 f 23 "600 600" 100 0 0 100        pointdist - ));
-DESCR("");
 DATA(insert OID = 144 (  int4ne                           PGUID 12 f t t t 2 f 16 "23 23" 100 0 0 100  int4ne - ));
 DESCR("not equal");
 DATA(insert OID = 145 (  int2ne                           PGUID 12 f t t t 2 f 16 "21 21" 100 0 0 100  int2ne - ));
@@ -386,27 +384,27 @@ DATA(insert OID = 184 (  oideq                       PGUID 12 f t t t 2 f 16 "26 26" 100 0 0 100
 DESCR("equal");
 DATA(insert OID = 185 (  oidne                    PGUID 12 f t t t 2 f 16 "26 26" 100 0 0 100  oidne - ));
 DESCR("not equal");
-DATA(insert OID = 186 (  box_same                 PGUID 11 f t t t 2 f 16 "603 603" 100 0 0 100        box_same - ));
+DATA(insert OID = 186 (  box_same                 PGUID 12 f t t t 2 f 16 "603 603" 100 0 0 100        box_same - ));
 DESCR("same as");
-DATA(insert OID = 187 (  box_contain      PGUID 11 f t t t 2 f 16 "603 603" 100 0 0 100        box_contain - ));
+DATA(insert OID = 187 (  box_contain      PGUID 12 f t t t 2 f 16 "603 603" 100 0 0 100        box_contain - ));
 DESCR("contains");
-DATA(insert OID = 188 (  box_left                 PGUID 11 f t t t 2 f 16 "603 603" 100 0 0 100        box_left - ));
+DATA(insert OID = 188 (  box_left                 PGUID 12 f t t t 2 f 16 "603 603" 100 0 0 100        box_left - ));
 DESCR("is left of");
-DATA(insert OID = 189 (  box_overleft     PGUID 11 f t t t 2 f 16 "603 603" 100 0 0 100        box_overleft - ));
+DATA(insert OID = 189 (  box_overleft     PGUID 12 f t t t 2 f 16 "603 603" 100 0 0 100        box_overleft - ));
 DESCR("overlaps, but does not extend to right of");
-DATA(insert OID = 190 (  box_overright    PGUID 11 f t t t 2 f 16 "603 603" 100 0 0 100        box_overright - ));
+DATA(insert OID = 190 (  box_overright    PGUID 12 f t t t 2 f 16 "603 603" 100 0 0 100        box_overright - ));
 DESCR("overlaps, but does not extend to left of");
-DATA(insert OID = 191 (  box_right                PGUID 11 f t t t 2 f 16 "603 603" 100 0 0 100        box_right - ));
-DESCR("is left of");
-DATA(insert OID = 192 (  box_contained    PGUID 11 f t t t 2 f 16 "603 603" 100 0 0 100        box_contained - ));
+DATA(insert OID = 191 (  box_right                PGUID 12 f t t t 2 f 16 "603 603" 100 0 0 100        box_right - ));
+DESCR("is right of");
+DATA(insert OID = 192 (  box_contained    PGUID 12 f t t t 2 f 16 "603 603" 100 0 0 100        box_contained - ));
 DESCR("contained in");
-DATA(insert OID = 193 (  rt_box_union     PGUID 11 f t t t 2 f 603 "603 603" 100 0 0 100  rt_box_union - ));
+DATA(insert OID = 193 (  rt_box_union     PGUID 12 f t t t 2 f 603 "603 603" 100 0 0 100  rt_box_union - ));
 DESCR("r-tree");
-DATA(insert OID = 194 (  rt_box_inter     PGUID 11 f t t t 2 f 603 "603 603" 100 0 0 100  rt_box_inter - ));
+DATA(insert OID = 194 (  rt_box_inter     PGUID 12 f t t t 2 f 603 "603 603" 100 0 0 100  rt_box_inter - ));
 DESCR("r-tree");
-DATA(insert OID = 195 (  rt_box_size      PGUID 11 f t t t 2 f 700 "603 700" 100 0 0 100  rt_box_size - ));
+DATA(insert OID = 195 (  rt_box_size      PGUID 12 f t t t 2 f 700 "603 700" 100 0 0 100  rt_box_size - ));
 DESCR("r-tree");
-DATA(insert OID = 196 (  rt_bigbox_size    PGUID 11 f t t t 2 f 700 "603 700" 100 0 0 100  rt_bigbox_size - ));
+DATA(insert OID = 196 (  rt_bigbox_size    PGUID 12 f t t t 2 f 700 "603 700" 100 0 0 100  rt_bigbox_size - ));
 DESCR("r-tree");
 DATA(insert OID = 197 (  rt_poly_union    PGUID 12 f t t t 2 f 604 "604 604" 100 0 0 100  rt_poly_union - ));
 DESCR("r-tree");
@@ -468,7 +466,7 @@ DESCR("larger of two");
 DATA(insert OID = 224 (  float8smaller    PGUID 11 f t t t 2 f 701 "701 701" 100 0 0 100  float8smaller - ));
 DESCR("smaller of two");
 
-DATA(insert OID = 225 (  lseg_center      PGUID 11 f t t t 1 f 600 "601" 100 0 0 100  lseg_center - ));
+DATA(insert OID = 225 (  lseg_center      PGUID 12 f t t t 1 f 600 "601" 100 0 0 100  lseg_center - ));
 DESCR("center of");
 DATA(insert OID = 226 (  path_center      PGUID 12 f t t t 1 f 600 "602" 100 0 0 100  path_center - ));
 DESCR("center of");
@@ -497,7 +495,7 @@ DATA(insert OID = 237 (  int2                          PGUID 12 f t t t 1 f  21 "701" 100 0 0 100  d
 DESCR("convert float8 to int2");
 DATA(insert OID = 238 (  int2                     PGUID 12 f t t t 1 f  21 "700" 100 0 0 100  ftoi2 - ));
 DESCR("convert float4 to int2");
-DATA(insert OID = 239 (  line_distance    PGUID 11 f t t t 2 f 701 "628 628" 100 0 0 100  line_distance - ));
+DATA(insert OID = 239 (  line_distance    PGUID 12 f t t t 2 f 701 "628 628" 100 0 0 100  line_distance - ));
 DESCR("distance between");
 
 DATA(insert OID = 240 (  nabstimein               PGUID 12 f t f t 1 f 702 "0" 100 0 0 100  nabstimein - ));
@@ -576,9 +574,9 @@ DESCR("");
 DATA(insert OID = 276 (  int2fac                  PGUID 12 f t t t 1 f 23 "21" 100 0 0 100  int2fac - ));
 DESCR("");
 
-DATA(insert OID = 277 (  inter_sl                 PGUID 11 f t t t 2 f 16 "601 628" 100 0 0 100        inter_sl - ));
+DATA(insert OID = 277 (  inter_sl                 PGUID 12 f t t t 2 f 16 "601 628" 100 0 0 100        inter_sl - ));
 DESCR("");
-DATA(insert OID = 278 (  inter_lb                 PGUID 11 f t t t 2 f 16 "628 603" 100 0 0 100        inter_lb - ));
+DATA(insert OID = 278 (  inter_lb                 PGUID 12 f t t t 2 f 16 "628 603" 100 0 0 100        inter_lb - ));
 DESCR("");
 
 DATA(insert OID = 279 (  float48mul               PGUID 11 f t t t 2 f 701 "700 701" 100 0 0 100  float48mul - ));
@@ -753,31 +751,31 @@ DESCR("btree less-equal-greater");
 DATA(insert OID = 360 (  bttextcmp                PGUID 12 f t t t 2 f 23 "25 25" 100 0 0 100  bttextcmp - ));
 DESCR("btree less-equal-greater");
 
-DATA(insert OID = 361 (  lseg_distance    PGUID 11 f t t t 2 f 701 "601 601" 100 0 0 100  lseg_distance - ));
+DATA(insert OID = 361 (  lseg_distance    PGUID 12 f t t t 2 f 701 "601 601" 100 0 0 100  lseg_distance - ));
 DESCR("distance between");
-DATA(insert OID = 362 (  lseg_interpt     PGUID 11 f t t t 2 f 600 "601 601" 100 0 0 100  lseg_interpt - ));
+DATA(insert OID = 362 (  lseg_interpt     PGUID 12 f t t t 2 f 600 "601 601" 100 0 0 100  lseg_interpt - ));
 DESCR("");
-DATA(insert OID = 363 (  dist_ps                  PGUID 11 f t t t 2 f 701 "600 601" 100 0 0 100  dist_ps - ));
+DATA(insert OID = 363 (  dist_ps                  PGUID 12 f t t t 2 f 701 "600 601" 100 0 0 100  dist_ps - ));
 DESCR("distance between");
-DATA(insert OID = 364 (  dist_pb                  PGUID 11 f t t t 2 f 701 "600 603" 100 0 0 100  dist_pb - ));
+DATA(insert OID = 364 (  dist_pb                  PGUID 12 f t t t 2 f 701 "600 603" 100 0 0 100  dist_pb - ));
 DESCR("distance between point and box");
-DATA(insert OID = 365 (  dist_sb                  PGUID 11 f t t t 2 f 701 "601 603" 100 0 0 100  dist_sb - ));
+DATA(insert OID = 365 (  dist_sb                  PGUID 12 f t t t 2 f 701 "601 603" 100 0 0 100  dist_sb - ));
 DESCR("distance between segment and box");
-DATA(insert OID = 366 (  close_ps                 PGUID 11 f t t t 2 f 600 "600 601" 100 0 0 100  close_ps - ));
+DATA(insert OID = 366 (  close_ps                 PGUID 12 f t t t 2 f 600 "600 601" 100 0 0 100  close_ps - ));
 DESCR("closest point on line segment");
-DATA(insert OID = 367 (  close_pb                 PGUID 11 f t t t 2 f 600 "600 603" 100 0 0 100  close_pb - ));
+DATA(insert OID = 367 (  close_pb                 PGUID 12 f t t t 2 f 600 "600 603" 100 0 0 100  close_pb - ));
 DESCR("closest point on box");
-DATA(insert OID = 368 (  close_sb                 PGUID 11 f t t t 2 f 600 "601 603" 100 0 0 100  close_sb - ));
+DATA(insert OID = 368 (  close_sb                 PGUID 12 f t t t 2 f 600 "601 603" 100 0 0 100  close_sb - ));
 DESCR("closest point to line segment on box");
-DATA(insert OID = 369 (  on_ps                    PGUID 11 f t t t 2 f 16 "600 601" 100 0 0 100        on_ps - ));
+DATA(insert OID = 369 (  on_ps                    PGUID 12 f t t t 2 f 16 "600 601" 100 0 0 100        on_ps - ));
 DESCR("point contained in segment");
 DATA(insert OID = 370 (  path_distance    PGUID 12 f t t t 2 f 701 "602 602" 100 0 1 0  path_distance - ));
 DESCR("distance between paths");
 DATA(insert OID = 371 (  dist_ppath               PGUID 12 f t t t 2 f 701 "600 602" 100 0 1 0  dist_ppath - ));
 DESCR("distance between point and path");
-DATA(insert OID = 372 (  on_sb                    PGUID 11 f t t t 2 f 16 "601 603" 100 0 0 100        on_sb - ));
+DATA(insert OID = 372 (  on_sb                    PGUID 12 f t t t 2 f 16 "601 603" 100 0 0 100        on_sb - ));
 DESCR("contained in");
-DATA(insert OID = 373 (  inter_sb                 PGUID 11 f t t t 2 f 16 "601 603" 100 0 0 100        inter_sb - ));
+DATA(insert OID = 373 (  inter_sb                 PGUID 12 f t t t 2 f 16 "601 603" 100 0 0 100        inter_sb - ));
 DESCR("intersects?");
 
 /* OIDS 400 - 499 */
@@ -954,11 +952,11 @@ DESCR("");
 DATA(insert OID = 724 (  set_bit                  PGUID 12 f t t t 3 f 17 "17 23 23" 100 0 0 100  byteaSetBit - ));
 DESCR("");
 
-DATA(insert OID = 725 (  dist_pl                  PGUID 11 f t t t 2 f 701 "600 628" 100 0 0 100  dist_pl - ));
+DATA(insert OID = 725 (  dist_pl                  PGUID 12 f t t t 2 f 701 "600 628" 100 0 0 100  dist_pl - ));
 DESCR("distance between point and line");
-DATA(insert OID = 726 (  dist_lb                  PGUID 11 f t t t 2 f 701 "628 603" 100 0 0 100  dist_lb - ));
+DATA(insert OID = 726 (  dist_lb                  PGUID 12 f t t t 2 f 701 "628 603" 100 0 0 100  dist_lb - ));
 DESCR("distance between line and box");
-DATA(insert OID = 727 (  dist_sl                  PGUID 11 f t t t 2 f 701 "601 628" 100 0 0 100  dist_sl - ));
+DATA(insert OID = 727 (  dist_sl                  PGUID 12 f t t t 2 f 701 "601 628" 100 0 0 100  dist_sl - ));
 DESCR("distance between lseg and line");
 DATA(insert OID = 728 (  dist_cpoly               PGUID 12 f t t t 2 f 701 "718 604" 100 0 0 100  dist_cpoly - ));
 DESCR("distance between");
@@ -1190,15 +1188,15 @@ DESCR("large object create");
 DATA(insert OID = 958 (  lo_tell                  PGUID 12 f t f t 1 f 23 "23" 100 0 0 100  lo_tell - ));
 DESCR("large object position");
 
-DATA(insert OID = 959 (  on_pl                    PGUID 11 f t t t 2 f  16 "600 628" 100 0 10 100  on_pl - ));
+DATA(insert OID = 959 (  on_pl                    PGUID 12 f t t t 2 f  16 "600 628" 100 0 10 100  on_pl - ));
 DESCR("point on line?");
-DATA(insert OID = 960 (  on_sl                    PGUID 11 f t t t 2 f  16 "601 628" 100 0 10 100  on_sl - ));
+DATA(insert OID = 960 (  on_sl                    PGUID 12 f t t t 2 f  16 "601 628" 100 0 10 100  on_sl - ));
 DESCR("lseg on line?");
-DATA(insert OID = 961 (  close_pl                 PGUID 11 f t t t 2 f 600 "600 628" 100 0 10 100  close_pl - ));
+DATA(insert OID = 961 (  close_pl                 PGUID 12 f t t t 2 f 600 "600 628" 100 0 10 100  close_pl - ));
 DESCR("closest point on line");
-DATA(insert OID = 962 (  close_sl                 PGUID 11 f t t t 2 f 600 "601 628" 100 0 10 100  close_sl - ));
+DATA(insert OID = 962 (  close_sl                 PGUID 12 f t t t 2 f 600 "601 628" 100 0 10 100  close_sl - ));
 DESCR("closest point to line segment on line");
-DATA(insert OID = 963 (  close_lb                 PGUID 11 f t t t 2 f 600 "628 603" 100 0 10 100  close_lb - ));
+DATA(insert OID = 963 (  close_lb                 PGUID 12 f t t t 2 f 600 "628 603" 100 0 10 100  close_lb - ));
 DESCR("closest point to line on box");
 
 DATA(insert OID = 964 (  lo_unlink                PGUID 12 f t f t 1 f  23 "26" 100 0 0 100    lo_unlink - ));
@@ -1208,17 +1206,17 @@ DESCR("get oid for regproc");
 
 DATA(insert OID = 973 (  path_inter               PGUID 12 f t t t 2 f  16 "602 602" 100 0 10 100  path_inter - ));
 DESCR("paths intersect?");
-DATA(insert OID = 975 (  area                     PGUID 11 f t t t 1 f 701 "603" 100 0 0 100  box_area - ));
+DATA(insert OID = 975 (  area                     PGUID 12 f t t t 1 f 701 "603" 100 0 0 100  box_area - ));
 DESCR("box area");
-DATA(insert OID = 976 (  width                    PGUID 11 f t t t 1 f 701 "603" 100 0 0 100  box_width - ));
+DATA(insert OID = 976 (  width                    PGUID 12 f t t t 1 f 701 "603" 100 0 0 100  box_width - ));
 DESCR("box width");
-DATA(insert OID = 977 (  height                           PGUID 11 f t t t 1 f 701 "603" 100 0 0 100  box_height - ));
+DATA(insert OID = 977 (  height                           PGUID 12 f t t t 1 f 701 "603" 100 0 0 100  box_height - ));
 DESCR("box height");
-DATA(insert OID = 978 (  box_distance     PGUID 11 f t t t 2 f 701 "603 603" 100 0 0 100  box_distance - ));
+DATA(insert OID = 978 (  box_distance     PGUID 12 f t t t 2 f 701 "603 603" 100 0 0 100  box_distance - ));
 DESCR("distance between boxes");
-DATA(insert OID = 980 (  box_intersect    PGUID 11 f t t t 2 f 603 "603 603" 100 0 0 100  box_intersect - ));
+DATA(insert OID = 980 (  box_intersect    PGUID 12 f t t t 2 f 603 "603 603" 100 0 0 100  box_intersect - ));
 DESCR("box intersection (another box)");
-DATA(insert OID = 981 (  diagonal                 PGUID 11 f t t t 1 f 601 "603" 100 0 0 100  box_diagonal - ));
+DATA(insert OID = 981 (  diagonal                 PGUID 12 f t t t 1 f 601 "603" 100 0 0 100  box_diagonal - ));
 DESCR("box diagonal");
 DATA(insert OID = 982 (  path_n_lt                PGUID 12 f t t t 2 f 16 "602 602" 100 0 0 100        path_n_lt - ));
 DESCR("less-than");
@@ -1232,29 +1230,29 @@ DATA(insert OID = 986 (  path_n_ge                 PGUID 12 f t t t 2 f 16 "602 602" 100 0 0
 DESCR("greater-than-or-equal");
 DATA(insert OID = 987 (  path_length      PGUID 12 f t t t 1 f 701 "602" 100 0 1 0  path_length - ));
 DESCR("sum of path segments");
-DATA(insert OID = 988 (  point_ne                 PGUID 11 f t t t 2 f 16 "600 600" 100 0 0 100        point_ne - ));
+DATA(insert OID = 988 (  point_ne                 PGUID 12 f t t t 2 f 16 "600 600" 100 0 0 100        point_ne - ));
 DESCR("not equal");
-DATA(insert OID = 989 (  point_vert               PGUID 11 f t t t 2 f 16 "600 600" 100 0 0 100        point_vert - ));
-DESCR("vertical?");
-DATA(insert OID = 990 (  point_horiz      PGUID 11 f t t t 2 f 16 "600 600" 100 0 0 100        point_horiz - ));
-DESCR("horizontal?");
-DATA(insert OID = 991 (  point_distance    PGUID 11 f t t t 2 f 701 "600 600" 100 0 0 100  point_distance - ));
+DATA(insert OID = 989 (  point_vert               PGUID 12 f t t t 2 f 16 "600 600" 100 0 0 100        point_vert - ));
+DESCR("vertically aligned?");
+DATA(insert OID = 990 (  point_horiz      PGUID 12 f t t t 2 f 16 "600 600" 100 0 0 100        point_horiz - ));
+DESCR("horizontally aligned?");
+DATA(insert OID = 991 (  point_distance    PGUID 12 f t t t 2 f 701 "600 600" 100 0 0 100  point_distance - ));
 DESCR("distance between");
-DATA(insert OID = 992 (  slope                    PGUID 11 f t t t 2 f 701 "600 600" 100 0 0 100  point_slope - ));
+DATA(insert OID = 992 (  slope                    PGUID 12 f t t t 2 f 701 "600 600" 100 0 0 100  point_slope - ));
 DESCR("slope between points");
-DATA(insert OID = 993 (  lseg                     PGUID 11 f t t t 2 f 601 "600 600" 100 0 0 100  lseg_construct - ));
+DATA(insert OID = 993 (  lseg                     PGUID 12 f t t t 2 f 601 "600 600" 100 0 0 100  lseg_construct - ));
 DESCR("convert points to line segment");
-DATA(insert OID = 994 (  lseg_intersect    PGUID 11 f t t t 2 f 16 "601 601" 100 0 0 100       lseg_intersect - ));
-DESCR("intersects?");
-DATA(insert OID = 995 (  lseg_parallel    PGUID 11 f t t t 2 f 16 "601 601" 100 0 0 100        lseg_parallel - ));
+DATA(insert OID = 994 (  lseg_intersect    PGUID 12 f t t t 2 f 16 "601 601" 100 0 0 100       lseg_intersect - ));
+DESCR("intersect?");
+DATA(insert OID = 995 (  lseg_parallel    PGUID 12 f t t t 2 f 16 "601 601" 100 0 0 100        lseg_parallel - ));
 DESCR("parallel?");
-DATA(insert OID = 996 (  lseg_perp                PGUID 11 f t t t 2 f 16 "601 601" 100 0 0 100        lseg_perp - ));
+DATA(insert OID = 996 (  lseg_perp                PGUID 12 f t t t 2 f 16 "601 601" 100 0 0 100        lseg_perp - ));
 DESCR("perpendicular?");
-DATA(insert OID = 997 (  lseg_vertical    PGUID 11 f t t t 1 f 16 "601" 100 0 0 100    lseg_vertical - ));
+DATA(insert OID = 997 (  lseg_vertical    PGUID 12 f t t t 1 f 16 "601" 100 0 0 100    lseg_vertical - ));
 DESCR("vertical?");
-DATA(insert OID = 998 (  lseg_horizontal   PGUID 11 f t t t 1 f 16 "601" 100 0 0 100   lseg_horizontal - ));
+DATA(insert OID = 998 (  lseg_horizontal   PGUID 12 f t t t 1 f 16 "601" 100 0 0 100   lseg_horizontal - ));
 DESCR("horizontal?");
-DATA(insert OID = 999 (  lseg_eq                  PGUID 11 f t t t 2 f 16 "601 601" 100 0 0 100        lseg_eq - ));
+DATA(insert OID = 999 (  lseg_eq                  PGUID 12 f t t t 2 f 16 "601 601" 100 0 0 100        lseg_eq - ));
 DESCR("equal");
 
 /* OIDS 1000 - 1999 */
@@ -1365,13 +1363,13 @@ DESCR("(internal)");
 DATA(insert OID = 1145 (  time_eq                 PGUID 12 f t t t 2 f 16 "1083 1083" 100 0 0 100  time_eq - ));
 DESCR("equal");
 
-DATA(insert OID = 1146 (  circle_add_pt    PGUID 11 f t t t 2 f 718 "718 600" 100 0 0 100  circle_add_pt - ));
+DATA(insert OID = 1146 (  circle_add_pt    PGUID 12 f t t t 2 f 718 "718 600" 100 0 0 100  circle_add_pt - ));
 DESCR("addition");
-DATA(insert OID = 1147 (  circle_sub_pt    PGUID 11 f t t t 2 f 718 "718 600" 100 0 0 100  circle_sub_pt - ));
+DATA(insert OID = 1147 (  circle_sub_pt    PGUID 12 f t t t 2 f 718 "718 600" 100 0 0 100  circle_sub_pt - ));
 DESCR("subtract");
-DATA(insert OID = 1148 (  circle_mul_pt    PGUID 11 f t t t 2 f 718 "718 600" 100 0 0 100  circle_mul_pt - ));
+DATA(insert OID = 1148 (  circle_mul_pt    PGUID 12 f t t t 2 f 718 "718 600" 100 0 0 100  circle_mul_pt - ));
 DESCR("multiply");
-DATA(insert OID = 1149 (  circle_div_pt    PGUID 11 f t t t 2 f 718 "718 600" 100 0 0 100  circle_div_pt - ));
+DATA(insert OID = 1149 (  circle_div_pt    PGUID 12 f t t t 2 f 718 "718 600" 100 0 0 100  circle_div_pt - ));
 DESCR("divide");
 
 DATA(insert OID = 1150 (  timestamp_in    PGUID 12 f t f t 1 f 1184 "0" 100 0 0 100    timestamp_in - ));
@@ -1730,27 +1728,27 @@ DESCR("convert (no-op)");
 DATA(insert OID = 1405 (  int4            PGUID 14 f t t t 1 f 23       "23" 100 0 0 100  "select $1" - ));
 DESCR("convert (no-op)");
 
-DATA(insert OID = 1406 (  isvertical           PGUID 11 f t t t 2 f    16 "600 600" 100 0 0 100        point_vert - ));
-DESCR("vertical?");
-DATA(insert OID = 1407 (  ishorizontal         PGUID 11 f t t t 2 f    16 "600 600" 100 0 0 100        point_horiz - ));
-DESCR("horizontal?");
-DATA(insert OID = 1408 (  isparallel           PGUID 11 f t t t 2 f    16 "601 601" 100 0 0 100        lseg_parallel - ));
+DATA(insert OID = 1406 (  isvertical           PGUID 12 f t t t 2 f    16 "600 600" 100 0 0 100        point_vert - ));
+DESCR("vertically aligned?");
+DATA(insert OID = 1407 (  ishorizontal         PGUID 12 f t t t 2 f    16 "600 600" 100 0 0 100        point_horiz - ));
+DESCR("horizontally aligned?");
+DATA(insert OID = 1408 (  isparallel           PGUID 12 f t t t 2 f    16 "601 601" 100 0 0 100        lseg_parallel - ));
 DESCR("parallel?");
-DATA(insert OID = 1409 (  isperp                       PGUID 11 f t t t 2 f    16 "601 601" 100 0 0 100        lseg_perp - ));
+DATA(insert OID = 1409 (  isperp                       PGUID 12 f t t t 2 f    16 "601 601" 100 0 0 100        lseg_perp - ));
 DESCR("perpendicular?");
-DATA(insert OID = 1410 (  isvertical           PGUID 11 f t t t 1 f    16 "601" 100 0 0 100    lseg_vertical - ));
+DATA(insert OID = 1410 (  isvertical           PGUID 12 f t t t 1 f    16 "601" 100 0 0 100    lseg_vertical - ));
 DESCR("vertical?");
-DATA(insert OID = 1411 (  ishorizontal         PGUID 11 f t t t 1 f    16 "601" 100 0 0 100    lseg_horizontal - ));
+DATA(insert OID = 1411 (  ishorizontal         PGUID 12 f t t t 1 f    16 "601" 100 0 0 100    lseg_horizontal - ));
 DESCR("horizontal?");
-DATA(insert OID = 1412 (  isparallel           PGUID 11 f t t t 2 f    16 "628 628" 100 0 0 100  line_parallel - ));
+DATA(insert OID = 1412 (  isparallel           PGUID 12 f t t t 2 f    16 "628 628" 100 0 0 100  line_parallel - ));
 DESCR("lines parallel?");
-DATA(insert OID = 1413 (  isperp                       PGUID 11 f t t t 2 f    16 "628 628" 100 0 0 100  line_perp - ));
+DATA(insert OID = 1413 (  isperp                       PGUID 12 f t t t 2 f    16 "628 628" 100 0 0 100  line_perp - ));
 DESCR("lines perpendicular?");
-DATA(insert OID = 1414 (  isvertical           PGUID 11 f t t t 1 f    16 "628" 100 0 0 100  line_vertical - ));
+DATA(insert OID = 1414 (  isvertical           PGUID 12 f t t t 1 f    16 "628" 100 0 0 100  line_vertical - ));
 DESCR("lines vertical?");
-DATA(insert OID = 1415 (  ishorizontal         PGUID 11 f t t t 1 f    16 "628" 100 0 0 100  line_horizontal - ));
+DATA(insert OID = 1415 (  ishorizontal         PGUID 12 f t t t 1 f    16 "628" 100 0 0 100  line_horizontal - ));
 DESCR("lines horizontal?");
-DATA(insert OID = 1416 (  point                                PGUID 11 f t t t 1 f 600 "718" 100 0 1 0        circle_center - ));
+DATA(insert OID = 1416 (  point                                PGUID 12 f t t t 1 f 600 "718" 100 0 1 0        circle_center - ));
 DESCR("center of");
 
 DATA(insert OID = 1417 (  isnottrue                    PGUID 12 f t t f 1 f 16 "16" 100 0 0 100  isnottrue - ));
@@ -1758,15 +1756,15 @@ DESCR("bool is not true (ie, false or unknown)");
 DATA(insert OID = 1418 (  isnotfalse           PGUID 12 f t t f 1 f 16 "16" 100 0 0 100  isnotfalse - ));
 DESCR("bool is not false (ie, true or unknown)");
 
-DATA(insert OID = 1421 (  box                          PGUID 11 f t t t 2 f 603 "600 600" 100 0 0 100  box - ));
+DATA(insert OID = 1421 (  box                          PGUID 12 f t t t 2 f 603 "600 600" 100 0 0 100  points_box - ));
 DESCR("convert points to box");
-DATA(insert OID = 1422 (  box_add                      PGUID 11 f t t t 2 f 603 "603 600" 100 0 0 100  box_add - ));
+DATA(insert OID = 1422 (  box_add                      PGUID 12 f t t t 2 f 603 "603 600" 100 0 0 100  box_add - ));
 DESCR("add point to box (translate)");
-DATA(insert OID = 1423 (  box_sub                      PGUID 11 f t t t 2 f 603 "603 600" 100 0 0 100  box_sub - ));
+DATA(insert OID = 1423 (  box_sub                      PGUID 12 f t t t 2 f 603 "603 600" 100 0 0 100  box_sub - ));
 DESCR("subtract point from box (translate)");
-DATA(insert OID = 1424 (  box_mul                      PGUID 11 f t t t 2 f 603 "603 600" 100 0 0 100  box_mul - ));
+DATA(insert OID = 1424 (  box_mul                      PGUID 12 f t t t 2 f 603 "603 600" 100 0 0 100  box_mul - ));
 DESCR("multiply box by point (scale)");
-DATA(insert OID = 1425 (  box_div                      PGUID 11 f t t t 2 f 603 "603 600" 100 0 0 100  box_div - ));
+DATA(insert OID = 1425 (  box_div                      PGUID 12 f t t t 2 f 603 "603 600" 100 0 0 100  box_div - ));
 DESCR("divide box by point (scale)");
 DATA(insert OID = 1426 (  path_contain_pt      PGUID 14 f t t t 2 f    16 "602 600" 100 0 0 100  "select on_ppath($2, $1)" - ));
 DESCR("path contains point?");
@@ -1801,15 +1799,15 @@ DESCR("multiply (rotate/scale path)");
 DATA(insert OID = 1439 (  path_div_pt          PGUID 12 f t t t 2 f 602 "602 600" 100 0 0 100  path_div_pt - ));
 DESCR("divide (rotate/scale path)");
 
-DATA(insert OID = 1440 (  point                                PGUID 11 f t t t 2 f 600 "701 701" 100 0 0 100  point - ));
+DATA(insert OID = 1440 (  point                                PGUID 12 f t t t 2 f 600 "701 701" 100 0 0 100  construct_point - ));
 DESCR("convert x, y to point");
-DATA(insert OID = 1441 (  point_add                    PGUID 11 f t t t 2 f 600 "600 600" 100 0 0 100  point_add - ));
+DATA(insert OID = 1441 (  point_add                    PGUID 12 f t t t 2 f 600 "600 600" 100 0 0 100  point_add - ));
 DESCR("add points (translate)");
-DATA(insert OID = 1442 (  point_sub                    PGUID 11 f t t t 2 f 600 "600 600" 100 0 0 100  point_sub - ));
+DATA(insert OID = 1442 (  point_sub                    PGUID 12 f t t t 2 f 600 "600 600" 100 0 0 100  point_sub - ));
 DESCR("subtract points (translate)");
-DATA(insert OID = 1443 (  point_mul                    PGUID 11 f t t t 2 f 600 "600 600" 100 0 0 100  point_mul - ));
+DATA(insert OID = 1443 (  point_mul                    PGUID 12 f t t t 2 f 600 "600 600" 100 0 0 100  point_mul - ));
 DESCR("multiply points (scale/rotate)");
-DATA(insert OID = 1444 (  point_div                    PGUID 11 f t t t 2 f 600 "600 600" 100 0 0 100  point_div - ));
+DATA(insert OID = 1444 (  point_div                    PGUID 12 f t t t 2 f 600 "600 600" 100 0 0 100  point_div - ));
 DESCR("divide points (scale/rotate)");
 
 DATA(insert OID = 1445 (  poly_npoints         PGUID 12 f t t t 1 f    23 "604" 100 0 0 100  poly_npoints - ));
@@ -1823,130 +1821,130 @@ DESCR("convert box to polygon");
 DATA(insert OID = 1449 (  polygon                      PGUID 12 f t t t 1 f 604 "602" 100 0 0 100  path_poly - ));
 DESCR("convert path to polygon");
 
-DATA(insert OID = 1450 (  circle_in                    PGUID 11 f t t t 1 f 718 "0" 100 0 1 0  circle_in - ));
+DATA(insert OID = 1450 (  circle_in                    PGUID 12 f t t t 1 f 718 "0" 100 0 1 0  circle_in - ));
 DESCR("(internal)");
-DATA(insert OID = 1451 (  circle_out           PGUID 11 f t t t 1 f    23      "0" 100 0 1 0  circle_out - ));
+DATA(insert OID = 1451 (  circle_out           PGUID 12 f t t t 1 f    23      "718" 100 0 1 0  circle_out - ));
 DESCR("(internal)");
-DATA(insert OID = 1452 (  circle_same          PGUID 11 f t t t 2 f    16 "718 718" 100 0 1 0  circle_same - ));
+DATA(insert OID = 1452 (  circle_same          PGUID 12 f t t t 2 f    16 "718 718" 100 0 1 0  circle_same - ));
 DESCR("same as");
-DATA(insert OID = 1453 (  circle_contain       PGUID 11 f t t t 2 f    16 "718 718" 100 0 1 0  circle_contain - ));
+DATA(insert OID = 1453 (  circle_contain       PGUID 12 f t t t 2 f    16 "718 718" 100 0 1 0  circle_contain - ));
 DESCR("contains");
-DATA(insert OID = 1454 (  circle_left          PGUID 11 f t t t 2 f    16 "718 718" 100 0 1 0  circle_left - ));
+DATA(insert OID = 1454 (  circle_left          PGUID 12 f t t t 2 f    16 "718 718" 100 0 1 0  circle_left - ));
 DESCR("is left of");
-DATA(insert OID = 1455 (  circle_overleft      PGUID 11 f t t t 2 f    16 "718 718" 100 0 1 0  circle_overleft - ));
+DATA(insert OID = 1455 (  circle_overleft      PGUID 12 f t t t 2 f    16 "718 718" 100 0 1 0  circle_overleft - ));
 DESCR("overlaps, but does not extend to right of");
-DATA(insert OID = 1456 (  circle_overright     PGUID 11 f t t t 2 f    16 "718 718" 100 0 1 0  circle_overright - ));
+DATA(insert OID = 1456 (  circle_overright     PGUID 12 f t t t 2 f    16 "718 718" 100 0 1 0  circle_overright - ));
 DESCR("");
-DATA(insert OID = 1457 (  circle_right         PGUID 11 f t t t 2 f    16 "718 718" 100 0 1 0  circle_right - ));
-DESCR("is left of");
-DATA(insert OID = 1458 (  circle_contained     PGUID 11 f t t t 2 f    16 "718 718" 100 0 1 0  circle_contained - ));
+DATA(insert OID = 1457 (  circle_right         PGUID 12 f t t t 2 f    16 "718 718" 100 0 1 0  circle_right - ));
+DESCR("is right of");
+DATA(insert OID = 1458 (  circle_contained     PGUID 12 f t t t 2 f    16 "718 718" 100 0 1 0  circle_contained - ));
 DESCR("");
-DATA(insert OID = 1459 (  circle_overlap       PGUID 11 f t t t 2 f    16 "718 718" 100 0 1 0  circle_overlap - ));
+DATA(insert OID = 1459 (  circle_overlap       PGUID 12 f t t t 2 f    16 "718 718" 100 0 1 0  circle_overlap - ));
 DESCR("overlaps");
-DATA(insert OID = 1460 (  circle_below         PGUID 11 f t t t 2 f    16 "718 718" 100 0 1 0  circle_below - ));
+DATA(insert OID = 1460 (  circle_below         PGUID 12 f t t t 2 f    16 "718 718" 100 0 1 0  circle_below - ));
 DESCR("is below");
-DATA(insert OID = 1461 (  circle_above         PGUID 11 f t t t 2 f    16 "718 718" 100 0 1 0  circle_above - ));
+DATA(insert OID = 1461 (  circle_above         PGUID 12 f t t t 2 f    16 "718 718" 100 0 1 0  circle_above - ));
 DESCR("is above");
-DATA(insert OID = 1462 (  circle_eq                    PGUID 11 f t t t 2 f    16 "718 718" 100 0 1 0  circle_eq - ));
-DESCR("equal");
-DATA(insert OID = 1463 (  circle_ne                    PGUID 11 f t t t 2 f    16 "718 718" 100 0 1 0  circle_ne - ));
-DESCR("not equal");
-DATA(insert OID = 1464 (  circle_lt                    PGUID 11 f t t t 2 f    16 "718 718" 100 0 1 0  circle_lt - ));
-DESCR("less-than");
-DATA(insert OID = 1465 (  circle_gt                    PGUID 11 f t t t 2 f    16 "718 718" 100 0 1 0  circle_gt - ));
-DESCR("greater-than");
-DATA(insert OID = 1466 (  circle_le                    PGUID 11 f t t t 2 f    16 "718 718" 100 0 1 0  circle_le - ));
-DESCR("less-than-or-equal");
-DATA(insert OID = 1467 (  circle_ge                    PGUID 11 f t t t 2 f    16 "718 718" 100 0 1 0  circle_ge - ));
-DESCR("greater-than-or-equal");
-DATA(insert OID = 1468 (  area                         PGUID 11 f t t t 1 f 701 "718" 100 0 1 0        circle_area - ));
+DATA(insert OID = 1462 (  circle_eq                    PGUID 12 f t t t 2 f    16 "718 718" 100 0 1 0  circle_eq - ));
+DESCR("equal by area");
+DATA(insert OID = 1463 (  circle_ne                    PGUID 12 f t t t 2 f    16 "718 718" 100 0 1 0  circle_ne - ));
+DESCR("not equal by area");
+DATA(insert OID = 1464 (  circle_lt                    PGUID 12 f t t t 2 f    16 "718 718" 100 0 1 0  circle_lt - ));
+DESCR("less-than by area");
+DATA(insert OID = 1465 (  circle_gt                    PGUID 12 f t t t 2 f    16 "718 718" 100 0 1 0  circle_gt - ));
+DESCR("greater-than by area");
+DATA(insert OID = 1466 (  circle_le                    PGUID 12 f t t t 2 f    16 "718 718" 100 0 1 0  circle_le - ));
+DESCR("less-than-or-equal by area");
+DATA(insert OID = 1467 (  circle_ge                    PGUID 12 f t t t 2 f    16 "718 718" 100 0 1 0  circle_ge - ));
+DESCR("greater-than-or-equal by area");
+DATA(insert OID = 1468 (  area                         PGUID 12 f t t t 1 f 701 "718" 100 0 1 0        circle_area - ));
 DESCR("area of circle");
-DATA(insert OID = 1469 (  diameter                     PGUID 11 f t t t 1 f 701 "718" 100 0 1 0        circle_diameter - ));
+DATA(insert OID = 1469 (  diameter                     PGUID 12 f t t t 1 f 701 "718" 100 0 1 0        circle_diameter - ));
 DESCR("diameter of circle");
-DATA(insert OID = 1470 (  radius                       PGUID 11 f t t t 1 f 701 "718" 100 0 1 0        circle_radius - ));
+DATA(insert OID = 1470 (  radius                       PGUID 12 f t t t 1 f 701 "718" 100 0 1 0        circle_radius - ));
 DESCR("radius of circle");
-DATA(insert OID = 1471 (  circle_distance      PGUID 11 f t t t 2 f 701 "718 718" 100 0 1 0    circle_distance - ));
+DATA(insert OID = 1471 (  circle_distance      PGUID 12 f t t t 2 f 701 "718 718" 100 0 1 0    circle_distance - ));
 DESCR("distance between");
-DATA(insert OID = 1472 (  circle_center                PGUID 11 f t t t 1 f 600 "718" 100 0 1 0        circle_center - ));
+DATA(insert OID = 1472 (  circle_center                PGUID 12 f t t t 1 f 600 "718" 100 0 1 0        circle_center - ));
 DESCR("center of");
-DATA(insert OID = 1473 (  circle                       PGUID 11 f t t t 2 f 718 "600 701" 100 0 1 0    circle - ));
+DATA(insert OID = 1473 (  circle                       PGUID 12 f t t t 2 f 718 "600 701" 100 0 1 0    cr_circle - ));
 DESCR("convert point and radius to circle");
 DATA(insert OID = 1474 (  circle                       PGUID 12 f t t t 1 f 718 "604" 100 0 1 0        poly_circle - ));
 DESCR("convert polygon to circle");
 DATA(insert OID = 1475 (  polygon                      PGUID 12 f t t t 2 f 604 "23 718" 100 0 1 0  circle_poly - ));
 DESCR("convert vertex count and circle to polygon");
-DATA(insert OID = 1476 (  dist_pc                      PGUID 11 f t t t 2 f 701 "600 718" 100 0 1 0    dist_pc - ));
+DATA(insert OID = 1476 (  dist_pc                      PGUID 12 f t t t 2 f 701 "600 718" 100 0 1 0    dist_pc - ));
 DESCR("distance between point and circle");
-DATA(insert OID = 1477 (  circle_contain_pt PGUID 11 f t t t 2 f       16 "718 600" 100 0 0 100  circle_contain_pt - ));
+DATA(insert OID = 1477 (  circle_contain_pt PGUID 12 f t t t 2 f       16 "718 600" 100 0 0 100  circle_contain_pt - ));
 DESCR("circle contains point?");
-DATA(insert OID = 1478 (  pt_contained_circle  PGUID 11 f t t t 2 f    16 "600 718" 100 0 0 100  pt_contained_circle - ));
+DATA(insert OID = 1478 (  pt_contained_circle  PGUID 12 f t t t 2 f    16 "600 718" 100 0 0 100  pt_contained_circle - ));
 DESCR("point inside circle?");
-DATA(insert OID = 1479 (  circle                       PGUID 11 f t t t 1 f 718 "603" 100 0 1 0        box_circle - ));
+DATA(insert OID = 1479 (  circle                       PGUID 12 f t t t 1 f 718 "603" 100 0 1 0        box_circle - ));
 DESCR("convert box to circle");
-DATA(insert OID = 1480 (  box                          PGUID 11 f t t t 1 f 603 "718" 100 0 1 0        circle_box - ));
+DATA(insert OID = 1480 (  box                          PGUID 12 f t t t 1 f 603 "718" 100 0 1 0        circle_box - ));
 DESCR("convert circle to box");
 DATA(insert OID = 1481 (  tinterval                     PGUID 12 f t f t 2 f 704 "702 702" 100 0 0 100 mktinterval - ));
 DESCR("convert to tinterval");
 
-DATA(insert OID = 1482 (  lseg_ne                      PGUID 11 f t t t 2 f    16 "601 601" 100 0 0 100  lseg_ne - ));
+DATA(insert OID = 1482 (  lseg_ne                      PGUID 12 f t t t 2 f    16 "601 601" 100 0 0 100  lseg_ne - ));
 DESCR("not equal");
-DATA(insert OID = 1483 (  lseg_lt                      PGUID 11 f t t t 2 f    16 "601 601" 100 0 0 100  lseg_lt - ));
-DESCR("less-than");
-DATA(insert OID = 1484 (  lseg_le                      PGUID 11 f t t t 2 f    16 "601 601" 100 0 0 100  lseg_le - ));
-DESCR("less-than-or-equal");
-DATA(insert OID = 1485 (  lseg_gt                      PGUID 11 f t t t 2 f    16 "601 601" 100 0 0 100  lseg_gt - ));
-DESCR("greater-than");
-DATA(insert OID = 1486 (  lseg_ge                      PGUID 11 f t t t 2 f    16 "601 601" 100 0 0 100  lseg_ge - ));
-DESCR("greater-than-or-equal");
-DATA(insert OID = 1487 (  lseg_length          PGUID 11 f t t t 1 f 701 "601" 100 0 1 0        lseg_length - ));
+DATA(insert OID = 1483 (  lseg_lt                      PGUID 12 f t t t 2 f    16 "601 601" 100 0 0 100  lseg_lt - ));
+DESCR("less-than by length");
+DATA(insert OID = 1484 (  lseg_le                      PGUID 12 f t t t 2 f    16 "601 601" 100 0 0 100  lseg_le - ));
+DESCR("less-than-or-equal by length");
+DATA(insert OID = 1485 (  lseg_gt                      PGUID 12 f t t t 2 f    16 "601 601" 100 0 0 100  lseg_gt - ));
+DESCR("greater-than by length");
+DATA(insert OID = 1486 (  lseg_ge                      PGUID 12 f t t t 2 f    16 "601 601" 100 0 0 100  lseg_ge - ));
+DESCR("greater-than-or-equal by length");
+DATA(insert OID = 1487 (  lseg_length          PGUID 12 f t t t 1 f 701 "601" 100 0 1 0        lseg_length - ));
 DESCR("distance between endpoints");
-DATA(insert OID = 1488 (  close_ls                     PGUID 11 f t t t 2 f 600 "628 601" 100 0 10 100  close_ls - ));
+DATA(insert OID = 1488 (  close_ls                     PGUID 12 f t t t 2 f 600 "628 601" 100 0 10 100  close_ls - ));
 DESCR("closest point to line on line segment");
-DATA(insert OID = 1489 (  close_lseg           PGUID 11 f t t t 2 f 600 "601 601" 100 0 10 100  close_lseg - ));
+DATA(insert OID = 1489 (  close_lseg           PGUID 12 f t t t 2 f 600 "601 601" 100 0 10 100  close_lseg - ));
 DESCR("closest point to line segment on line segment");
 
-DATA(insert OID = 1490 (  line_in                      PGUID 11 f t t t 1 f 628 "0" 100 0 0 100        line_in - ));
+DATA(insert OID = 1490 (  line_in                      PGUID 12 f t t t 1 f 628 "0" 100 0 0 100        line_in - ));
 DESCR("(internal)");
-DATA(insert OID = 1491 (  line_out                     PGUID 11 f t t t 1 f 23  "0" 100 0 0 100        line_out - ));
+DATA(insert OID = 1491 (  line_out                     PGUID 12 f t t t 1 f 23  "628" 100 0 0 100      line_out - ));
 DESCR("(internal)");
-DATA(insert OID = 1492 (  line_eq                      PGUID 11 f t t t 2 f    16 "628 628" 100 0 0 100  line_eq - ));
+DATA(insert OID = 1492 (  line_eq                      PGUID 12 f t t t 2 f  16 "628 628" 100 0 0 100  line_eq - ));
 DESCR("lines equal?");
-DATA(insert OID = 1493 (  line                         PGUID 11 f t t t 2 f 628 "600 600" 100 0 0 100  line_construct_pp - ));
+DATA(insert OID = 1493 (  line                         PGUID 12 f t t t 2 f 628 "600 600" 100 0 0 100  line_construct_pp - ));
 DESCR("line from points");
-DATA(insert OID = 1494 (  line_interpt         PGUID 11 f t t t 2 f 600 "628 628" 100 0 0 100  line_interpt - ));
+DATA(insert OID = 1494 (  line_interpt         PGUID 12 f t t t 2 f 600 "628 628" 100 0 0 100  line_interpt - ));
 DESCR("intersection point");
-DATA(insert OID = 1495 (  line_intersect       PGUID 11 f t t t 2 f    16 "628 628" 100 0 0 100  line_intersect - ));
+DATA(insert OID = 1495 (  line_intersect       PGUID 12 f t t t 2 f    16 "628 628" 100 0 0 100  line_intersect - ));
 DESCR("lines intersect?");
-DATA(insert OID = 1496 (  line_parallel                PGUID 11 f t t t 2 f    16 "628 628" 100 0 0 100  line_parallel - ));
+DATA(insert OID = 1496 (  line_parallel                PGUID 12 f t t t 2 f    16 "628 628" 100 0 0 100  line_parallel - ));
 DESCR("lines parallel?");
-DATA(insert OID = 1497 (  line_perp                    PGUID 11 f t t t 2 f    16 "628 628" 100 0 0 100  line_perp - ));
+DATA(insert OID = 1497 (  line_perp                    PGUID 12 f t t t 2 f    16 "628 628" 100 0 0 100  line_perp - ));
 DESCR("lines perpendicular?");
-DATA(insert OID = 1498 (  line_vertical                PGUID 11 f t t t 1 f    16 "628" 100 0 0 100  line_vertical - ));
+DATA(insert OID = 1498 (  line_vertical                PGUID 12 f t t t 1 f    16 "628" 100 0 0 100  line_vertical - ));
 DESCR("lines vertical?");
-DATA(insert OID = 1499 (  line_horizontal      PGUID 11 f t t t 1 f    16 "628" 100 0 0 100  line_horizontal - ));
+DATA(insert OID = 1499 (  line_horizontal      PGUID 12 f t t t 1 f    16 "628" 100 0 0 100  line_horizontal - ));
 DESCR("lines horizontal?");
 
 /* OIDS 1500 - 1599 */
 
-DATA(insert OID = 1530 (  length                       PGUID 11 f t t t 1 f 701 "601" 100 0 1 0        lseg_length - ));
+DATA(insert OID = 1530 (  length                       PGUID 12 f t t t 1 f 701 "601" 100 0 1 0        lseg_length - ));
 DESCR("distance between endpoints");
 DATA(insert OID = 1531 (  length                       PGUID 12 f t t t 1 f 701 "602" 100 0 1 0        path_length - ));
 DESCR("sum of path segments");
 
 
-DATA(insert OID = 1532 (  point                                PGUID 11 f t t t 1 f 600 "601" 100 0 0 100  lseg_center - ));
+DATA(insert OID = 1532 (  point                                PGUID 12 f t t t 1 f 600 "601" 100 0 0 100  lseg_center - ));
 DESCR("center of");
 DATA(insert OID = 1533 (  point                                PGUID 12 f t t t 1 f 600 "602" 100 0 0 100  path_center - ));
 DESCR("center of");
-DATA(insert OID = 1534 (  point                                PGUID 11 f t t t 1 f 600 "603" 100 1 0 100  box_center - ));
+DATA(insert OID = 1534 (  point                                PGUID 12 f t t t 1 f 600 "603" 100 1 0 100  box_center - ));
 DESCR("center of");
 DATA(insert OID = 1540 (  point                                PGUID 12 f t t t 1 f 600 "604" 100 0 0 100  poly_center - ));
 DESCR("center of");
-DATA(insert OID = 1541 (  lseg                         PGUID 11 f t t t 1 f 601 "603" 100 0 0 100  box_diagonal - ));
-DESCR("");
-DATA(insert OID = 1542 (  center                       PGUID 11 f t t t 1 f 600 "603" 100 1 0 100  box_center - ));
+DATA(insert OID = 1541 (  lseg                         PGUID 12 f t t t 1 f 601 "603" 100 0 0 100  box_diagonal - ));
+DESCR("diagonal of");
+DATA(insert OID = 1542 (  center                       PGUID 12 f t t t 1 f 600 "603" 100 1 0 100  box_center - ));
 DESCR("center of");
-DATA(insert OID = 1543 (  center                       PGUID 11 f t t t 1 f 600 "718" 100 0 1 0        circle_center - ));
+DATA(insert OID = 1543 (  center                       PGUID 12 f t t t 1 f 600 "718" 100 0 1 0        circle_center - ));
 DESCR("center of");
 DATA(insert OID = 1544 (  polygon                      PGUID 14 f t t t 1 f 604 "718" 100 0 0 100  "select polygon(12, $1)" - ));
 DESCR("convert circle to 12-vertex polygon");
index f23f3b64968cb5a58cba3e5d3e1bedcda1c10466..84193ca0ef80b72b84fc2cdf04bd0e4291c5d4f5 100644 (file)
@@ -6,7 +6,7 @@
  * Portions Copyright (c) 1996-2000, PostgreSQL, Inc
  * Portions Copyright (c) 1994, Regents of the University of California
  *
- * $Id: geo_decls.h,v 1.30 2000/07/29 18:46:05 tgl Exp $
+ * $Id: geo_decls.h,v 1.31 2000/07/30 20:43:49 tgl Exp $
  *
  * NOTE
  *       These routines do *not* use the float types from adt/.
 #ifdef EPSILON
 #define FPzero(A)                              (fabs(A) <= EPSILON)
 #define FPeq(A,B)                              (fabs((A) - (B)) <= EPSILON)
+#define FPne(A,B)                              (fabs((A) - (B)) > EPSILON)
 #define FPlt(A,B)                              ((B) - (A) > EPSILON)
 #define FPle(A,B)                              ((A) - (B) <= EPSILON)
 #define FPgt(A,B)                              ((A) - (B) > EPSILON)
 #define FPge(A,B)                              ((B) - (A) <= EPSILON)
 #else
-#define FPzero(A)                              (A == 0)
-#define FPnzero(A)                             (A != 0)
-#define FPeq(A,B)                              (A == B)
-#define FPne(A,B)                              (A != B)
-#define FPlt(A,B)                              (A < B)
-#define FPle(A,B)                              (A <= B)
-#define FPgt(A,B)                              (A > B)
-#define FPge(A,B)                              (A >= B)
+#define FPzero(A)                              ((A) == 0)
+#define FPeq(A,B)                              ((A) == (B))
+#define FPne(A,B)                              ((A) != (B))
+#define FPlt(A,B)                              ((A) < (B))
+#define FPle(A,B)                              ((A) <= (B))
+#define FPgt(A,B)                              ((A) > (B))
+#define FPge(A,B)                              ((A) >= (B))
 #endif
 
 #define HYPOT(A, B)                            sqrt((A) * (A) + (B) * (B))
@@ -187,125 +187,115 @@ typedef struct
  */
 
 /* public point routines */
-extern Point *point_in(char *str);
-extern char *point_out(Point *pt);
-extern bool point_left(Point *pt1, Point *pt2);
-extern bool point_right(Point *pt1, Point *pt2);
-extern bool point_above(Point *pt1, Point *pt2);
-extern bool point_below(Point *pt1, Point *pt2);
-extern bool point_vert(Point *pt1, Point *pt2);
-extern bool point_horiz(Point *pt1, Point *pt2);
-extern bool point_eq(Point *pt1, Point *pt2);
-extern bool point_ne(Point *pt1, Point *pt2);
-extern int32 pointdist(Point *p1, Point *p2);
-extern double *point_distance(Point *pt1, Point *pt2);
-extern double *point_slope(Point *pt1, Point *pt2);
+extern Datum point_in(PG_FUNCTION_ARGS);
+extern Datum point_out(PG_FUNCTION_ARGS);
+extern Datum construct_point(PG_FUNCTION_ARGS);
+extern Datum point_left(PG_FUNCTION_ARGS);
+extern Datum point_right(PG_FUNCTION_ARGS);
+extern Datum point_above(PG_FUNCTION_ARGS);
+extern Datum point_below(PG_FUNCTION_ARGS);
+extern Datum point_vert(PG_FUNCTION_ARGS);
+extern Datum point_horiz(PG_FUNCTION_ARGS);
+extern Datum point_eq(PG_FUNCTION_ARGS);
+extern Datum point_ne(PG_FUNCTION_ARGS);
+extern Datum point_distance(PG_FUNCTION_ARGS);
+extern Datum point_slope(PG_FUNCTION_ARGS);
+extern Datum point_add(PG_FUNCTION_ARGS);
+extern Datum point_sub(PG_FUNCTION_ARGS);
+extern Datum point_mul(PG_FUNCTION_ARGS);
+extern Datum point_div(PG_FUNCTION_ARGS);
 
 /* private routines */
 extern double point_dt(Point *pt1, Point *pt2);
 extern double point_sl(Point *pt1, Point *pt2);
 
-extern Point *point(float8 *x, float8 *y);
-extern Point *point_add(Point *p1, Point *p2);
-extern Point *point_sub(Point *p1, Point *p2);
-extern Point *point_mul(Point *p1, Point *p2);
-extern Point *point_div(Point *p1, Point *p2);
-
 /* public lseg routines */
-extern LSEG *lseg_in(char *str);
-extern char *lseg_out(LSEG *ls);
-extern bool lseg_intersect(LSEG *l1, LSEG *l2);
-extern bool lseg_parallel(LSEG *l1, LSEG *l2);
-extern bool lseg_perp(LSEG *l1, LSEG *l2);
-extern bool lseg_vertical(LSEG *lseg);
-extern bool lseg_horizontal(LSEG *lseg);
-extern bool lseg_eq(LSEG *l1, LSEG *l2);
-extern bool lseg_ne(LSEG *l1, LSEG *l2);
-extern bool lseg_lt(LSEG *l1, LSEG *l2);
-extern bool lseg_le(LSEG *l1, LSEG *l2);
-extern bool lseg_gt(LSEG *l1, LSEG *l2);
-extern bool lseg_ge(LSEG *l1, LSEG *l2);
-extern LSEG *lseg_construct(Point *pt1, Point *pt2);
-extern double *lseg_length(LSEG *lseg);
-extern double *lseg_distance(LSEG *l1, LSEG *l2);
-extern Point *lseg_center(LSEG *lseg);
-extern Point *lseg_interpt(LSEG *l1, LSEG *l2);
-extern double *dist_pl(Point *pt, LINE *line);
-extern double *dist_ps(Point *pt, LSEG *lseg);
+extern Datum lseg_in(PG_FUNCTION_ARGS);
+extern Datum lseg_out(PG_FUNCTION_ARGS);
+extern Datum lseg_intersect(PG_FUNCTION_ARGS);
+extern Datum lseg_parallel(PG_FUNCTION_ARGS);
+extern Datum lseg_perp(PG_FUNCTION_ARGS);
+extern Datum lseg_vertical(PG_FUNCTION_ARGS);
+extern Datum lseg_horizontal(PG_FUNCTION_ARGS);
+extern Datum lseg_eq(PG_FUNCTION_ARGS);
+extern Datum lseg_ne(PG_FUNCTION_ARGS);
+extern Datum lseg_lt(PG_FUNCTION_ARGS);
+extern Datum lseg_le(PG_FUNCTION_ARGS);
+extern Datum lseg_gt(PG_FUNCTION_ARGS);
+extern Datum lseg_ge(PG_FUNCTION_ARGS);
+extern Datum lseg_construct(PG_FUNCTION_ARGS);
+extern Datum lseg_length(PG_FUNCTION_ARGS);
+extern Datum lseg_distance(PG_FUNCTION_ARGS);
+extern Datum lseg_center(PG_FUNCTION_ARGS);
+extern Datum lseg_interpt(PG_FUNCTION_ARGS);
+extern Datum dist_pl(PG_FUNCTION_ARGS);
+extern Datum dist_ps(PG_FUNCTION_ARGS);
 extern Datum dist_ppath(PG_FUNCTION_ARGS);
-extern double *dist_pb(Point *pt, BOX *box);
-extern double *dist_sl(LSEG *lseg, LINE *line);
-extern double *dist_sb(LSEG *lseg, BOX *box);
-extern double *dist_lb(LINE *line, BOX *box);
-extern Point *close_lseg(LSEG *l1, LSEG *l2);
-extern Point *close_pl(Point *pt, LINE *line);
-extern Point *close_ps(Point *pt, LSEG *lseg);
-extern Point *close_pb(Point *pt, BOX *box);
-extern Point *close_sl(LSEG *lseg, LINE *line);
-extern Point *close_sb(LSEG *lseg, BOX *box);
-extern Point *close_ls(LINE *line, LSEG *lseg);
-extern Point *close_lb(LINE *line, BOX *box);
-extern bool on_pl(Point *pt, LINE *line);
-extern bool on_ps(Point *pt, LSEG *lseg);
-extern bool on_pb(Point *pt, BOX *box);
+extern Datum dist_pb(PG_FUNCTION_ARGS);
+extern Datum dist_sl(PG_FUNCTION_ARGS);
+extern Datum dist_sb(PG_FUNCTION_ARGS);
+extern Datum dist_lb(PG_FUNCTION_ARGS);
+extern Datum close_lseg(PG_FUNCTION_ARGS);
+extern Datum close_pl(PG_FUNCTION_ARGS);
+extern Datum close_ps(PG_FUNCTION_ARGS);
+extern Datum close_pb(PG_FUNCTION_ARGS);
+extern Datum close_sl(PG_FUNCTION_ARGS);
+extern Datum close_sb(PG_FUNCTION_ARGS);
+extern Datum close_ls(PG_FUNCTION_ARGS);
+extern Datum close_lb(PG_FUNCTION_ARGS);
+extern Datum on_pl(PG_FUNCTION_ARGS);
+extern Datum on_ps(PG_FUNCTION_ARGS);
+extern Datum on_pb(PG_FUNCTION_ARGS);
 extern Datum on_ppath(PG_FUNCTION_ARGS);
-extern bool on_sl(LSEG *lseg, LINE *line);
-extern bool on_sb(LSEG *lseg, BOX *box);
-extern bool inter_sl(LSEG *lseg, LINE *line);
-extern bool inter_sb(LSEG *lseg, BOX *box);
-extern bool inter_lb(LINE *line, BOX *box);
-
-/* private lseg routines */
+extern Datum on_sl(PG_FUNCTION_ARGS);
+extern Datum on_sb(PG_FUNCTION_ARGS);
+extern Datum inter_sl(PG_FUNCTION_ARGS);
+extern Datum inter_sb(PG_FUNCTION_ARGS);
+extern Datum inter_lb(PG_FUNCTION_ARGS);
 
 /* public line routines */
-extern LINE *line_in(char *str);
-extern char *line_out(LINE *line);
-extern Point *line_interpt(LINE *l1, LINE *l2);
-extern double *line_distance(LINE *l1, LINE *l2);
-extern LINE *line_construct_pp(Point *pt1, Point *pt2);
-extern bool line_intersect(LINE *l1, LINE *l2);
-extern bool line_parallel(LINE *l1, LINE *l2);
-extern bool line_perp(LINE *l1, LINE *l2);
-extern bool line_vertical(LINE *line);
-extern bool line_horizontal(LINE *line);
-extern bool line_eq(LINE *l1, LINE *l2);
-
-/* private line routines */
+extern Datum line_in(PG_FUNCTION_ARGS);
+extern Datum line_out(PG_FUNCTION_ARGS);
+extern Datum line_interpt(PG_FUNCTION_ARGS);
+extern Datum line_distance(PG_FUNCTION_ARGS);
+extern Datum line_construct_pp(PG_FUNCTION_ARGS);
+extern Datum line_intersect(PG_FUNCTION_ARGS);
+extern Datum line_parallel(PG_FUNCTION_ARGS);
+extern Datum line_perp(PG_FUNCTION_ARGS);
+extern Datum line_vertical(PG_FUNCTION_ARGS);
+extern Datum line_horizontal(PG_FUNCTION_ARGS);
+extern Datum line_eq(PG_FUNCTION_ARGS);
 
 /* public box routines */
-extern BOX *box_in(char *str);
-extern char *box_out(BOX *box);
-extern bool box_same(BOX *box1, BOX *box2);
-extern bool box_overlap(BOX *box1, BOX *box2);
-extern bool box_overleft(BOX *box1, BOX *box2);
-extern bool box_left(BOX *box1, BOX *box2);
-extern bool box_right(BOX *box1, BOX *box2);
-extern bool box_overright(BOX *box1, BOX *box2);
-extern bool box_contained(BOX *box1, BOX *box2);
-extern bool box_contain(BOX *box1, BOX *box2);
-extern bool box_below(BOX *box1, BOX *box2);
-extern bool box_above(BOX *box1, BOX *box2);
-extern bool box_lt(BOX *box1, BOX *box2);
-extern bool box_gt(BOX *box1, BOX *box2);
-extern bool box_eq(BOX *box1, BOX *box2);
-extern bool box_le(BOX *box1, BOX *box2);
-extern bool box_ge(BOX *box1, BOX *box2);
-extern Point *box_center(BOX *box);
-extern double *box_area(BOX *box);
-extern double *box_width(BOX *box);
-extern double *box_height(BOX *box);
-extern double *box_distance(BOX *box1, BOX *box2);
-extern Point *box_center(BOX *box);
-extern BOX *box_intersect(BOX *box1, BOX *box2);
-extern LSEG *box_diagonal(BOX *box);
-extern BOX *box(Point *p1, Point *p2);
-extern BOX *box_add(BOX *box, Point *p);
-extern BOX *box_sub(BOX *box, Point *p);
-extern BOX *box_mul(BOX *box, Point *p);
-extern BOX *box_div(BOX *box, Point *p);
-
-/* private routines */
-extern double box_dt(BOX *box1, BOX *box2);
+extern Datum box_in(PG_FUNCTION_ARGS);
+extern Datum box_out(PG_FUNCTION_ARGS);
+extern Datum box_same(PG_FUNCTION_ARGS);
+extern Datum box_overlap(PG_FUNCTION_ARGS);
+extern Datum box_overleft(PG_FUNCTION_ARGS);
+extern Datum box_left(PG_FUNCTION_ARGS);
+extern Datum box_right(PG_FUNCTION_ARGS);
+extern Datum box_overright(PG_FUNCTION_ARGS);
+extern Datum box_contained(PG_FUNCTION_ARGS);
+extern Datum box_contain(PG_FUNCTION_ARGS);
+extern Datum box_below(PG_FUNCTION_ARGS);
+extern Datum box_above(PG_FUNCTION_ARGS);
+extern Datum box_lt(PG_FUNCTION_ARGS);
+extern Datum box_gt(PG_FUNCTION_ARGS);
+extern Datum box_eq(PG_FUNCTION_ARGS);
+extern Datum box_le(PG_FUNCTION_ARGS);
+extern Datum box_ge(PG_FUNCTION_ARGS);
+extern Datum box_area(PG_FUNCTION_ARGS);
+extern Datum box_width(PG_FUNCTION_ARGS);
+extern Datum box_height(PG_FUNCTION_ARGS);
+extern Datum box_distance(PG_FUNCTION_ARGS);
+extern Datum box_center(PG_FUNCTION_ARGS);
+extern Datum box_intersect(PG_FUNCTION_ARGS);
+extern Datum box_diagonal(PG_FUNCTION_ARGS);
+extern Datum points_box(PG_FUNCTION_ARGS);
+extern Datum box_add(PG_FUNCTION_ARGS);
+extern Datum box_sub(PG_FUNCTION_ARGS);
+extern Datum box_mul(PG_FUNCTION_ARGS);
+extern Datum box_div(PG_FUNCTION_ARGS);
 
 /* public path routines */
 extern Datum path_in(PG_FUNCTION_ARGS);
@@ -347,7 +337,6 @@ extern Datum poly_contain(PG_FUNCTION_ARGS);
 extern Datum poly_contained(PG_FUNCTION_ARGS);
 extern Datum poly_contain_pt(PG_FUNCTION_ARGS);
 extern Datum pt_contained_poly(PG_FUNCTION_ARGS);
-
 extern Datum poly_distance(PG_FUNCTION_ARGS);
 extern Datum poly_npoints(PG_FUNCTION_ARGS);
 extern Datum poly_center(PG_FUNCTION_ARGS);
@@ -356,52 +345,48 @@ extern Datum poly_path(PG_FUNCTION_ARGS);
 extern Datum box_poly(PG_FUNCTION_ARGS);
 
 /* public circle routines */
-extern CIRCLE *circle_in(char *str);
-extern char *circle_out(CIRCLE *circle);
-extern bool circle_same(CIRCLE *circle1, CIRCLE *circle2);
-extern bool circle_overlap(CIRCLE *circle1, CIRCLE *circle2);
-extern bool circle_overleft(CIRCLE *circle1, CIRCLE *circle2);
-extern bool circle_left(CIRCLE *circle1, CIRCLE *circle2);
-extern bool circle_right(CIRCLE *circle1, CIRCLE *circle2);
-extern bool circle_overright(CIRCLE *circle1, CIRCLE *circle2);
-extern bool circle_contained(CIRCLE *circle1, CIRCLE *circle2);
-extern bool circle_contain(CIRCLE *circle1, CIRCLE *circle2);
-extern bool circle_below(CIRCLE *circle1, CIRCLE *circle2);
-extern bool circle_above(CIRCLE *circle1, CIRCLE *circle2);
-
-extern bool circle_eq(CIRCLE *circle1, CIRCLE *circle2);
-extern bool circle_ne(CIRCLE *circle1, CIRCLE *circle2);
-extern bool circle_lt(CIRCLE *circle1, CIRCLE *circle2);
-extern bool circle_gt(CIRCLE *circle1, CIRCLE *circle2);
-extern bool circle_le(CIRCLE *circle1, CIRCLE *circle2);
-extern bool circle_ge(CIRCLE *circle1, CIRCLE *circle2);
-extern bool circle_contain_pt(CIRCLE *circle, Point *point);
-extern bool pt_contained_circle(Point *point, CIRCLE *circle);
-extern CIRCLE *circle_add_pt(CIRCLE *circle, Point *point);
-extern CIRCLE *circle_sub_pt(CIRCLE *circle, Point *point);
-extern CIRCLE *circle_mul_pt(CIRCLE *circle, Point *point);
-extern CIRCLE *circle_div_pt(CIRCLE *circle, Point *point);
-extern double *circle_diameter(CIRCLE *circle);
-extern double *circle_radius(CIRCLE *circle);
-extern double *circle_distance(CIRCLE *circle1, CIRCLE *circle2);
-extern double *dist_pc(Point *point, CIRCLE *circle);
+extern Datum circle_in(PG_FUNCTION_ARGS);
+extern Datum circle_out(PG_FUNCTION_ARGS);
+extern Datum circle_same(PG_FUNCTION_ARGS);
+extern Datum circle_overlap(PG_FUNCTION_ARGS);
+extern Datum circle_overleft(PG_FUNCTION_ARGS);
+extern Datum circle_left(PG_FUNCTION_ARGS);
+extern Datum circle_right(PG_FUNCTION_ARGS);
+extern Datum circle_overright(PG_FUNCTION_ARGS);
+extern Datum circle_contained(PG_FUNCTION_ARGS);
+extern Datum circle_contain(PG_FUNCTION_ARGS);
+extern Datum circle_below(PG_FUNCTION_ARGS);
+extern Datum circle_above(PG_FUNCTION_ARGS);
+extern Datum circle_eq(PG_FUNCTION_ARGS);
+extern Datum circle_ne(PG_FUNCTION_ARGS);
+extern Datum circle_lt(PG_FUNCTION_ARGS);
+extern Datum circle_gt(PG_FUNCTION_ARGS);
+extern Datum circle_le(PG_FUNCTION_ARGS);
+extern Datum circle_ge(PG_FUNCTION_ARGS);
+extern Datum circle_contain_pt(PG_FUNCTION_ARGS);
+extern Datum pt_contained_circle(PG_FUNCTION_ARGS);
+extern Datum circle_add_pt(PG_FUNCTION_ARGS);
+extern Datum circle_sub_pt(PG_FUNCTION_ARGS);
+extern Datum circle_mul_pt(PG_FUNCTION_ARGS);
+extern Datum circle_div_pt(PG_FUNCTION_ARGS);
+extern Datum circle_diameter(PG_FUNCTION_ARGS);
+extern Datum circle_radius(PG_FUNCTION_ARGS);
+extern Datum circle_distance(PG_FUNCTION_ARGS);
+extern Datum dist_pc(PG_FUNCTION_ARGS);
 extern Datum dist_cpoly(PG_FUNCTION_ARGS);
-extern Point *circle_center(CIRCLE *circle);
-extern CIRCLE *circle(Point *center, float8 *radius);
-extern CIRCLE *box_circle(BOX *box);
-extern BOX *circle_box(CIRCLE *circle);
+extern Datum circle_center(PG_FUNCTION_ARGS);
+extern Datum cr_circle(PG_FUNCTION_ARGS);
+extern Datum box_circle(PG_FUNCTION_ARGS);
+extern Datum circle_box(PG_FUNCTION_ARGS);
 extern Datum poly_circle(PG_FUNCTION_ARGS);
 extern Datum circle_poly(PG_FUNCTION_ARGS);
-
-/* private routines */
-extern double *circle_area(CIRCLE *circle);
-extern double circle_dt(CIRCLE *circle1, CIRCLE *circle2);
+extern Datum circle_area(PG_FUNCTION_ARGS);
 
 /* support routines for the rtree access method (rtproc.c) */
-extern BOX *rt_box_union(BOX *a, BOX *b);
-extern BOX *rt_box_inter(BOX *a, BOX *b);
-extern void rt_box_size(BOX *a, float *size);
-extern void rt_bigbox_size(BOX *a, float *size);
+extern Datum rt_box_union(PG_FUNCTION_ARGS);
+extern Datum rt_box_inter(PG_FUNCTION_ARGS);
+extern Datum rt_box_size(PG_FUNCTION_ARGS);
+extern Datum rt_bigbox_size(PG_FUNCTION_ARGS);
 extern Datum rt_poly_size(PG_FUNCTION_ARGS);
 extern Datum rt_poly_union(PG_FUNCTION_ARGS);
 extern Datum rt_poly_inter(PG_FUNCTION_ARGS);
index a8742b7d207a3cc73f3c95827849b5580bcfec01..19147eadd11c27da5fb42cca482dceec9e119344 100644 (file)
@@ -45,7 +45,7 @@ CREATE FUNCTION boxarea(box)
 CREATE FUNCTION interpt_pp(path, path)
    RETURNS point
    AS '_OBJWD_/regress_DLSUFFIX_'
-   LANGUAGE 'c';
+   LANGUAGE 'newC';
 
 CREATE FUNCTION reverse_name(name)
    RETURNS name
index 105b35858abceff0ef8c43359da7cc9770d7ec6a..32e0f5ff38327d24b250c8c017c73e00e7f29885 100644 (file)
@@ -35,7 +35,7 @@ CREATE FUNCTION boxarea(box)
 CREATE FUNCTION interpt_pp(path, path)
    RETURNS point
    AS '_OBJWD_/regress_DLSUFFIX_'
-   LANGUAGE 'c';
+   LANGUAGE 'newC';
 CREATE FUNCTION reverse_name(name)
    RETURNS name
    AS '_OBJWD_/regress_DLSUFFIX_'
index 964e83466701cee7b569bd7fa9cbb2d56ac6db79..e5fbf962a4ffd869035fd4ed6e987b282a5953f8 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * $Header: /cvsroot/pgsql/src/test/regress/regress.c,v 1.42 2000/07/29 18:46:12 tgl Exp $
+ * $Header: /cvsroot/pgsql/src/test/regress/regress.c,v 1.43 2000/07/30 20:43:54 tgl Exp $
  */
 
 #include <float.h>                             /* faked on sunos */
 
 typedef TupleTableSlot *TUPLE;
 
-extern double *regress_dist_ptpath(Point *pt, PATH *path);
-extern double *regress_path_dist(PATH *p1, PATH *p2);
+extern Datum regress_dist_ptpath(PG_FUNCTION_ARGS);
+extern Datum regress_path_dist(PG_FUNCTION_ARGS);
 extern PATH *poly2path(POLYGON *poly);
-extern Point *interpt_pp(PATH *p1, PATH *p2);
+extern Datum interpt_pp(PG_FUNCTION_ARGS);
 extern void regress_lseg_construct(LSEG *lseg, Point *pt1, Point *pt2);
 extern Datum overpaid(PG_FUNCTION_ARGS);
 extern Datum boxarea(PG_FUNCTION_ARGS);
@@ -29,24 +29,22 @@ extern char *reverse_name(char *string);
 /*
 ** Distance from a point to a path
 */
-double *
-regress_dist_ptpath(pt, path)
-Point     *pt;
-PATH      *path;
+Datum
+regress_dist_ptpath(PG_FUNCTION_ARGS)
 {
-       double     *result;
-       double     *tmp;
+       Point      *pt = PG_GETARG_POINT_P(0);
+       PATH       *path = PG_GETARG_PATH_P(1);
+       float8          result = 0.0;   /* keep compiler quiet */
+       float8          tmp;
        int                     i;
        LSEG            lseg;
 
        switch (path->npts)
        {
                case 0:
-                       result = palloc(sizeof(double));
-                       *result = Abs((double) DBL_MAX);        /* +infinity */
-                       break;
+                       PG_RETURN_NULL();
                case 1:
-                       result = point_distance(pt, &path->p[0]);
+                       result = point_dt(pt, &path->p[0]);
                        break;
                default:
 
@@ -55,51 +53,57 @@ PATH           *path;
                         * distance from the point to any of its constituent segments.
                         */
                        Assert(path->npts > 1);
-                       result = palloc(sizeof(double));
                        for (i = 0; i < path->npts - 1; ++i)
                        {
                                regress_lseg_construct(&lseg, &path->p[i], &path->p[i + 1]);
-                               tmp = dist_ps(pt, &lseg);
-                               if (i == 0 || *tmp < *result)
-                                       *result = *tmp;
-                               pfree(tmp);
-
+                               tmp = DatumGetFloat8(DirectFunctionCall2(dist_ps,
+                                                                                                       PointPGetDatum(pt),
+                                                                                                       LsegPGetDatum(&lseg)));
+                               if (i == 0 || tmp < result)
+                                       result = tmp;
                        }
                        break;
        }
-       return result;
+       PG_RETURN_FLOAT8(result);
 }
 
 /* this essentially does a cartesian product of the lsegs in the
    two paths, and finds the min distance between any two lsegs */
-double *
-regress_path_dist(p1, p2)
-PATH      *p1;
-PATH      *p2;
+Datum
+regress_path_dist(PG_FUNCTION_ARGS)
 {
-       double     *min,
-                          *tmp;
+       PATH       *p1 = PG_GETARG_PATH_P(0);
+       PATH       *p2 = PG_GETARG_PATH_P(1);
+       bool            have_min = false;
+       float8          min = 0.0;              /* initialize to keep compiler quiet */
+       float8          tmp;
        int                     i,
                                j;
        LSEG            seg1,
                                seg2;
 
-       regress_lseg_construct(&seg1, &p1->p[0], &p1->p[1]);
-       regress_lseg_construct(&seg2, &p2->p[0], &p2->p[1]);
-       min = lseg_distance(&seg1, &seg2);
-
        for (i = 0; i < p1->npts - 1; i++)
+       {
                for (j = 0; j < p2->npts - 1; j++)
                {
                        regress_lseg_construct(&seg1, &p1->p[i], &p1->p[i + 1]);
                        regress_lseg_construct(&seg2, &p2->p[j], &p2->p[j + 1]);
 
-                       if (*min < *(tmp = lseg_distance(&seg1, &seg2)))
-                               *min = *tmp;
-                       pfree(tmp);
+                       tmp = DatumGetFloat8(DirectFunctionCall2(lseg_distance,
+                                                                                                        LsegPGetDatum(&seg1),
+                                                                                                        LsegPGetDatum(&seg2)));
+                       if (!have_min || tmp < min)
+                       {
+                               min = tmp;
+                               have_min = true;
+                       }
                }
+       }
 
-       return min;
+       if (! have_min)
+               PG_RETURN_NULL();
+
+       PG_RETURN_FLOAT8(min);
 }
 
 PATH *
@@ -124,43 +128,43 @@ POLYGON    *poly;
                                                                                         CStringGetDatum(output)));
 }
 
-/* return the point where two paths intersect. Assumes that they do. */
-Point *
-interpt_pp(p1, p2)
-PATH      *p1;
-PATH      *p2;
+/* return the point where two paths intersect, or NULL if no intersection. */
+Datum
+interpt_pp(PG_FUNCTION_ARGS)
 {
-
-       Point      *retval;
+       PATH       *p1 = PG_GETARG_PATH_P(0);
+       PATH       *p2 = PG_GETARG_PATH_P(1);
        int                     i,
                                j;
        LSEG            seg1,
                                seg2;
-
-#ifdef NOT_USED
-       LINE       *ln;
-
-#endif
        bool            found;                  /* We've found the intersection */
 
        found = false;                          /* Haven't found it yet */
 
        for (i = 0; i < p1->npts - 1 && !found; i++)
+       {
+               regress_lseg_construct(&seg1, &p1->p[i], &p1->p[i + 1]);
                for (j = 0; j < p2->npts - 1 && !found; j++)
                {
-                       regress_lseg_construct(&seg1, &p1->p[i], &p1->p[i + 1]);
                        regress_lseg_construct(&seg2, &p2->p[j], &p2->p[j + 1]);
-                       if (lseg_intersect(&seg1, &seg2))
+                       if (DatumGetBool(DirectFunctionCall2(lseg_intersect,
+                                                                                                LsegPGetDatum(&seg1),
+                                                                                                LsegPGetDatum(&seg2))))
                                found = true;
                }
+       }
 
-#ifdef NOT_USED
-       ln = line_construct_pp(&seg2.p[0], &seg2.p[1]);
-       retval = interpt_sl(&seg1, ln);
-#endif
-       retval = lseg_interpt(&seg1, &seg2);
+       if (!found)
+               PG_RETURN_NULL();
 
-       return retval;
+       /* Note: DirectFunctionCall2 will kick out an error if lseg_interpt()
+        * returns NULL, but that should be impossible since we know the two
+        * segments intersect.
+        */
+       PG_RETURN_DATUM(DirectFunctionCall2(lseg_interpt,
+                                                                               LsegPGetDatum(&seg1),
+                                                                               LsegPGetDatum(&seg2)));
 }