logo       

[PATCH][RFC] fix 64bit log4cxx_intptr_t: msg#00011

Subject: [PATCH][RFC] fix 64bit log4cxx_intptr_t
Hi there

I took matters into my own hands regarding log4cxx bugs when using it in 
static variables and I have the following patches (the one bellow is the 
first one). This patches are made/tested on the log4cxx version 
distributed from http://littletux.homelinux.org/log4cxx/ (the CVS one 
does not compile for me with GNU autotools).

The following patch removes a warning when compiling on AMD64 because 
log4cxx uses "int" to store pointers when sizeof(int) == 4 and 
sizeof(void*) == 8 on AMD64. It is generally more portable to use long 
(still this is not a perfect solution, a perfect solution involves 
proper autoconf checks and/or usage of <stdint.h> intptr_t type).

--- include/log4cxx/private/log4cxx_private.h.in        (revision 692)
+++ include/log4cxx/private/log4cxx_private.h.in        (revision 694)
@@ -46,7 +46,7 @@
 //
 //   unsigned int same size as void*
 //
-typedef unsigned int log4cxx_intptr_t;
+typedef unsigned long log4cxx_intptr_t;
 
 #define LOG4CXX_LOCALE_ENCODING_UTF8 0
 #define LOG4CXX_LOCALE_ENCODING_ISO_8859_1 0


-- 
Mihai RUSU                                    Email: dizzy@xxxxxxxxx
GPG : http://dizzy.roedu.net/dizzy-gpg.txt    WWW: http://dizzy.roedu.net
                       "Linux is obsolete" -- AST



<Prev in Thread] Current Thread [Next in Thread>
Google Custom Search

Recently Viewed:
science.linguis...    culture.sf.lite...    video.mplayer.c...    yellowdog.gener...    ietf.rfc822/199...    emacs.help/2002...    redhat.release....    kernel.speakup/...    java.openejb.de...    debian.devel.gt...    xfree86.newbie/...    bug-tracking.ma...    pam/2003-05/msg...    games.devel.ope...    user-groups.lin...    music.pancham/2...    network.mq.deve...    web.html.genera...    arklinux.bugs/2...    linux.ecasound/...    qnx.openqnx.dev...    org.user-groups...    file-systems.sf...    trustix.contrib...   
Home | blog view | USPTO Patent Archive | advertise | OSDir is an inevitable website. super tiny logo

Free Magazines

Cisco News
Receive a free quarterly e-newsletter with exclusive articles on how Cisco IT uses its own products and solutions to enable the business.
subscribe

Systems Management News, the newspaper for IT systems administration and data center managers! Each issue of Systems Management News is chock-full of news and analysis to help you understand what's happening in your field.
subscribe

The Enterprise Newsweekly eWeek is the essential technology information source for builders of e-business.
subscribe

Oracle Magazine Oracle Magazine contains technology strategy articles, sample code, tips, Oracle and partner news, how to articles for developers and DBAs, and more. Oracle (NASDAQ: ORCL) is the world's largest enterprise software company.
subscribe

Total Telecom Total Telecom is "The Economist of the communications industry".
subscribe