From: Jelte Jansen <jelte@isc.org>
Date: Mon, 4 Mar 2013 10:02:31 +0000 (+0100)
Subject: [2746] Clear duplicated vector at global initialization
X-Git-Tag: bind10-1.1.0beta1-release~53^2~1^2~5
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=c1233a28b0dfd3f287897a17542a3874f5401ddb;p=thirdparty%2Fkea.git

[2746] Clear duplicated vector at global initialization
---

diff --git a/src/lib/log/logger_manager.cc b/src/lib/log/logger_manager.cc
index 77893d079b..8d0978a2ba 100644
--- a/src/lib/log/logger_manager.cc
+++ b/src/lib/log/logger_manager.cc
@@ -133,7 +133,7 @@ LoggerManager::init(const std::string& root, isc::log::Severity severity,
              i != duplicates.end(); ++i) {
             LOG_WARN(logger, LOG_DUPLICATE_MESSAGE_ID).arg(*i);
         }
-
+        MessageInitializer::clearDuplicates();
     }
 
     // Replace any messages with local ones (if given)
diff --git a/src/lib/log/tests/message_dictionary_unittest.cc b/src/lib/log/tests/message_dictionary_unittest.cc
index b8bded364c..848ccb3213 100644
--- a/src/lib/log/tests/message_dictionary_unittest.cc
+++ b/src/lib/log/tests/message_dictionary_unittest.cc
@@ -189,9 +189,9 @@ TEST_F(MessageDictionaryTest, GlobalTest) {
 // new symbol.
 
 TEST_F(MessageDictionaryTest, GlobalLoadTest) {
+    // There were duplicates but the vector should be cleared in init() now
     vector<string>& duplicates = MessageInitializer::getDuplicates();
-    ASSERT_EQ(1, duplicates.size());
-    EXPECT_EQ(string("LOG_DUPLICATE_NAMESPACE"), duplicates[0]);
+    ASSERT_EQ(0, duplicates.size());
 
     string text = MessageDictionary::globalDictionary().getText("NEWSYM");
     EXPECT_EQ(string("new symbol added"), text);