Re: [PATCH] Revive line type - Mailing list pgsql-hackers

From Peter Eisentraut
Subject Re: [PATCH] Revive line type
Date
Msg-id 1380674566.22785.16.camel@vanquo.pezone.net
Whole thread Raw
In response to Re: [PATCH] Revive line type  (Jeevan Chalke <jeevan.chalke@enterprisedb.com>)
Responses Re: [PATCH] Revive line type  (Jeevan Chalke <jeevan.chalke@enterprisedb.com>)
List pgsql-hackers
On Wed, 2013-09-25 at 14:26 +0530, Jeevan Chalke wrote:
> So no issues from my side.
> 
> However, do we still need this in close_pl() ?
> 
> #ifdef NOT_USED
>     if (FPeq(line->A, -1.0) && FPzero(line->B))
>     {                            /* vertical */
>     }
> #endif

No, that can be removed.

> Also close_sl, close_lb and dist_lb are NOT yet implemented. It will
> be good if we have those. But I don't think we should wait for those
> functions to be implemented.

Right, those are separate projects.






pgsql-hackers by date:

Previous
From: Peter Eisentraut
Date:
Subject: Re: Completing PL support for Event Triggers
Next
From: Peter Eisentraut
Date:
Subject: Re: information schema parameter_default implementation