Refactored the QMF engine to adhere to the following rules regarding

the pimpl (Pointer to Implementation) pattern:

1) Impl classes have constructors matching the public constructors
2) Additional Impl constructors are accessed through a static factory function
3) All linkages to objects are to the public object
4) If a back-link (from Impl to public) is needed, the Impl class must be
   derived from boost::noncopyable
5) All public classes have non-default copy constructors that make a copy of the
   Impl class



git-svn-id: https://svn.apache.org/repos/asf/qpid/trunk@816770 13f79535-47bb-0310-9956-ffa450edef68

1 files changed, 2 insertions, 1 deletions

diff --git a/qpid/cpp/src/qmf/ResilientConnection.cpp b/qpid/cpp/src/qmf/ResilientConnection.cpp
index 7ec03cf4da..be7837b829 100644
--- a/qpid/cpp/src/qmf/ResilientConnection.cpp
+++ b/qpid/cpp/src/qmf/ResilientConnection.cpp
@@ -38,6 +38,7 @@
 #include <vector>
 #include <set>
 #include <boost/intrusive_ptr.hpp>
+#include <boost/noncopyable.hpp>
 
 using namespace std;
 using namespace qmf;
@@ -77,7 +78,7 @@ namespace qmf {
         void stop();
     };
 
-    class ResilientConnectionImpl : public qpid::sys::Runnable {
+    class ResilientConnectionImpl : public qpid::sys::Runnable, public boost::noncopyable {
     public:
         ResilientConnectionImpl(const ConnectionSettings& settings);
         ~ResilientConnectionImpl();