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/02/25 20:59:58 UTC

svn commit: r1293674 - /activemq/activemq-cpp/trunk/activemq-cpp/src/main/decaf/util/AbstractQueue.h

Author: tabish
Date: Sat Feb 25 19:59:58 2012
New Revision: 1293674

URL: http://svn.apache.org/viewvc?rev=1293674&view=rev
Log:
fix for: https://issues.apache.org/jira/browse/AMQCPP-389

Modified:
    activemq/activemq-cpp/trunk/activemq-cpp/src/main/decaf/util/AbstractQueue.h

Modified: activemq/activemq-cpp/trunk/activemq-cpp/src/main/decaf/util/AbstractQueue.h
URL: http://svn.apache.org/viewvc/activemq/activemq-cpp/trunk/activemq-cpp/src/main/decaf/util/AbstractQueue.h?rev=1293674&r1=1293673&r2=1293674&view=diff
==============================================================================
--- activemq/activemq-cpp/trunk/activemq-cpp/src/main/decaf/util/AbstractQueue.h (original)
+++ activemq/activemq-cpp/trunk/activemq-cpp/src/main/decaf/util/AbstractQueue.h Sat Feb 25 19:59:58 2012
@@ -58,9 +58,9 @@ namespace util {
          * This implementation returns true if offer succeeds, else throws an
          * IllegalStateException.
          */
-        virtual bool add( const E& value ) {
+        virtual bool add(const E& value) {
 
-            if( offer( value ) ) {
+            if (this->offer(value )) {
                 return true;
             }
 
@@ -75,14 +75,14 @@ namespace util {
          * throws an IllegalArgumentException if so, otherwise it delegates the add to
          * the AbstractCollection's addAll implementation.
          */
-        virtual bool addAll( const Collection<E>& collection ) {
+        virtual bool addAll(const Collection<E>& collection) {
 
-            if( this == &collection ) {
+            if (this == &collection) {
                 throw decaf::lang::exceptions::IllegalArgumentException(
                     __FILE__, __LINE__, "A Queue cannot be added to itself." );
             }
 
-            return AbstractCollection<E>::addAll( collection );
+            return AbstractCollection<E>::addAll(collection);
         }
 
         using AbstractCollection<E>::remove;
@@ -95,7 +95,7 @@ namespace util {
         virtual E remove() {
 
             E result;
-            if( this->poll( result ) == true ) {
+            if (this->poll(result) == true) {
                 return result;
             }
 
@@ -112,7 +112,7 @@ namespace util {
         virtual E element() const {
 
             E result;
-            if( this->peek( result ) == true ) {
+            if (this->peek( result ) == true) {
                 return result;
             }
 
@@ -127,7 +127,7 @@ namespace util {
          */
         virtual void clear() {
 
-            if( this->isEmpty() ) {
+            if (this->isEmpty()) {
                 return;
             }
 
@@ -135,10 +135,9 @@ namespace util {
             bool successful = true;
 
             do {
-                successful = this->poll( result );
-            } while( successful );
+                successful = this->poll(result);
+            } while(successful);
         }
-
     };
 
 }}