From: | Bruce Momjian <maillist(at)candle(dot)pha(dot)pa(dot)us> |
---|---|
To: | tgl(at)sss(dot)pgh(dot)pa(dot)us (Tom Lane) |
Cc: | pgsql-hackers(at)postgreSQL(dot)org |
Subject: | Re: [HACKERS] geo_ops.c is broken |
Date: | 1998-08-23 22:15:21 |
Message-ID: | 199808232215.SAA25087@candle.pha.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Looks like they are gone now.
> The recently checked-in backend/utils/adt/geo_ops.c does not compile
> under gcc 2.7.2.2, because it has
>
> #define LINEDEBUG
>
> followed by several instances of
>
> #if LINEDEBUG
>
> ... in other words #if with an empty expression, which I think gcc
> has every right to object to. Either the #if's ought to be #ifdef's
> or LINEDEBUG has to be #define'd as 1 rather than empty.
>
> I changed the #if's to #ifdef's and found that it *still* doesn't
> compile, because the code within the if's is seriously broken ---
> it's got lines like
> result->C = pt1->x;
> where result is a char * and pt1 isn't even declared.
>
> Backing out the #define LINEDEBUG makes it compile, but who checked
> this stuff in anyway?
>
> grumble, tom lane
>
>
--
Bruce Momjian | 830 Blythe Avenue
maillist(at)candle(dot)pha(dot)pa(dot)us | Drexel Hill, Pennsylvania 19026
+ If your life is a hard drive, | (610) 353-9879(w)
+ Christ can be your backup. | (610) 853-3000(h)
From | Date | Subject | |
---|---|---|---|
Next Message | Bruce Momjian | 1998-08-23 22:16:12 | Re: [HACKERS] Regression test status (was type coersion) |
Previous Message | Bruce Momjian | 1998-08-23 22:13:04 | Re: [HACKERS] Problem with parser |