You are viewing a plain text version of this content. The canonical link for it is here.
Posted to droids-commits@incubator.apache.org by rf...@apache.org on 2011/11/16 00:34:49 UTC
svn commit: r1202485 -
/incubator/droids/trunk/droids-core/src/main/java/org/apache/droids/impl/SimpleTaskQueueWithHistory.java
Author: rfrovarp
Date: Wed Nov 16 00:34:48 2011
New Revision: 1202485
URL: http://svn.apache.org/viewvc?rev=1202485&view=rev
Log:
Fixes DROIDS-153
Modified:
incubator/droids/trunk/droids-core/src/main/java/org/apache/droids/impl/SimpleTaskQueueWithHistory.java
Modified: incubator/droids/trunk/droids-core/src/main/java/org/apache/droids/impl/SimpleTaskQueueWithHistory.java
URL: http://svn.apache.org/viewvc/incubator/droids/trunk/droids-core/src/main/java/org/apache/droids/impl/SimpleTaskQueueWithHistory.java?rev=1202485&r1=1202484&r2=1202485&view=diff
==============================================================================
--- incubator/droids/trunk/droids-core/src/main/java/org/apache/droids/impl/SimpleTaskQueueWithHistory.java (original)
+++ incubator/droids/trunk/droids-core/src/main/java/org/apache/droids/impl/SimpleTaskQueueWithHistory.java Wed Nov 16 00:34:48 2011
@@ -59,15 +59,11 @@ public class SimpleTaskQueueWithHistory<
@Override
public boolean addAll(Collection<? extends T> c)
{
- HashSet<String> ids = new HashSet<String>();
+ boolean hasChanged = false;
for (T e : c) {
- ids.add(e.getId());
- }
- if (previous.addAll(ids)) {
- return super.addAll(c);
- } else {
- return false;
+ hasChanged = hasChanged || this.add(e);
}
+ return hasChanged;
}
public void clearHistory()