* cpp/lib/client/ClientMessage.h/.cpp: Added Message constructor taking data.

* cpp/lib/client/IncomingMessage.cpp: Simplified message aggregation.
* cpp/lib/common/framing/AMQ*Body.h: remove unnecessary virtual inheritance.
* cpp/lib/common/framing/AMQMethodBody.h: add bool isRequest, isResponse
* cpp/lib/common/sys/Mutex.h (class ScopedUnlock): Added ScopedUnlock
* cpp/lib/common/sys/ThreadSafeQueue.h: Thread-safe queue template.
* cpp/tests/BrokerChannelTest.cpp: Renamed from ChannelTest.
* cpp/tests/ClientChannelTest.cpp: Test client API.
* cpp/tests/RequestResponseTest.cpp: Removed empty test.
* cpp/lib/client/Connection.h,.cpp:
 - non-static channelIdCounter
 - No close() in dtor.


git-svn-id: https://svn.apache.org/repos/asf/incubator/qpid/branches/qpid.0-9@508705 13f79535-47bb-0310-9956-ffa450edef68

2 files changed, 110 insertions, 0 deletions

diff --git a/cpp/lib/common/sys/Mutex.h b/cpp/lib/common/sys/Mutex.h
index 4022da2f6e..87d537fb9e 100644
--- a/cpp/lib/common/sys/Mutex.h
+++ b/cpp/lib/common/sys/Mutex.h
@@ -46,12 +46,23 @@ class ScopedLock
     L& mutex;
 };
 
+template <class L>
+class ScopedUnlock
+{
+  public:
+    ScopedUnlock(L& l) : mutex(l) { l.unlock(); }
+    ~ScopedUnlock() { mutex.lock(); }
+  private:
+    L& mutex;
+};
+    
 /**
  * Mutex lock.
  */
 class Mutex : private boost::noncopyable {
   public:
     typedef ScopedLock<Mutex> ScopedLock;
+    typedef ScopedUnlock<Mutex> ScopedUnlock;
     
     inline Mutex();
     inline ~Mutex();
diff --git a/cpp/lib/common/sys/ThreadSafeQueue.h b/cpp/lib/common/sys/ThreadSafeQueue.h
new file mode 100644
index 0000000000..89454d693f
--- /dev/null
+++ b/cpp/lib/common/sys/ThreadSafeQueue.h
@@ -0,0 +1,99 @@
+#ifndef _sys_ThreadSafeQueue_h
+#define _sys_ThreadSafeQueue_h
+
+/*
+ *
+ * Copyright (c) 2006 The Apache Software Foundation
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *    http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ *
+ */
+
+#include <deque>
+#include "ProducerConsumer.h"
+#include "Exception.h"
+
+namespace qpid {
+namespace sys {
+
+/**
+ * A thread safe queue template.
+ */
+template <class T, class ContainerType=std::deque<T> >
+class ThreadSafeQueue
+{
+  public:
+    struct QueueStoppedException : public Exception {};
+
+    ThreadSafeQueue() {}
+
+    /** Push a value onto the back of the queue */
+    void push(const T& value) {
+        ProducerConsumer::ProducerLock producer(pc);
+        if (producer.isOk()) {
+            producer.confirm();
+            container.push_back(value);
+        }
+    }
+
+    /** Pop a value from the front of the queue. Waits till value is available.
+     *@throw QueueStoppedException if queue is stopped while waiting.
+     */
+    T pop() {
+        ProducerConsumer::ConsumerLock consumer(pc);
+        if (consumer.isOk()) {
+            consumer.confirm();
+            T value(container.front());
+            container.pop_front();
+            return value;
+        }
+        throw QueueStoppedException();
+    }
+
+    /**
+     * If a value becomes available within the timeout, set outValue
+     * and return true. Otherwise return false;
+     */
+    bool pop(T& outValue, const Time& timeout) {
+        ProducerConsumer::ConsumerLock consumer(pc, timeout);
+        if (consumer.isOk()) {
+            consumer.confirm();
+            outValue = container.front();
+            container.pop_front();
+            return true;
+        }
+        return false;
+    }
+
+    /** Interrupt threads waiting in pop() */
+    void stop() { pc.stop(); }
+
+    /** True if queue is stopped */
+    bool isStopped() { return pc.isStopped(); }
+
+    /** Size of the queue */
+    size_t size() { ProducerConsumer::Lock l(pc); return container.size(); }
+
+    /** True if queue is empty */
+    bool empty() { ProducerConsumer::Lock l(pc); return container.empty(); }
+
+  private:
+    ProducerConsumer pc;
+    ContainerType container;
+};
+
+}} // namespace qpid::sys
+
+
+
+#endif  /*!_sys_ThreadSafeQueue_h*/