Re: do {} while (0) nitpick
David Steele <david@pgmasters.net>
From: David Steele <david@pgmasters.net>
To: Andrew Dunstan <andrew.dunstan@2ndquadrant.com>,
Tom Lane <tgl@sss.pgh.pa.us>
Cc: Bruce Momjian <bruce@momjian.us>,
John Naylor <john.naylor@2ndquadrant.com>,
PostgreSQL-development <pgsql-hackers@postgresql.org>
Date: 2020-05-06T22:39:01Z
Lists: pgsql-hackers
On 5/6/20 6:28 PM, Andrew Dunstan wrote: > On 5/6/20 3:24 PM, Tom Lane wrote: > >> BTW, I looked around and could not find a package-provided ppport.h >> at all on my Red Hat systems. What package is it in? > > perl-Devel-PPPort contains a perl module that will write the file for > you like this: > > perl -MDevel::PPPort -e 'Devel::PPPort::WriteFile();' FWIW, pgBackRest always shipped with the newest version of ppport.h we were able to generate. This never caused any issues, but neither did we have problems that forced us to upgrade. The documentation seems to encourage this behavior: Don't direct the users of your module to download Devel::PPPort . They are most probably no XS writers. Also, don't make ppport.h optional. Rather, just take the most recent copy of ppport.h that you can find (e.g. by generating it with the latest Devel::PPPort release from CPAN), copy it into your project, adjust your project to use it, and distribute the header along with your module. Regards, -- -David david@pgmasters.net
Commits
-
Get rid of trailing semicolons in C macro definitions.
- ad80d3ea27b1 9.5.22 landed
- a8df87d8638c 9.6.18 landed
- a2098b60216c 11.8 landed
- 49718aa9ee5e 10.13 landed
- c08da32f133b 12.3 landed
- 0da06d9faf9e 13.0 landed