[Apt-listbugs-commits] [apt-listbugs] 02/02: fix an off-by-one bug in ParseStep handling

[Francesco Poli]

This is an automated email from the git hooks/post-receive script.

frx-guest pushed a commit to branch master
in repository apt-listbugs.

commit 20b962bc4a4bf7109cdf9671678c675c97e16993
Author: Francesco Poli (wintermute) <invernomuto at paranoici.org>
Date:   Tue Aug 11 00:02:53 2015 +0200

    fix an off-by-one bug in ParseStep handling
---
 debian/changelog              | 2 ++
 lib/aptlistbugs/debian/bts.rb | 2 +-
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/debian/changelog b/debian/changelog
index be45d91..4ebe5d0 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -3,6 +3,8 @@ apt-listbugs (0.1.18) UNRELEASED; urgency=medium
   * added QueryStep configuration option; on network errors, apt-listbugs
     now proposes to the user to retry one package at a time (as well as
     one bug report at a time)
+  * fixed a bug in ParseStep handling: parse bugs in batches of ParseStep
+    bug reports, not ParseStep+1
 
  -- Francesco Poli (wintermute) <invernomuto at paranoici.org>  Mon, 10 Aug 2015 22:42:49 +0200
 
diff --git a/lib/aptlistbugs/debian/bts.rb b/lib/aptlistbugs/debian/bts.rb
index 4f43c6f..f7e749c 100644
--- a/lib/aptlistbugs/debian/bts.rb
+++ b/lib/aptlistbugs/debian/bts.rb
@@ -92,7 +92,7 @@ module Debian
           # parsestep bugs
           reqbugs.each { |bug|
             sa << bug
-            if sa.length > parsestep
+            if sa.length >= parsestep
               soap.get_status(sa, ma_copies).each { |b| bugs << b }
               offset_i += sa.length
               sa = Debian::BTSSOAP::StringArray.new

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/apt-listbugs/apt-listbugs.git