develooper Front page | perl.perl5.porters | Postings from August 2001

RE: [PATCH 3/4] -Wall cleanups: config.h (yes, config.h)

Thread Previous
From:
Richard Soderberg
Date:
August 2, 2001 01:02
Subject:
RE: [PATCH 3/4] -Wall cleanups: config.h (yes, config.h)
Message ID:
NAEKLNAAHLMBPMPNBMLEKEPJDJAA.rs@oregonnet.com
From: Jarkko Hietaniemi [mailto:jhi@iki.fi]
Subject: Re: [PATCH 3/4] -Wall cleanups: config.h (yes, config.h)
>
> On Tue, Jul 31, 2001 at 11:16:37PM -0700, Richard Soderberg wrote:
> > Applying this patch removes a compilation warning from
> taint.c:18 - changing
> > the below define stops the warning.
> >
> > --- config.h~	Sat Jul 28 23:28:58 2001
> > +++ config.h	Sat Jul 28 23:28:25 2001
> > @@ -3105,7 +3105,7 @@
> >  /* Uid_t_f:
> >   *	This symbol defines the format string used for printing a Uid_t.
> >   */
> > -#define	Uid_t_f		"lu"		/**/
> > +#define	Uid_t_f		"ul"		/**/
>
> "%ul"?  I'm sorry but that is not s?printf.  Well, it is, but only the
> "%u" part.  The "l" will be a literal following the formatted number.
>
> So... it seems that "u" would be the correct answer.  What are your
> uidsize, uvsize, and uvuformat from config.sh?

This same issue appears on RedHat linux as well.

Yup, %u in Uid_t_f cures the warning everywhere.

From FreeBSD-5.0-CURRENT and RedHat 6.2:

uidsize='4'
uvsize='4'
uvuformat='lu'

--- config.h~	Thu Aug  2 00:12:06 2001
+++ config.h	Thu Aug  2 00:55:12 2001
@@ -3105,7 +3105,7 @@
 /* Uid_t_f:
  *	This symbol defines the format string used for printing a Uid_t.
  */
-#define	Uid_t_f		"lu"		/**/
+#define	Uid_t_f		"u"		/**/

 /* Uid_t_sign:
  *	This symbol holds the signedess of a Uid_t.

R.


Thread Previous


nntp.perl.org: Perl Programming lists via nntp and http.
Comments to Ask Bjørn Hansen at ask@perl.org | Group listing | About