On Thu, 31 Jul 2003, Tom Lane wrote:

> Gavin Sherry <[EMAIL PROTECTED]> writes:
> > 2) since we're accepting numeric arguments, the patch tests for floats. If
> > a numeric is passed with non-zero decimal portion, an error is raised
> > since (from memory) they are undefined.
> 
> There is a standard mathematical definition for it (gamma function,
> IIRC) but this is probably plenty good enough for our purposes.  I would
> suggest though that you reject fractions before you short-circuit for
> x <= 1.

Oops.

> 
> > 3) I have not removed factorial([int2|int4|int8]), not their operator
> > counterparts since I didn't know what people would want done with these.
> 
> We had already decided to nuke the int2 and int4 versions, since they
> overflow far too easily.  I'd go with nuking int8 too and providing only
> the numeric variant ...

What are your feelings about numeric argument vs. int4/int8 arguments?

> 
> > +   int8_to_numericvar((int64)1, &one);
> > + 
> > +   ret = cmp_var(&fact, &one);     
> 
> Uh, why not use const_one?

Umm.. didn't notice it :-).

Thanks,

Gavin


---------------------------(end of broadcast)---------------------------
TIP 4: Don't 'kill -9' the postmaster

Reply via email to