summaryrefslogtreecommitdiff
path: root/examples/menuconfig.py
diff options
context:
space:
mode:
authorUlf Magnusson <ulfalizer@gmail.com>2018-02-07 02:30:16 +0100
committerUlf Magnusson <ulfalizer@gmail.com>2018-02-07 03:55:56 +0100
commitf66cd7155158943987570937be37b09d9ca58028 (patch)
tree463bd909a1b04062e3176d61c866a62cbb400f33 /examples/menuconfig.py
parentbb60c1158e8ab98308962abc14ad52e99f7e7f33 (diff)
Allow "n"/"m"/"y" as aliases for 0/1/2 in set_value()
More experience working with the API convinced me that it's worth it. Gets rid of ugly conversions in the menuconfig.py and oldconfig.py examples, and streamlines some things internally as well. Include two other small fixes as well: - Make warnings generated by Choice.set_value() match those generated by Symbol.set_value(). - Get rid of the input stripping in menuconfig.py. It's not like the interface is usable as-is anyway, and it just complicates the example.
Diffstat (limited to 'examples/menuconfig.py')
-rw-r--r--examples/menuconfig.py14
1 files changed, 1 insertions, 13 deletions
diff --git a/examples/menuconfig.py b/examples/menuconfig.py
index b55af6d..753ac40 100644
--- a/examples/menuconfig.py
+++ b/examples/menuconfig.py
@@ -258,19 +258,7 @@ def get_value_from_user(sc):
.format(", ".join([TRI_TO_STR[val] for val in sc.assignable]))
prompt += ": "
- val_str = input(prompt).strip()
- if sc.type in (BOOL, TRISTATE):
- if val_str not in STR_TO_TRI:
- print("'{}' is not a valid tristate value".format(val_str))
- return False
-
- # I was thinking of having set_value() accept "n", "m", "y" as well as
- # a convenience for BOOL / TRISTATE symbols. Consistently using 0, 1, 2
- # makes the format clearer though. That's the best format in all ways
- # except for readability (where it isn't horrible either).
- val = STR_TO_TRI[val_str]
- else:
- val = val_str
+ val = input(prompt)
# Automatically add a "0x" prefix for hex symbols, like the menuconfig
# interface does. This isn't done when loading .config files, hence why