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

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

Thread Previous | Thread Next
From:
Jarkko Hietaniemi
Date:
August 1, 2001 07:00
Subject:
Re: [PATCH 3/4] -Wall cleanups: config.h (yes, config.h)
Message ID:
20010801090040.G4053@chaos.wustl.edu
On Tue, Jul 31, 2001 at 11:16:37PM -0700, Richard Soderberg wrote:
> This is the third in a series of patches designed to remove warnings from
> the compilation of Perl on FreeBSD-CURRENT, last updated Jun 19, 2001.
> 
> Applying this patch removes a compilation warning from taint.c:18 - changing
> the below define stops the warning.
> 
> All tests passed.
> 
> Note - I know this file is generated, and in fact I had to fudge timestamps
> to keep it from regenerating.  I'd like to see the proper patch for this, as
> I got lost in a maze of twisty Configure mazes, all alike.
> 
> R.
> 
> --- 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?

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

-- 
$jhi++; # http://www.iki.fi/jhi/
        # There is this special biologist word we use for 'stable'.
        # It is 'dead'. -- Jack Cohen

Thread Previous | Thread Next


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