From a2a37de7db88e32c1aa75d14ba724043fbd9a5a2 Mon Sep 17 00:00:00 2001 From: Philipp Gesang Date: Tue, 2 May 2017 15:06:24 +0200 Subject: [PATCH] remove redundant test MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit The first part of the condition always evaluates to True since it’s the precondition to entering that branch. --- deltatar/deltatar.py | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/deltatar/deltatar.py b/deltatar/deltatar.py index b7cf6ad..71912b6 100644 --- a/deltatar/deltatar.py +++ b/deltatar/deltatar.py @@ -1715,7 +1715,7 @@ class RestoreHelper(object): # when member is set, then we can assume everything is right and we # just have to restore the path - if not member: + if member is None: vol_no = file_data.get('volume', -1) # sanity check if not isinstance(vol_no, int) or vol_no < 0: @@ -1724,7 +1724,7 @@ class RestoreHelper(object): # setup the volume that needs to be read. only needed when member is # not set - if not member and index_data['curr_vol_no'] != vol_no: + if index_data['curr_vol_no'] != vol_no: index_data['curr_vol_no'] = vol_no backup_path = os.path.dirname(index_data['path']) vol_name = self._deltatar.volume_name_func(backup_path, -- 1.7.1