Skip to content

Commit

Permalink
Fix the arguments --text, --data, --rodata, --bss and --metric in ben…
Browse files Browse the repository at this point in the history
…chmark_size

This fix addresses issue #119 by making the implementation of the arguments
--text, --data, --rodata, --bss and --metric of benchmark_size.py match the
documenation.

Files changed:

        * benchmark_size.py: action='apend' is replaced by nargs='+' for
        --text, --data, --rodata, --bss and --metric.
  • Loading branch information
Roger-Shepherd authored and PaoloS02 committed Apr 8, 2021
1 parent e4ab4a9 commit d5686b7
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions benchmark_size.py
Original file line number Diff line number Diff line change
Expand Up @@ -110,35 +110,35 @@ def build_parser():
'--text',
type=str,
default=[],
action='append',
nargs='+',
help='Section name(s) containing code'
)
parser.add_argument(
'--data',
type=str,
default=[],
action='append',
nargs='+',
help='Section name(s) containing non-zero initialized writable data'
)
parser.add_argument(
'--rodata',
type=str,
default=[],
action='append',
nargs='+',
help='Section name(s) containing read only data'
)
parser.add_argument(
'--bss',
type=str,
default=[],
action='append',
nargs='+',
help='Section name(s) containing zero initialized writable data'
)
parser.add_argument(
'--metric',
type=str,
default=[],
action='append',
nargs='+',
choices=['text', 'rodata', 'data', 'bss'],
help='Sections to include in metric: one or more of "text", "rodata", '
+ '"data" or "bss". Default "text"',
Expand Down

0 comments on commit d5686b7

Please sign in to comment.