You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@logging.apache.org by ts...@apache.org on 2019/02/21 18:44:58 UTC

[logging-log4cxx] branch reconsider_logcxx_400 created (now efd5499)

This is an automated email from the ASF dual-hosted git repository.

tschoening pushed a change to branch reconsider_logcxx_400
in repository https://gitbox.apache.org/repos/asf/logging-log4cxx.git.


      at efd5499  Revert LOGCXX-400, to check if "unsigned" woudl be the correct fix instead.

This branch includes the following new commits:

     new efd5499  Revert LOGCXX-400, to check if "unsigned" woudl be the correct fix instead.

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.



[logging-log4cxx] 01/01: Revert LOGCXX-400, to check if "unsigned" woudl be the correct fix instead.

Posted by ts...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

tschoening pushed a commit to branch reconsider_logcxx_400
in repository https://gitbox.apache.org/repos/asf/logging-log4cxx.git

commit efd54990e8feafe64c0825ca90722fddc11eacee
Author: Thorsten Schöning <ts...@am-soft.de>
AuthorDate: Thu Feb 21 19:44:50 2019 +0100

    Revert LOGCXX-400, to check if "unsigned" woudl be the correct fix instead.
---
 src/test/cpp/helpers/transcodertestcase.cpp | 2 +-
 src/test/cpp/xml/domtestcase.cpp            | 8 ++++----
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/test/cpp/helpers/transcodertestcase.cpp b/src/test/cpp/helpers/transcodertestcase.cpp
index cc394f0..6eafc15 100644
--- a/src/test/cpp/helpers/transcodertestcase.cpp
+++ b/src/test/cpp/helpers/transcodertestcase.cpp
@@ -331,7 +331,7 @@ public:
         }
 
         void encodeCharsetName3() {
-            logchar unsupported[] = { 0x1F, 0x7F, static_cast<logchar>(0x80), static_cast<logchar>(0x81), 0x00 };
+            logchar unsupported[] = { 0x1F, 0x7F, 0x80, 0x81, 0x00 };
             std::string encoded(Transcoder::encodeCharsetName(LogString(unsupported)));
             LOGUNIT_ASSERT_EQUAL(std::string("????"), encoded);
         }
diff --git a/src/test/cpp/xml/domtestcase.cpp b/src/test/cpp/xml/domtestcase.cpp
index a1491c7..9b6862e 100644
--- a/src/test/cpp/xml/domtestcase.cpp
+++ b/src/test/cpp/xml/domtestcase.cpp
@@ -186,9 +186,9 @@ public:
 			DOMConfigurator::configure(LOG4CXX_TEST_STR("input/xml/DOMTestCase3.xml"));
 			LOG4CXX_INFO(logger, "File name is expected to end with a superscript 3");
 #if LOG4CXX_LOGCHAR_IS_UTF8
-			const logchar fname[] = { 0x6F, 0x75, 0x74, 0x70, 0x75, 0x74, 0x2F, 0x64, 0x6F, 0x6D, static_cast<logchar>(0xC2), static_cast<logchar>(0xB3), 0 };
+			const logchar fname[] = { 0x6F, 0x75, 0x74, 0x70, 0x75, 0x74, 0x2F, 0x64, 0x6F, 0x6D, 0xC2, 0xB3, 0 };
 #else
-			const logchar fname[] = { 0x6F, 0x75, 0x74, 0x70, 0x75, 0x74, 0x2F, 0x64, 0x6F, 0x6D, static_cast<logchar>(0xB3), 0 };
+			const logchar fname[] = { 0x6F, 0x75, 0x74, 0x70, 0x75, 0x74, 0x2F, 0x64, 0x6F, 0x6D, 0xB3, 0 };
 #endif
 			File file;
 			file.setPath(fname);
@@ -205,9 +205,9 @@ public:
 			DOMConfigurator::configure(LOG4CXX_TEST_STR("input/xml/DOMTestCase4.xml"));
 			LOG4CXX_INFO(logger, "File name is expected to end with an ideographic 4");
 #if LOG4CXX_LOGCHAR_IS_UTF8
-			const logchar fname[] = { 0x6F, 0x75, 0x74, 0x70, 0x75, 0x74, 0x2F, 0x64, 0x6F, 0x6D, static_cast<logchar>(0xE3), static_cast<logchar>(0x86), static_cast<logchar>(0x95), 0 };
+			const logchar fname[] = { 0x6F, 0x75, 0x74, 0x70, 0x75, 0x74, 0x2F, 0x64, 0x6F, 0x6D, 0xE3, 0x86, 0x95, 0 };
 #else
-			const logchar fname[] = { 0x6F, 0x75, 0x74, 0x70, 0x75, 0x74, 0x2F, 0x64, 0x6F, 0x6D, static_cast<logchar>(0x3195), 0 };
+			const logchar fname[] = { 0x6F, 0x75, 0x74, 0x70, 0x75, 0x74, 0x2F, 0x64, 0x6F, 0x6D, 0x3195, 0 };
 #endif
 			File file;
 			file.setPath(fname);