openstack/deb-python-openstacksdk
Code Issues Proposed changes

Get rid of some useless code

Browse Source 
For some reason this argument parsing code in examples had
a useless if statement.

Change-Id: I9ef9fdb48f922b1fd95fafe0286de8f901b142a7
This commit is contained in:
Terry Howe
2015-01-23 12:06:39 -07:00
parent 00c2a13004
commit 763c741860
1 changed files with 4 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
examples/common.py
View File

@@ -117,18 +117,11 @@ class UserPreferenceAction(argparse.Action):
var = var.replace('OS_API_', '') var = var.replace('OS_API_', '')
var = var.lower() var = var.lower()
for kvp in values.split(','): for kvp in values.split(','):
if var == 'region': if '=' in kvp:
if '=' in kvp: service, value = kvp.split('=')
service, value = kvp.split('=')
else:
service = cls.pref.ALL
value = kvp
else: else:
if '=' in kvp: service = cls.pref.ALL
service, value = kvp.split('=') value = kvp
else:
service = cls.pref.ALL
value = kvp
if var == 'name': if var == 'name':
cls.pref.set_name(service, value) cls.pref.set_name(service, value)
elif var == 'region': elif var == 'region':