simplify control flow in RestoreHelper methods
authorPhilipp Gesang <philipp.gesang@intra2net.com>
Wed, 2 Nov 2016 16:42:33 +0000 (17:42 +0100)
committerPhilipp Gesang <philipp.gesang@intra2net.com>
Thu, 3 Nov 2016 09:57:22 +0000 (10:57 +0100)
commit7273719cca856677d25102d805f6f96e36173731
tree40a214822a294c3cbfa7b32d15d84da63c1bff1a
parent617367ce684a770181972b277ed1cc2be7f3dd34
simplify control flow in RestoreHelper methods

Make the control flow more obvious. The code in question was
introduced with commit ea6d3c3e… but did not make sense back then
either because cur_index which is the constant $1$ was compared
to the literal constant $0$:

+        cur_index = 1
+        while cur_index < len(self._data):
+            data = self._data[cur_index]

+                if cur_index == 0:

This bogus test was since removed but the convoluted ``while``
loop survived. Instead, access index 1 only once using an integer
literal.
deltatar/deltatar.py