Chromium Code Reviews| Index: native_client_sdk/src/tools/decode_dump.py |
| diff --git a/native_client_sdk/src/tools/decode_dump.py b/native_client_sdk/src/tools/decode_dump.py |
| index 74d07a03fb6699d059d7cd36c4d8a114ee23eeb9..1adc0a5854014c76513cd31047080f63748dbdd7 100755 |
| --- a/native_client_sdk/src/tools/decode_dump.py |
| +++ b/native_client_sdk/src/tools/decode_dump.py |
| @@ -9,7 +9,7 @@ Currently this produces a simple stack trace. |
| """ |
| import json |
| -import optparse |
| +import argparse |
|
binji
2014/11/13 23:57:03
sort
Sam Clegg
2014/11/30 17:55:12
Done.
|
| import os |
| import posixpath |
| import subprocess |
| @@ -177,18 +177,18 @@ class CoreDecoder(object): |
| def Main(args): |
| - parser = optparse.OptionParser( |
| + parser = argparse.ArgumentParser( |
| usage='USAGE: %prog [options] <core.json>') |
|
binji
2014/11/13 23:57:03
remove usage
Sam Clegg
2014/11/30 17:55:12
Done.
|
| - parser.add_option('-m', '--main-nexe', dest='main_nexe', |
| + parser.add_argument('-m', '--main-nexe', dest='main_nexe', |
| help='nexe to resolve NaClMain references from') |
| - parser.add_option('-n', '--nmf', dest='nmf_filename', default='-', |
| + parser.add_argument('-n', '--nmf', dest='nmf_filename', default='-', |
| help='nmf to resolve references from') |
| - parser.add_option('-a', '--addr2line', dest='addr2line', |
| + parser.add_argument('-a', '--addr2line', dest='addr2line', |
| help='path to appropriate addr2line') |
| - parser.add_option('-L', '--library-path', dest='library_paths', |
| + parser.add_argument('-L', '--library-path', dest='library_paths', |
| action='append', default=[], |
| help='path to search for shared libraries') |
| - parser.add_option('-p', '--platform', dest='platform', |
| + parser.add_argument('-p', '--platform', dest='platform', |
| help='platform in a style match nmf files') |
| options, args = parser.parse_args(args) |
|
binji
2014/11/13 23:57:03
options = ...
Sam Clegg
2014/11/30 17:55:12
Done.
|
| if len(args) != 1: |
|
binji
2014/11/13 23:57:03
add positional arg
Sam Clegg
2014/11/30 17:55:12
Done.
|