From: | Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> |
---|---|
To: | sugita(at)sra(dot)co(dot)jp |
Cc: | pgsql-patches(at)postgresql(dot)org |
Subject: | Re: Make factorial(0::int2) return 1, as per spec. |
Date: | 2002-06-07 23:42:16 |
Message-ID: | 200206072342.g57NgGR07100@candle.pha.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-patches |
Thanks. Patch applied. Seems I fixed the int4 case but missed int2.
---------------------------------------------------------------------------
sugita(at)sra(dot)co(dot)jp wrote:
> Attached is a patch to make factorial(0::int2) return 1.
>
> Index: backend/utils/adt/int.c
> ===================================================================
> RCS file: /projects/cvsroot/pgsql/src/backend/utils/adt/int.c,v
> retrieving revision 1.48
> diff -u -3 -p -r1.48 int.c
> --- backend/utils/adt/int.c 2002/02/23 01:01:30 1.48
> +++ backend/utils/adt/int.c 2002/05/29 15:56:11
> @@ -800,7 +800,9 @@ int2fac(PG_FUNCTION_ARGS)
> int16 arg1 = PG_GETARG_INT16(0);
> int32 result;
>
> - if (arg1 < 1)
> + if (arg1 == 0)
> + result = 1;
> + else if (arg1 < 1)
> result = 0;
> else
> for (result = 1; arg1 > 0; --arg1)
>
> ---------------------------(end of broadcast)---------------------------
> TIP 1: subscribe and unsubscribe commands go to majordomo(at)postgresql(dot)org
--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 853-3000
+ If your life is a hard drive, | 830 Blythe Avenue
+ Christ can be your backup. | Drexel Hill, Pennsylvania 19026
From | Date | Subject | |
---|---|---|---|
Next Message | Bruce Momjian | 2002-06-07 23:43:24 | Re: Make factorial(0::int2) return 1, as per spec. |
Previous Message | Joe Conway | 2002-06-07 23:11:20 | Re: [HACKERS] revised sample SRF C function; proposed SRF API |