Code cleanup

git-svn-id: https://svn.apache.org/repos/asf/activemq/activemq-cpp/trunk@690390 13f79535-47bb-0310-9956-ffa450edef68
apache · Aug 29, 2008 · 5b9444d · 5b9444d
1 parent 9506550
commit 5b9444d
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 15 deletions.
diff --git a/src/main/activemq/connector/openwire/commands/ActiveMQMapMessage.h b/src/main/activemq/connector/openwire/commands/ActiveMQMapMessage.h
@@ -38,6 +38,11 @@ namespace commands{
     class ActiveMQMapMessage :
         public ActiveMQMessageBase<cms::MapMessage> {
 
+    private:
+
+        // Map Structure to hold unmarshalled Map Data
+        mutable util::PrimitiveMap* map;
+
     public:
 
         const static unsigned char ID_ACTIVEMQMAPMESSAGE = 25;
@@ -336,11 +341,6 @@ namespace commands{
         virtual void checkMapIsUnmarshalled() const
             throw ( decaf::lang::exceptions::NullPointerException );
 
-    private:
-
-        // Map Structure to hold unmarshalled Map Data
-        mutable util::PrimitiveMap* map;
-
     };
 
 }}}}

diff --git a/src/main/activemq/connector/openwire/commands/ActiveMQMessageBase.h b/src/main/activemq/connector/openwire/commands/ActiveMQMessageBase.h
@@ -43,6 +43,11 @@ namespace commands{
     class ActiveMQMessageBase : public T,
                                 public Message,
                                 public core::ActiveMQMessage {
+    private:
+
+        core::ActiveMQAckHandler* ackHandler;
+        int redeliveryCount;
+        util::PrimitiveMap properties;
 
     public:
 
@@ -228,7 +233,7 @@ namespace commands{
          * Acknowledges all consumed messages of the session
          * of this consumed message.
          */
-        virtual void acknowledge(void) const throw( cms::CMSException ) {
+        virtual void acknowledge() const throw( cms::CMSException ) {
             try{
                 this->getAckHandler()->acknowledgeMessage( this );
             }
@@ -555,7 +560,7 @@ namespace commands{
          * Get the Correlation Id for this message
          * @return string representation of the correlation Id
          */
-        virtual std::string getCMSCorrelationID(void) const {
+        virtual std::string getCMSCorrelationID() const {
             return this->getCorrelationId();
         }
 
@@ -571,7 +576,7 @@ namespace commands{
          * Gets the DeliveryMode for this message
          * @return DeliveryMode enumerated value.
          */
-        virtual int getCMSDeliveryMode(void) const {
+        virtual int getCMSDeliveryMode() const {
             return this->isPersistent();
         }
 
@@ -725,12 +730,6 @@ namespace commands{
             this->setType( type );
         }
 
-    private:
-
-        core::ActiveMQAckHandler* ackHandler;
-        int redeliveryCount;
-        util::PrimitiveMap properties;
-
     };
 
 }}}}

diff --git a/src/main/activemq/connector/openwire/commands/ActiveMQTextMessage.h b/src/main/activemq/connector/openwire/commands/ActiveMQTextMessage.h
@@ -71,7 +71,7 @@ namespace commands{
          * such as its type and value of its elements.
          * @return formatted string useful for debugging.
          */
-        virtual std::string toString() const{
+        virtual std::string toString() const {
             std::ostringstream stream;
 
             stream << "Begin Class = ActiveMQTextMessage" << std::endl;