Re: compile warnings in CVS - Mailing list pgsql-hackers

From Bruce Momjian
Subject Re: compile warnings in CVS
Date
Msg-id 200208180006.g7I06IX14006@candle.pha.pa.us
Whole thread Raw
In response to Re: compile warnings in CVS  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
Yes, very nanny-ish.  Not sure how to turn it off.

---------------------------------------------------------------------------

Tom Lane wrote:
> Neil Conway <nconway@klamath.dyndns.org> writes:
> > pg_controldata.c: In function `main':
> > pg_controldata.c:91: warning: `%c' yields only last 2 digits of year in some locales
> > pg_controldata.c:93: warning: `%c' yields only last 2 digits of year in some locales
> 
> Yeah.  I was willing to ignore that while pg_controldata was in contrib,
> but it's much more annoying when it's in the main tree.  Anyone know if
> gcc has a --not-quite-so-nannyish warnings mode?
> 
> IMHO %c is a perfectly reasonable format choice --- the strftime man
> page defines it as
>           %c        Locale's appropriate date and time representation.
> While we could go over to some %Y-%M-etc-etc notation, that doesn't
> strike me as a step forward.  pg_controldata's output should be
> conveniently human-readable IMHO, and that means following local
> conventions.
> 
> Another alternative is
>     char *fmt = "%c";
>     ...
>     strftime(..., fmt, ...);
> 
> which I think will probably defeat gcc's check (haven't tried it
> though).
> 
> Does anyone want to argue that %c is actually a bad choice?  I think
> gcc's just being unreasonable here, but maybe I'm missing something
> (and no, Y2K arguments won't change my mind).
> 
>             regards, tom lane
> 
> ---------------------------(end of broadcast)---------------------------
> TIP 3: if posting/reading through Usenet, please send an appropriate
> subscribe-nomail command to majordomo@postgresql.org so that your
> message can get through to the mailing list cleanly
> 

--  Bruce Momjian                        |  http://candle.pha.pa.us pgman@candle.pha.pa.us               |  (610)
359-1001+  If your life is a hard drive,     |  13 Roberts Road +  Christ can be your backup.        |  Newtown Square,
Pennsylvania19073
 


pgsql-hackers by date:

Previous
From: Bruce Momjian
Date:
Subject: Re: compile warnings in CVS
Next
From: Rod Taylor
Date:
Subject: cvs-tip broken