Re: [HACKERS] geo_ops.c is broken

Поиск
Список
Период
Сортировка
От Bruce Momjian
Тема Re: [HACKERS] geo_ops.c is broken
Дата
Msg-id 199808232215.SAA25087@candle.pha.pa.us
обсуждение исходный текст
Ответ на geo_ops.c is broken  (Tom Lane <tgl@sss.pgh.pa.us>)
Список 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@candle.pha.pa.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)

В списке pgsql-hackers по дате отправления:

Предыдущее
От: Bruce Momjian
Дата:
Сообщение: Re: [HACKERS] Regression test status (was type coersion)
Следующее
От: Bruce Momjian
Дата:
Сообщение: Re: [HACKERS] Regression test status (was type coersion)