From 8faa774026ade012084caae58ee07d334f37b9e4 Mon Sep 17 00:00:00 2001
From: Alan Conway <aconway@apache.org>
Date: Tue, 25 Mar 2008 13:34:44 +0000
Subject: =?UTF-8?q?Fix=20compile=20errors/warnings=20with=20gcc=204.3=20?=
 =?UTF-8?q?=20-=20added=20missing=20#includes=20that=20were=20implicitly?=
 =?UTF-8?q?=20included=20via=20old=20headers.=20=20-=20add=20namespace-qua?=
 =?UTF-8?q?lifiers=20to=20fix=20"changes=20meaning=20of=20name"=20warnings?=
 =?UTF-8?q?.=20=20-=20./qpid/ptr=5Fmap.h:51:=20fixed=20"qualified=20return?=
 =?UTF-8?q?=20value"=20warning.=20=20-=20use=20const=20char*=20for=20"conv?=
 =?UTF-8?q?ersion=20from=20string=20constant=20to=20=E2=80=98char*?=
 =?UTF-8?q?=E2=80=99"=20warnings?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Applied patch from https://issues.apache.org/jira/browse/QPID-869
remove depenency on boost/date_time, causes warnings with gcc 4.3.


git-svn-id: https://svn.apache.org/repos/asf/incubator/qpid/trunk@640806 13f79535-47bb-0310-9956-ffa450edef68
---
 qpid/cpp/src/tests/logging.cpp | 28 +++++++++++-----------------
 1 file changed, 11 insertions(+), 17 deletions(-)

(limited to 'qpid/cpp/src/tests/logging.cpp')

diff --git a/qpid/cpp/src/tests/logging.cpp b/qpid/cpp/src/tests/logging.cpp
index 2c0ed08105..201c935319 100644
--- a/qpid/cpp/src/tests/logging.cpp
+++ b/qpid/cpp/src/tests/logging.cpp
@@ -155,12 +155,6 @@ BOOST_AUTO_TEST_CASE(testLoggerFormat) {
     l.select(Selector(critical));
     TestOutput* out=new TestOutput(l);
 
-    // Time format is YYY-Month-dd hh:mm:ss
-    l.format(Logger::TIME);
-    QPID_LOG(critical, "foo");
-    string re("\\d\\d\\d\\d-[A-Z][a-z]+-\\d\\d \\d\\d:\\d\\d:\\d\\d foo\n");
-    BOOST_CHECK_REGEX(re, out->last());
-
     l.format(Logger::FILE);
     QPID_LOG(critical, "foo");
     BOOST_CHECK_EQUAL(out->last(), string(__FILE__)+": foo\n");
@@ -178,7 +172,7 @@ BOOST_AUTO_TEST_CASE(testLoggerFormat) {
 
     l.format(~0);               // Everything
     QPID_LOG(critical, "foo");
-    re=".* critical \\[[0-9a-f]*] "+string(__FILE__)+":\\d+:void .*testLoggerFormat.*\\(\\): foo\n";
+    string re=".* critical \\[[0-9a-f]*] "+string(__FILE__)+":\\d+:void .*testLoggerFormat.*\\(\\): foo\n";
     BOOST_CHECK_REGEX(re, out->last());
 }
 
@@ -259,7 +253,7 @@ Statement statement(
 #define ARGC(argv) (sizeof(argv)/sizeof(char*))
 
 BOOST_AUTO_TEST_CASE(testOptionsParse) {
-    char* argv[]={
+    const char* argv[]={
         0,
         "--log-enable", "error+:foo",
         "--log-enable", "debug:bar",
@@ -272,7 +266,7 @@ BOOST_AUTO_TEST_CASE(testOptionsParse) {
         "--log-function", "YES"
     };
     qpid::log::Options opts;
-    opts.parse(ARGC(argv), argv);
+    opts.parse(ARGC(argv), const_cast<char**>(argv));
     vector<string> expect=list_of("error+:foo")("debug:bar")("info");
     BOOST_CHECK_EQUAL(expect, opts.selectors);
     expect=list_of("x")("y");
@@ -294,14 +288,14 @@ BOOST_AUTO_TEST_CASE(testOptionsDefault) {
 }
 
 BOOST_AUTO_TEST_CASE(testSelectorFromOptions) {
-    char* argv[]={
+    const char* argv[]={
         0,
         "--log-enable", "error+:foo",
         "--log-enable", "debug:bar",
         "--log-enable", "info"
     };
     qpid::log::Options opts;
-    opts.parse(ARGC(argv), argv);
+    opts.parse(ARGC(argv), const_cast<char**>(argv));
     vector<string> expect=list_of("error+:foo")("debug:bar")("info");
     BOOST_CHECK_EQUAL(expect, opts.selectors);
     Selector s(opts);
@@ -317,27 +311,27 @@ BOOST_AUTO_TEST_CASE(testOptionsFormat) {
     {
         Options opts;
         BOOST_CHECK_EQUAL(Logger::TIME|Logger::LEVEL, l.format(opts));
-        char* argv[]={
+        const char* argv[]={
             0,
             "--log-time", "no", 
             "--log-level", "no",
             "--log-source", "1",
             "--log-thread",  "1"
         };
-        opts.parse(ARGC(argv), argv);
+        opts.parse(ARGC(argv), const_cast<char**>(argv));
         BOOST_CHECK_EQUAL(
             Logger::FILE|Logger::LINE|Logger::THREAD, l.format(opts));
     }
     {
         Options opts;           // Clear.
-        char* argv[]={
+        const char* argv[]={
             0,
             "--log-level", "no",
             "--log-thread", "true",
             "--log-function", "YES",
             "--log-time", "YES"
         };
-        opts.parse(ARGC(argv), argv);
+        opts.parse(ARGC(argv), const_cast<char**>(argv));
         BOOST_CHECK_EQUAL(
             Logger::THREAD|Logger::FUNCTION|Logger::TIME,
             l.format(opts));
@@ -348,14 +342,14 @@ BOOST_AUTO_TEST_CASE(testLoggerConfigure) {
     Logger& l=Logger::instance();
     l.clear();
     Options opts;
-    char* argv[]={
+    const char* argv[]={
         0,
         "--log-time", "no", 
         "--log-source", "yes",
         "--log-output", "logging.tmp",
         "--log-enable", "critical"
     };
-    opts.parse(ARGC(argv), argv);
+    opts.parse(ARGC(argv), const_cast<char**>(argv));
     l.configure(opts, "test");
     QPID_LOG(critical, "foo"); int srcline=__LINE__;
     ifstream log("logging.tmp");
-- 
cgit v1.2.1