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

Поиск
Список
Период
Сортировка
От Martijn van Oosterhout
Тема Re: Updates of SE-PostgreSQL 8.4devel patches (r1403)
Дата
Msg-id 20090114073459.GB19315@svana.org
обсуждение исходный текст
Ответ на Re: Updates of SE-PostgreSQL 8.4devel patches (r1403)  (Alvaro Herrera <alvherre@commandprompt.com>)
Ответы Re: Updates of SE-PostgreSQL 8.4devel patches (r1403)  (KaiGai Kohei <kaigai@ak.jp.nec.com>)
Re: Updates of SE-PostgreSQL 8.4devel patches (r1403)  (Alvaro Herrera <alvherre@commandprompt.com>)
Re: Updates of SE-PostgreSQL 8.4devel patches (r1403)  (Tom Lane <tgl@sss.pgh.pa.us>)
Список pgsql-hackers
On Tue, Jan 13, 2009 at 10:05:45AM -0300, Alvaro Herrera wrote:
> pgace.h: you have a bunch of "static inline" functions in here.  As far
> as I know this doesn't work in compilers other than GCC :-(  See
> pg_list.h (list_head) for an example.  I think we can tolerate this for
> the three functions in pg_list.h because they are so few and so tiny,
> but I'm not sure about PGACE because they are a large lot.  On the other
> hand, turning them to real functions would be a performance hit.

Really? C99 requires it and MSVC does support it. At least the other
compilers whose name I remembered (HP, Sun) support it also. I'd be
surprised if a compiler didn't since it's the form of inline that most
matches what people expect to happen.

Do you have an example?

http://www.greenend.org.uk/rjk/2003/03/inline.html
http://hi.baidu.com/junru/blog/item/4d8db11339050c856438db7a.html

Have a nice day,
--
Martijn van Oosterhout   <kleptog@svana.org>   http://svana.org/kleptog/
> Please line up in a tree and maintain the heap invariant while
> boarding. Thank you for flying nlogn airlines.

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

Предыдущее
От: KaiGai Kohei
Дата:
Сообщение: Updates of SE-PostgreSQL 8.4devel patches (r1408)
Следующее
От: KaiGai Kohei
Дата:
Сообщение: Re: Updates of SE-PostgreSQL 8.4devel patches (r1403)