Re: Updates of SE-PostgreSQL 8.4devel patches (r1389)

Поиск
Список
Период
Сортировка
От Tom Lane
Тема Re: Updates of SE-PostgreSQL 8.4devel patches (r1389)
Дата
Msg-id 23722.1231336498@sss.pgh.pa.us
обсуждение исходный текст
Ответ на Re: Updates of SE-PostgreSQL 8.4devel patches (r1389)  (Alvaro Herrera <alvherre@commandprompt.com>)
Ответы Re: Updates of SE-PostgreSQL 8.4devel patches (r1389)  (Alvaro Herrera <alvherre@commandprompt.com>)
Список pgsql-hackers
Alvaro Herrera <alvherre@commandprompt.com> writes:
> Oh, the patch also removes a bunch of "continue" statements that, as far
> as I can tell, no longer work after the macros were wrapped in
> do { ... } while (0) :-(  I don't see any nice way to put the facility
> back.

Hmm ... I guess you could make the wrapping be "if (...) { ... } else {}"
instead of do/while, but I'm pretty dubious of having a continue in the
macros anyway --- that's an even stronger assumption about the context
the macro is being used in than the original gripe.

What you seem to be supposing is that the only possible use pattern
for these macros is a for-loop containing nothing but calls to one
or another of the macros.  If so, shouldn't you be wrapping things
up at an even higher level?  Maybe turn the whole thing into
table-driven code?
        regards, tom lane


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

Предыдущее
От: Andrew Dunstan
Дата:
Сообщение: Re: HAVE_FSEEKO for WIN32
Следующее
От: Magnus Hagander
Дата:
Сообщение: Re: [BUGS] BUG #4186: set lc_messages does not work