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. Regards Rüdiger
