You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@activemq.apache.org by ta...@apache.org on 2012/11/21 18:34:34 UTC

svn commit: r1412205 - in /activemq/activemq-cpp/trunk/activemq-cpp/src: examples/cmstemplate-stress/ main/activemq/state/ main/decaf/internal/util/concurrent/ main/decaf/lang/ main/decaf/util/ main/decaf/util/concurrent/locks/

Author: tabish
Date: Wed Nov 21 17:34:32 2012
New Revision: 1412205

URL: http://svn.apache.org/viewvc?rev=1412205&view=rev
Log:
Fix a bunch of compiler warnings

Modified:
    activemq/activemq-cpp/trunk/activemq-cpp/src/examples/cmstemplate-stress/CmsMessageCreator.cpp
    activemq/activemq-cpp/trunk/activemq-cpp/src/examples/cmstemplate-stress/MessagingTask.cpp
    activemq/activemq-cpp/trunk/activemq-cpp/src/examples/cmstemplate-stress/MessagingTask.h
    activemq/activemq-cpp/trunk/activemq-cpp/src/examples/cmstemplate-stress/Receiver.h
    activemq/activemq-cpp/trunk/activemq-cpp/src/examples/cmstemplate-stress/Sender.h
    activemq/activemq-cpp/trunk/activemq-cpp/src/main/activemq/state/ConnectionStateTracker.cpp
    activemq/activemq-cpp/trunk/activemq-cpp/src/main/decaf/internal/util/concurrent/Threading.cpp
    activemq/activemq-cpp/trunk/activemq-cpp/src/main/decaf/lang/ArrayPointer.h
    activemq/activemq-cpp/trunk/activemq-cpp/src/main/decaf/util/MapEntry.h
    activemq/activemq-cpp/trunk/activemq-cpp/src/main/decaf/util/concurrent/locks/AbstractQueuedSynchronizer.cpp

Modified: activemq/activemq-cpp/trunk/activemq-cpp/src/examples/cmstemplate-stress/CmsMessageCreator.cpp
URL: http://svn.apache.org/viewvc/activemq/activemq-cpp/trunk/activemq-cpp/src/examples/cmstemplate-stress/CmsMessageCreator.cpp?rev=1412205&r1=1412204&r2=1412205&view=diff
==============================================================================
--- activemq/activemq-cpp/trunk/activemq-cpp/src/examples/cmstemplate-stress/CmsMessageCreator.cpp (original)
+++ activemq/activemq-cpp/trunk/activemq-cpp/src/examples/cmstemplate-stress/CmsMessageCreator.cpp Wed Nov 21 17:34:32 2012
@@ -27,8 +27,7 @@ using namespace activemq::core;
 using namespace cmstemplate;
 
 ////////////////////////////////////////////////////////////////////////////////
-CmsMessageCreator::CmsMessageCreator(const std::string& txt) {
-    m_txt = txt;
+CmsMessageCreator::CmsMessageCreator(const std::string& txt) : m_txt(txt) {
 }
 
 ////////////////////////////////////////////////////////////////////////////////

Modified: activemq/activemq-cpp/trunk/activemq-cpp/src/examples/cmstemplate-stress/MessagingTask.cpp
URL: http://svn.apache.org/viewvc/activemq/activemq-cpp/trunk/activemq-cpp/src/examples/cmstemplate-stress/MessagingTask.cpp?rev=1412205&r1=1412204&r2=1412205&view=diff
==============================================================================
--- activemq/activemq-cpp/trunk/activemq-cpp/src/examples/cmstemplate-stress/MessagingTask.cpp (original)
+++ activemq/activemq-cpp/trunk/activemq-cpp/src/examples/cmstemplate-stress/MessagingTask.cpp Wed Nov 21 17:34:32 2012
@@ -21,9 +21,7 @@ using namespace std;
 using namespace cmstemplate;
 
 ////////////////////////////////////////////////////////////////////////////////
-MessagingTask::MessagingTask(Receiver* receiver, const string& message) {
-    m_receiver = receiver;
-    m_message = string(message);
+MessagingTask::MessagingTask(Receiver* receiver, const string& message) : m_receiver(receiver), m_message(message) {
 }
 
 ////////////////////////////////////////////////////////////////////////////////

Modified: activemq/activemq-cpp/trunk/activemq-cpp/src/examples/cmstemplate-stress/MessagingTask.h
URL: http://svn.apache.org/viewvc/activemq/activemq-cpp/trunk/activemq-cpp/src/examples/cmstemplate-stress/MessagingTask.h?rev=1412205&r1=1412204&r2=1412205&view=diff
==============================================================================
--- activemq/activemq-cpp/trunk/activemq-cpp/src/examples/cmstemplate-stress/MessagingTask.h (original)
+++ activemq/activemq-cpp/trunk/activemq-cpp/src/examples/cmstemplate-stress/MessagingTask.h Wed Nov 21 17:34:32 2012
@@ -29,6 +29,11 @@ namespace cmstemplate {
         Receiver* m_receiver;
         std::string m_message;
 
+    private:
+
+        MessagingTask(const MessagingTask&);
+        MessagingTask& operator= (const MessagingTask&);
+
     public:
 
         MessagingTask(Receiver* m_receiver, const std::string& message);

Modified: activemq/activemq-cpp/trunk/activemq-cpp/src/examples/cmstemplate-stress/Receiver.h
URL: http://svn.apache.org/viewvc/activemq/activemq-cpp/trunk/activemq-cpp/src/examples/cmstemplate-stress/Receiver.h?rev=1412205&r1=1412204&r2=1412205&view=diff
==============================================================================
--- activemq/activemq-cpp/trunk/activemq-cpp/src/examples/cmstemplate-stress/Receiver.h (original)
+++ activemq/activemq-cpp/trunk/activemq-cpp/src/examples/cmstemplate-stress/Receiver.h Wed Nov 21 17:34:32 2012
@@ -65,6 +65,11 @@ namespace cmstemplate {
         void DecreaseNumOfMessagingTasks();
         long GetNumOfMessagingTasks();
 
+    private:
+
+        Receiver(const Receiver&);
+        Receiver& operator= (const Receiver&);
+
     public:
 
         Receiver(const std::string& url, const std::string& queueOrTopicName, bool isTopic, long long receiveTimeout = 2000, bool useThreadPool = true);

Modified: activemq/activemq-cpp/trunk/activemq-cpp/src/examples/cmstemplate-stress/Sender.h
URL: http://svn.apache.org/viewvc/activemq/activemq-cpp/trunk/activemq-cpp/src/examples/cmstemplate-stress/Sender.h?rev=1412205&r1=1412204&r2=1412205&view=diff
==============================================================================
--- activemq/activemq-cpp/trunk/activemq-cpp/src/examples/cmstemplate-stress/Sender.h (original)
+++ activemq/activemq-cpp/trunk/activemq-cpp/src/examples/cmstemplate-stress/Sender.h Wed Nov 21 17:34:32 2012
@@ -33,6 +33,11 @@ namespace cmstemplate {
         decaf::util::concurrent::Mutex m_cmsTemplateMutex;
         activemq::cmsutil::CmsTemplate* m_cmsTemplate;
 
+    private:
+
+        Sender(const Sender&);
+        Sender& operator= (const Sender&);
+
     public:
 
         Sender(const std::string& url, const std::string& queueOrTopicName,

Modified: activemq/activemq-cpp/trunk/activemq-cpp/src/main/activemq/state/ConnectionStateTracker.cpp
URL: http://svn.apache.org/viewvc/activemq/activemq-cpp/trunk/activemq-cpp/src/main/activemq/state/ConnectionStateTracker.cpp?rev=1412205&r1=1412204&r2=1412205&view=diff
==============================================================================
--- activemq/activemq-cpp/trunk/activemq-cpp/src/main/activemq/state/ConnectionStateTracker.cpp (original)
+++ activemq/activemq-cpp/trunk/activemq-cpp/src/main/activemq/state/ConnectionStateTracker.cpp Wed Nov 21 17:34:32 2012
@@ -76,6 +76,7 @@ namespace state {
 ConnectionStateTracker::ConnectionStateTracker() : TRACKED_RESPONSE_MARKER( new Tracked() ),
                                                    connectionStates(),
                                                    messageCache(),
+                                                   messagePullCache(),
                                                    trackTransactions(false),
                                                    restoreSessions(true),
                                                    restoreConsumers(true),

Modified: activemq/activemq-cpp/trunk/activemq-cpp/src/main/decaf/internal/util/concurrent/Threading.cpp
URL: http://svn.apache.org/viewvc/activemq/activemq-cpp/trunk/activemq-cpp/src/main/decaf/internal/util/concurrent/Threading.cpp?rev=1412205&r1=1412204&r2=1412205&view=diff
==============================================================================
--- activemq/activemq-cpp/trunk/activemq-cpp/src/main/decaf/internal/util/concurrent/Threading.cpp (original)
+++ activemq/activemq-cpp/trunk/activemq-cpp/src/main/decaf/internal/util/concurrent/Threading.cpp Wed Nov 21 17:34:32 2012
@@ -50,6 +50,11 @@ namespace {
 
         ThreadHandle* thread;
 
+    private:
+
+        SuspendedCompletionCondition(const SuspendedCompletionCondition&);
+        SuspendedCompletionCondition& operator= (const SuspendedCompletionCondition&);
+
     public:
 
         SuspendedCompletionCondition(ThreadHandle* thread) : thread(thread) {
@@ -65,6 +70,11 @@ namespace {
 
         ThreadHandle* handle;
 
+    private:
+
+        MonitorWaitCompletionCondition(const MonitorWaitCompletionCondition&);
+        MonitorWaitCompletionCondition& operator= (const MonitorWaitCompletionCondition&);
+
     public:
 
         MonitorWaitCompletionCondition(ThreadHandle* handle) : handle(handle) {
@@ -975,6 +985,11 @@ namespace {
         ThreadHandle* self;
         ThreadHandle* target;
 
+    private:
+
+        JoinCompletionCondition(const JoinCompletionCondition&);
+        JoinCompletionCondition& operator= (const JoinCompletionCondition&);
+
     public:
 
         JoinCompletionCondition(ThreadHandle* self, ThreadHandle* target) : self(self), target(target) {
@@ -1100,6 +1115,11 @@ namespace {
 
         ThreadHandle* handle;
 
+    private:
+
+        SleepCompletionCondition(const SleepCompletionCondition&);
+        SleepCompletionCondition& operator= (const SleepCompletionCondition&);
+
     public:
 
         SleepCompletionCondition(ThreadHandle* handle) : handle(handle) {
@@ -1239,6 +1259,11 @@ namespace {
 
         ThreadHandle* handle;
 
+    private:
+
+        ParkCompletionCondition(const ParkCompletionCondition&);
+        ParkCompletionCondition& operator= (const ParkCompletionCondition&);
+
     public:
 
         ParkCompletionCondition(ThreadHandle* handle) : handle(handle) {

Modified: activemq/activemq-cpp/trunk/activemq-cpp/src/main/decaf/lang/ArrayPointer.h
URL: http://svn.apache.org/viewvc/activemq/activemq-cpp/trunk/activemq-cpp/src/main/decaf/lang/ArrayPointer.h?rev=1412205&r1=1412204&r2=1412205&view=diff
==============================================================================
--- activemq/activemq-cpp/trunk/activemq-cpp/src/main/decaf/lang/ArrayPointer.h (original)
+++ activemq/activemq-cpp/trunk/activemq-cpp/src/main/decaf/lang/ArrayPointer.h Wed Nov 21 17:34:32 2012
@@ -52,6 +52,13 @@ namespace lang {
     private:
 
         struct ArrayData {
+        private:
+
+            ArrayData(const ArrayData&);
+            ArrayData& operator= (const ArrayData&);
+
+        public:
+
             T* value;
             int length;
             decaf::util::concurrent::atomic::AtomicInteger refs;

Modified: activemq/activemq-cpp/trunk/activemq-cpp/src/main/decaf/util/MapEntry.h
URL: http://svn.apache.org/viewvc/activemq/activemq-cpp/trunk/activemq-cpp/src/main/decaf/util/MapEntry.h?rev=1412205&r1=1412204&r2=1412205&view=diff
==============================================================================
--- activemq/activemq-cpp/trunk/activemq-cpp/src/main/decaf/util/MapEntry.h (original)
+++ activemq/activemq-cpp/trunk/activemq-cpp/src/main/decaf/util/MapEntry.h Wed Nov 21 17:34:32 2012
@@ -35,9 +35,19 @@ namespace util {
         MapEntry() : key(), value() {
         }
 
+        MapEntry(const MapEntry& other) : key(other.getKey()), value(other.getValue()) {
+        }
+
         MapEntry(const K& key, const V& value) : key(key), value(value) {
         }
 
+        MapEntry& operator= (const MapEntry& other) {
+            this->key = other.getKey();
+            this->value = other.getValue();
+
+            return *this;
+        }
+
         virtual ~MapEntry() {};
 
         virtual void setKey(K key) {

Modified: activemq/activemq-cpp/trunk/activemq-cpp/src/main/decaf/util/concurrent/locks/AbstractQueuedSynchronizer.cpp
URL: http://svn.apache.org/viewvc/activemq/activemq-cpp/trunk/activemq-cpp/src/main/decaf/util/concurrent/locks/AbstractQueuedSynchronizer.cpp?rev=1412205&r1=1412204&r2=1412205&view=diff
==============================================================================
--- activemq/activemq-cpp/trunk/activemq-cpp/src/main/decaf/util/concurrent/locks/AbstractQueuedSynchronizer.cpp (original)
+++ activemq/activemq-cpp/trunk/activemq-cpp/src/main/decaf/util/concurrent/locks/AbstractQueuedSynchronizer.cpp Wed Nov 21 17:34:32 2012
@@ -369,7 +369,7 @@ namespace locks {
     public:
 
         SynchronizerState(AbstractQueuedSynchronizer* parent) :
-            parent(parent), state(0), head(), tail() {
+            parent(parent), state(0), head(), tail(), nodePool() {
         }
 
         virtual ~SynchronizerState() {