On Jan 11, 2008 5:04 PM, Ruediger Pluem <rpluem@apache.org> wrote:
>
>
> On 01/11/2008 10:32 PM, Eric Covener wrote:
> > - id = apr_psprintf(r->pool, "%x:%lx:%x", getpid(), time(NULL),
> > + id = apr_psprintf(r->pool, "%" APR_PID_T_FMT "x:%lx:%x", getpid(),
> >
> > Stray 'x' in the new format string? No harm AFAICT.
>
> On purpose. Before we had hex aoutput of the pid (don't know why) and there
> is no APR_PID_T_FMT_HEX- So I concat APR_PID_T_FMT and x to get a hex output
> I people thing that a hex output of a pid is stupid, well then lets remove the
> x, but the initial intention of the commit was to remove compiler warning no
> changing code.
For APR_PID_T_FMT of 'd' (my vanilla 32-bit linux system) the 'x' ends
up in the output as in:
+26737x:4787e90e:0|GET /
HTTP/1.0|User-Agent:Wget/1.10.2|Accept:*/*|Host:localhost|Connection:Keep-Alive
-26737x:4787e90e:0
--
Eric Covener
covener@gmail.com
|