Re: Significant oversight in that #include-removal script

From: Martijn van Oosterhout <kleptog(at)svana(dot)org>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Alvaro Herrera <alvherre(at)commandprompt(dot)com>, Bruce Momjian <bruce(at)momjian(dot)us>, pgsql-hackers(at)postgreSQL(dot)org
Subject: Re: Significant oversight in that #include-removal script
Date: 2009-01-08 20:33:43
Message-ID: 20090108203343.GA31566@svana.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Wed, Jan 07, 2009 at 05:49:12PM -0500, Tom Lane wrote:
> > It would be easy if the compiler were to have an option to throw a
> > warning when it finds a non-static variable that doesn't have a
> > corresponding extern declaration.
>
> Yeah, I think this is hopeless (or at least not worth the cost) without
> compiler support --- I was just idly wondering if newer gcc's might have
> such an option.

GNU ld has a --warn-common option which activates some warnings
relating to this. It won't check the type, but it will check the size.

It does note that this might produce spurious warnings due to
sloppiness in libraries, but it might help.

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

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Stephen Frost 2009-01-08 20:34:51 Re: New patch for Column-level privileges
Previous Message Heikki Linnakangas 2009-01-08 20:31:44 Re: Hot standby, slot ids and stuff