On Mon, Jan 16, 2012, at 23:06, Bert Huijben wrote: > > > > -----Original Message----- > > From: danie...@apache.org [mailto:danie...@apache.org] > > Sent: maandag 16 januari 2012 23:04 > > To: comm...@subversion.apache.org > > Subject: svn commit: r1232189 - /subversion/trunk/contrib/server- > > side/mod_setlocale/mod_setlocale.c > > > > Author: danielsh > > Date: Mon Jan 16 22:03:41 2012 > > New Revision: 1232189 > > > > URL: http://svn.apache.org/viewvc?rev=1232189&view=rev > > Log: > > Remove a debugging scaffolding. > > > > * contrib/server-side/mod_setlocale/mod_setlocale.c > > (setlocale_create_server_config): Reduce the severity of a logged message > > to APLOG_DEBUG. > > > > Modified: > > subversion/trunk/contrib/server-side/mod_setlocale/mod_setlocale.c > > > > Modified: subversion/trunk/contrib/server- > > side/mod_setlocale/mod_setlocale.c > > URL: http://svn.apache.org/viewvc/subversion/trunk/contrib/server- > > side/mod_setlocale/mod_setlocale.c?rev=1232189&r1=1232188&r2=1232189 > > &view=diff > > ========================================================== > > ==================== > > --- subversion/trunk/contrib/server-side/mod_setlocale/mod_setlocale.c > > (original) > > +++ subversion/trunk/contrib/server-side/mod_setlocale/mod_setlocale.c > > Mon Jan 16 22:03:41 2012 > > @@ -115,7 +115,7 @@ static void * > > setlocale_create_server_config(apr_pool_t *p, server_rec *s) > > { > > setlocale_config_rec *cfg = apr_pcalloc(p, sizeof(*cfg)); > > - ap_log_error(APLOG_MARK, APLOG_EMERG, 0, s, > > + ap_log_error(APLOG_MARK, APLOG_DEBUG, 0, s, > > "create: 0x%08x", cfg); > > Doesn't this assume a 32 bit machine? > > Is %p for a pointer portable enough? >
Pretty sure that I tried %p and got an error. I suppose there is an APR_SOMETHING_T_FMT macro this should use? Or else it'll obviously need a cast (so the format string matches the arguments). > Bert > > > return cfg; > > } > > > > >
