Reduce redundancy in get_level_by_argv with tryremove.

voussoirkit/vlogging.py

@@ -66,21 +66,23 @@ def get_level_by_argv(argv):
     '''
     argv = argv[:]
 
-    if '--loud' in argv:
+    def tryremove(lst, item):
+        try:
+            lst.remove(item)
+            return True
+        except ValueError:
+            return False
+
+    if tryremove(argv, '--loud'):
         level = LOUD
-        argv.remove('--loud')
+    elif tryremove(argv, '--debug'):
-    elif '--debug' in argv:
         level = DEBUG
-        argv.remove('--debug')
+    elif tryremove(argv, '--warning'):
-    elif '--warning' in argv:
         level = WARNING
-        argv.remove('--warning')
+    elif tryremove(argv, '--quiet'):
-    elif '--quiet' in argv:
         level = ERROR
-        argv.remove('--quiet')
+    elif tryremove(argv, '--silent'):
-    elif '--silent' in argv:
         level = SILENT
-        argv.remove('--silent')
     else:
         level = INFO