From 88c1fd2f5c6aec91eb4e13dee1c4b1b703b3d886 Mon Sep 17 00:00:00 2001 From: hut Date: Sat, 14 Nov 2015 03:26:31 +0100 Subject: ext.rifle: slight efficiency improvement over last commit Don't check for all the other possible function values if function=='ext' was already established. --- ranger/ext/rifle.py | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/ranger/ext/rifle.py b/ranger/ext/rifle.py index 3c54cfc2..c43de24f 100755 --- a/ranger/ext/rifle.py +++ b/ranger/ext/rifle.py @@ -194,11 +194,12 @@ class Rifle(object): function = rule[0] argument = rule[1] if len(rule) > 1 else '' - if function == 'ext' and os.path.isfile(files[0]): - partitions = os.path.basename(files[0]).rpartition('.') - if not partitions[0]: - return False - return bool(re.search('^(' + argument + ')$', partitions[2].lower())) + if function == 'ext': + if os.path.isfile(files[0]): + partitions = os.path.basename(files[0]).rpartition('.') + if not partitions[0]: + return False + return bool(re.search('^(' + argument + ')$', partitions[2].lower())) elif function == 'name': return bool(re.search(argument, os.path.basename(files[0]))) elif function == 'match': -- cgit 1.4.1-2-gfad0