From 001823fc4249277e222a8b14c83498b8deebfabb Mon Sep 17 00:00:00 2001
From: Arne H Juul <arnej@yahoo-inc.com>
Date: Thu, 12 Jan 2017 11:57:22 +0100
Subject: fix bug

if abortOnError was false, the old code would
always return "true" even with numPending == 0.
---
 .../src/main/java/com/yahoo/feedhandler/FeedResponse.java           | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

(limited to 'vespaclient-core')

diff --git a/vespaclient-core/src/main/java/com/yahoo/feedhandler/FeedResponse.java b/vespaclient-core/src/main/java/com/yahoo/feedhandler/FeedResponse.java
index cdd4ac76c4c..7492a48414b 100755
--- a/vespaclient-core/src/main/java/com/yahoo/feedhandler/FeedResponse.java
+++ b/vespaclient-core/src/main/java/com/yahoo/feedhandler/FeedResponse.java
@@ -123,8 +123,10 @@ public final class FeedResponse extends HttpResponse implements SharedSender.Res
                 log.finest(str);
                 addError(str);
             }
-            isAborted = abortOnError;
-            return !abortOnError;
+            if (abortOnError) {
+                isAborted = true;
+                return false;
+            }
         }
         return numPending > 0;
     }
-- 
cgit v1.2.3