Re: DTrace probe patch for OS X Leopard

Поиск
Список
Период
Сортировка
От Robert Lor
Тема Re: DTrace probe patch for OS X Leopard
Дата
Msg-id 47C86ACA.5000201@sun.com
обсуждение исходный текст
Ответ на Re: DTrace probe patch for OS X Leopard  (Tom Lane <tgl@sss.pgh.pa.us>)
Ответы Re: DTrace probe patch for OS X Leopard  (Tom Lane <tgl@sss.pgh.pa.us>)
Re: DTrace probe patch for OS X Leopard  (Alvaro Herrera <alvherre@commandprompt.com>)
Список pgsql-patches
Tom Lane wrote:
> We still have what I consider a big problem with the names of the
> macros.  Perhaps that could be fixed by passing the auto-generated
> file through a sed script to put a prefix on the macro names before
> we start to use it?
>

Post processing the auto generated header may work, but I think it could
be unnecessarily complicated and error proned. First, the formats of the
header between Mac OS X and Solaris are different, and I'm pretty sure
it'll be different for FreeBSD when it comes out with Dtrace support.
Second, if there are changes in the  content of the header in the
future, the sed script may break. I can investigate this approach
further, but I personally prefer a solution this is less error proned.

And don't think adding a simple comment before the macro call is
sufficient? This can be documented so everyone knows the convention.

Regards,
-Robert

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

Предыдущее
От: Alvaro Herrera
Дата:
Сообщение: sinval.c / sinvaladt.c restructuring
Следующее
От: Robert Lor
Дата:
Сообщение: Re: DTrace probe patch for OS X Leopard