perf: use argparse in compare-logs.py

Change-Id: If14a4705fcda5ffdac2a6e5915a7d8b5bf322905

1 files changed, 9 insertions, 27 deletions

diff --git a/perf/compare-logs.py b/perf/compare-logs.py
index 82e6d0c4..90a60daa 100755
--- a/perf/compare-logs.py
+++ b/perf/compare-logs.py
@@ -25,7 +25,8 @@
 # in which case the provisions of the GPLv3+ or the LGPLv3+ are applicable
 # instead of those above.
 
-import sys, os, getopt
+import sys, os
+import argparse
 
 def readFile(url):
     d = {}
@@ -37,35 +38,16 @@ def readFile(url):
            d[key] = float(val)
     return d
 
-def usage():
-    message = """usage: {program} [option]..."
- -h | --help:      print usage information
- --old=URL     path to old file to be compared
- --new=URL     path to new file to be comparted"""
-
-    print(message.format(program = os.path.basename(sys.argv[0])))
-
-
 if __name__ == "__main__":
 
-    try:
-        opts,args = getopt.getopt(sys.argv[1:], "o:n:h",
-                ["old=", "new=", "help"])
-    except getopt.GetoptError:
-        usage()
-        sys.exit(1)
+    parser = argparse.ArgumentParser()
+
+    parser.add_argument('--old', action='store', dest="old", required=True)
+    parser.add_argument('--new', action="store", dest="new", required=True)
 
-    for opt, arg in opts:
-        if opt in ('-h', '--help'):
-            usage()
-            sys.exit(1)
-        elif opt in ('-o', '--old'):
-            old_url = arg
-        elif opt in ('-n', '--new'):
-            new_url = arg
-        else:
-            usage()
-            sys.exit(1)
+    results = parser.parse_args()
+    old_url = results.old
+    new_url = results.new
 
     if not os.path.exists(old_url):
         print("File " + old_url + " doesn't exist!")