From 18e02a5b6da498a3a6880ee43a77df841ca57079 Mon Sep 17 00:00:00 2001 From: Andrew Stitcher Date: Tue, 28 Jul 2009 03:23:49 +0000 Subject: Add a default constructor to LinkRegistry so that you don't need a broker to create one for the store tests git-svn-id: https://svn.apache.org/repos/asf/qpid/trunk/qpid@798383 13f79535-47bb-0310-9956-ffa450edef68 --- cpp/src/qpid/broker/LinkRegistry.cpp | 17 +++++++++++++++-- cpp/src/qpid/broker/LinkRegistry.h | 1 + 2 files changed, 16 insertions(+), 2 deletions(-) (limited to 'cpp/src') diff --git a/cpp/src/qpid/broker/LinkRegistry.cpp b/cpp/src/qpid/broker/LinkRegistry.cpp index fce3a88cd1..10e5359493 100644 --- a/cpp/src/qpid/broker/LinkRegistry.cpp +++ b/cpp/src/qpid/broker/LinkRegistry.cpp @@ -36,8 +36,21 @@ namespace _qmf = qmf::org::apache::qpid::broker; #define LINK_MAINT_INTERVAL 2 -LinkRegistry::LinkRegistry (Broker* _broker) : broker(_broker), parent(0), store(0), passive(false), passiveChanged(false), - realm(broker ? broker->getOptions().realm : "") +// TODO: This constructor is only used by the store unit tests - +// That probably indicates that LinkRegistry isn't correctly +// factored: The persistence element and maintenance element +// should be factored separately +LinkRegistry::LinkRegistry () : + broker(0), + parent(0), store(0), passive(false), passiveChanged(false), + realm("") +{ +} + +LinkRegistry::LinkRegistry (Broker* _broker) : + broker(_broker), + parent(0), store(0), passive(false), passiveChanged(false), + realm("") { timer.add (intrusive_ptr (new Periodic(*this))); } diff --git a/cpp/src/qpid/broker/LinkRegistry.h b/cpp/src/qpid/broker/LinkRegistry.h index bfdf0d491b..8d1a252f54 100644 --- a/cpp/src/qpid/broker/LinkRegistry.h +++ b/cpp/src/qpid/broker/LinkRegistry.h @@ -75,6 +75,7 @@ namespace broker { static std::string createKey(const TcpAddress& address); public: + LinkRegistry (); // Only used in store tests LinkRegistry (Broker* _broker); std::pair, bool> declare(std::string& host, -- cgit v1.2.1