apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jeff Trawick <trawi...@bellsouth.net>
Subject Re: cvs commit: apr/test test_apr.h testmd5.c testproc.c
Date Mon, 14 May 2001 00:15:54 GMT
"William A. Rowe, Jr." <lists@rowe-clan.net> writes:

> > trawick     01/05/13 08:34:18
> > 
> >   Modified:    test     test_apr.h testmd5.c testproc.c
> >   Log:
> >   fix some bad parms to printf/fprintf
> >   
> >   -            printf("Error was %ld : %s\n", rv, strerror(rv)); \
> >   +            printf("Error was %d : %s\n", rv, strerror(rv)); \
> 
> Do I smell a need for an APR_STATUS_T_FMT ?  It doesn't seem we can assume
> this on 64 bit machines (?).

apr_status_t is always int so %d is always correct (though I suppose
the code in apr_errno.h could change).

I wouldn't mind seeing APR_STATUS_T_FMT though... Why should the user
stop to wonder which APR types are always the same and which are not?

-- 
Jeff Trawick | trawickj@bellsouth.net | PGP public key at web site:
       http://www.geocities.com/SiliconValley/Park/9289/
             Born in Roswell... married an alien...


Mime
View raw message