]> granicus.if.org Git - nethack/commit
Remove explicit 'none' opt from autounlock handler
authorMichael Meyer <me@entrez.cc>
Fri, 7 Oct 2022 16:39:18 +0000 (12:39 -0400)
committernhmall <nhmall@nethack.org>
Sat, 15 Oct 2022 16:28:53 +0000 (12:28 -0400)
commit4c98ba493bb7eaa7c556c4b720a7b6df7ea74d0e
treedbfb4132572e4f7944003baf5921ab0120b4b421
parent555196d334034d91c66150b1c98fe942fe400bb2
Remove explicit 'none' opt from autounlock handler

The autounlock handler included an explicit 'none' option, a choice that
gave it a different UX from similar existing compound option handlers
(e.g. paranoid_confirm or pickup_types), which set 'none' simply by
deselecting all options.  It didn't make the menu any easier to use (at
least in my experience), since in order to go from some combination of
options to 'none', you'd have to deselect everything anyway (which on
its own was enough to set 'none', so there was no reason to explicitly
select it after doing so).

Make the autounlock handler work like other compound option handlers,
such that deselecting all options is the way to set 'none', and there is
no explicit 'none' option included in the list.
src/options.c