Bug#800466: pytables: diff for NMU version 3.2.1-1.1

[Gilles Filippini]

Control: tags 800466 + patch
Control: tags 800466 + pending

[Replace XX with correct value]
Dear maintainer,

I've prepared an NMU for pytables (versioned as 3.2.1-1.1) and
uploaded it to DELAYED/XX. Please feel free to tell me if I
should delay it longer.

Regards,

_g.

diff -Nru pytables-3.2.1/debian/changelog pytables-3.2.1/debian/changelog
--- pytables-3.2.1/debian/changelog	2015-08-14 21:42:34.000000000 +0200
+++ pytables-3.2.1/debian/changelog	2015-10-02 20:01:14.000000000 +0200
@@ -1,3 +1,12 @@
+pytables (3.2.1-1.1) unstable; urgency=medium
+
+  * Non-maintainer upload.
+  * New patch: 0006-Fix-upstream-issue-481.patch
+    Fix test suite errors such as 'ValueError: could not broadcast
+    input array from shape (2) into shape (0)' (closes: #800466).
+
+ -- Gilles Filippini <pini at debian.org>  Fri, 02 Oct 2015 20:01:01 +0200
+
 pytables (3.2.1-1) unstable; urgency=medium
 
   * New upstream version
diff -Nru pytables-3.2.1/debian/patches/0006-Fix-upstream-issue-481.patch pytables-3.2.1/debian/patches/0006-Fix-upstream-issue-481.patch
--- pytables-3.2.1/debian/patches/0006-Fix-upstream-issue-481.patch	1970-01-01 01:00:00.000000000 +0100
+++ pytables-3.2.1/debian/patches/0006-Fix-upstream-issue-481.patch	2015-09-29 21:06:16.000000000 +0200
@@ -0,0 +1,40 @@
+From 44dba04d7d72f150a91553f4eb455684dfef0913 Mon Sep 17 00:00:00 2001
+From: Andrea Bedini <andrea at andreabedini.com>
+Date: Wed, 12 Aug 2015 10:09:43 +1000
+Subject: [PATCH] Be careful with signs while interating backward.
+
+This is a quick and possibily partial fix for #481.
+
+It seems that C dictates that istopb - istartb - 1 is an unsigned
+expression (because istartb is unsigned and istopb is signed but not big
+enough to contain istartb).
+
+With this fix I am betting that 1 - istopb + istartb is non-negative. It seems
+to work but this function should be rewritten more clearly.
+---
+ tables/tableextension.pyx | 5 +++--
+ 1 file changed, 3 insertions(+), 2 deletions(-)
+
+Index: pytables-3.2.1/tables/tableextension.pyx
+===================================================================
+--- pytables-3.2.1.orig/tables/tableextension.pyx
++++ pytables-3.2.1/tables/tableextension.pyx
+@@ -1241,7 +1241,7 @@ cdef class Row:
+         istartb = (j+istep) % inrowsinbuf
+         inextelement = inextelement + istep
+         i = i + inrowsinbuf
+-    elif 0 > istep:
++    elif istep < 0:
+       inrowsinbuf = self.nrowsinbuf
+       #istartb = self.startb
+       istartb = self.nrowsinbuf - 1
+@@ -1258,7 +1258,8 @@ cdef class Row:
+           i = i - inrowsinbuf
+           continue
+         # Compute the end for this iteration
+-        stopr = startr + ((istopb - istartb - 1) / istep)
++        # (we know we are going backward so try to keep indices positive)
++        stopr = startr + (1 - istopb + istartb) / (-istep)
+         # Read a chunk
+         inrowsread = inrowsread + self.table._read_records(i - inrowsinbuf + 1,
+                                                            inrowsinbuf, self.iobuf)
diff -Nru pytables-3.2.1/debian/patches/series pytables-3.2.1/debian/patches/series
--- pytables-3.2.1/debian/patches/series	2015-08-14 21:42:34.000000000 +0200
+++ pytables-3.2.1/debian/patches/series	2015-09-29 21:05:56.000000000 +0200
@@ -3,3 +3,4 @@
 0003-Never-use-the-msse2-flag-explicitly.patch
 0004-Do-not-fetch-icons-for-external-web-sites.patch
 0005-Fix-setitem-return-value.patch
+0006-Fix-upstream-issue-481.patch