Re: DTrace probe patch for OS X Leopard - Mailing list pgsql-patches

From Robert Lor
Subject Re: DTrace probe patch for OS X Leopard
Date
Msg-id 47C86ACA.5000201@sun.com
Whole thread Raw
In response to Re: DTrace probe patch for OS X Leopard  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: DTrace probe patch for OS X Leopard
Re: DTrace probe patch for OS X Leopard
List 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 by date:

Previous
From: Alvaro Herrera
Date:
Subject: sinval.c / sinvaladt.c restructuring
Next
From: Robert Lor
Date:
Subject: Re: DTrace probe patch for OS X Leopard