On Sat, Oct 29, 2005 at 01:04:04PM +0200, Andreas Fester wrote: > -----BEGIN PGP SIGNED MESSAGE----- > Hash: SHA1 > > Hi, > > I have investigated a more portable way of getting rid > of the warning on AMD64 regarding the log4cxx_intptr_t type. > This type is only used in StringHelper::formatHex(), > which iteself is again only used to convert the thread > id returned by apr_os_thread_current(). > The APR itself already provides a portable way of converting > an apr_os_thread_id to a string by passing a special format > specifier to apr_snprintf(). Thus, my solution (patch attached) > is to use this portable method and to completely remove the > not-so-portable log4cxx_intptr_t type. > Drawback(?): this specific format specifier for apr_os_thread_id > was introduced as new feature with APR 1.2, which would mean > that APR 1.2 will be mandatory (not a big issue in my opinion) > starting with this patch. > Any comments?
Hi Andreas I have none, sounds like a better solution from my point of view (hope that I don't have to patch APR 1.2 too with the bugs I reported on this list some time ago). Mihai RUSU Email: [EMAIL PROTECTED] GPG : http://dizzy.roedu.net/dizzy-gpg.txt WWW: http://dizzy.roedu.net "Linux is obsolete" -- AST
