]> granicus.if.org Git - python/commitdiff
Wrapped a long line.
authorFred Drake <fdrake@acm.org>
Wed, 12 Dec 2001 06:20:34 +0000 (06:20 +0000)
committerFred Drake <fdrake@acm.org>
Wed, 12 Dec 2001 06:20:34 +0000 (06:20 +0000)
Converted to use "".startswith() to avoid slicing (& temp string creation).

Lib/getopt.py

index 82cf04d76a253ddbe1936ec3fffd11501d815682..f1dc7ac446522370bb9063cd4a1cff2dc6fea6b7 100644 (file)
@@ -68,7 +68,7 @@ def getopt(args, shortopts, longopts = []):
         if args[0] == '--':
             args = args[1:]
             break
-        if args[0][:2] == '--':
+        if args[0].startswith('--'):
             opts, args = do_longs(opts, args[0][2:], longopts, args[1:])
         else:
             opts, args = do_shorts(opts, args[0][1:], shortopts, args[1:])
@@ -124,7 +124,8 @@ def do_shorts(opts, optstring, shortopts, args):
         if short_has_arg(opt, shortopts):
             if optstring == '':
                 if not args:
-                    raise GetoptError('option -%s requires argument' % opt, opt)
+                    raise GetoptError('option -%s requires argument' % opt,
+                                      opt)
                 optstring, args = args[0], args[1:]
             optarg, optstring = optstring, ''
         else:
@@ -135,7 +136,7 @@ def do_shorts(opts, optstring, shortopts, args):
 def short_has_arg(opt, shortopts):
     for i in range(len(shortopts)):
         if opt == shortopts[i] != ':':
-            return shortopts[i+1:i+2] == ':'
+            return shortopts.startswith(':', i+1)
     raise GetoptError('option -%s not recognized' % opt, opt)
 
 if __name__ == '__main__':