You are viewing a plain text version of this content. The canonical link for it is here.
Posted to log4cxx-dev@logging.apache.org by ca...@apache.org on 2005/11/03 16:25:17 UTC

svn commit: r330566 - /logging/log4cxx/trunk/src/loggingevent.cpp

Author: carnold
Date: Thu Nov  3 07:25:15 2005
New Revision: 330566

URL: http://svn.apache.org/viewcvs?rev=330566&view=rev
Log:
LOGCXX-110: So that is what that unnecessary cast was for

Modified:
    logging/log4cxx/trunk/src/loggingevent.cpp

Modified: logging/log4cxx/trunk/src/loggingevent.cpp
URL: http://svn.apache.org/viewcvs/logging/log4cxx/trunk/src/loggingevent.cpp?rev=330566&r1=330565&r2=330566&view=diff
==============================================================================
--- logging/log4cxx/trunk/src/loggingevent.cpp (original)
+++ logging/log4cxx/trunk/src/loggingevent.cpp Thu Nov  3 07:25:15 2005
@@ -203,7 +203,7 @@
    result[1] = 'x';
    apr_snprintf(result+2, (sizeof result) - 2, "%pt", &threadId);
 
-   LOG4CXX_DECODE_CHAR(str, result);
+   LOG4CXX_DECODE_CHAR(str, (const char*) result);
    return str;
 #else
    return LOG4CXX_STR("0x00000000");



Re: svn commit: r330566 - /logging/log4cxx/trunk/src/loggingevent.cpp

Posted by Andreas Fester <An...@gmx.de>.
carnold@apache.org wrote:
> LOGCXX-110: So that is what that unnecessary cast was for

:D

[...]
> -   LOG4CXX_DECODE_CHAR(str, result);
> +   LOG4CXX_DECODE_CHAR(str, (const char*) result);
[...]

Indeed, without this cast the compiler can not determine the
correct function overload....

-- 
Andreas Fester
mailto:andreas@littletux.net
WWW: http://www.littletux.net
ICQ: 326674288