]> 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)
committerPatR <rankin@nethack.org>
Thu, 13 Oct 2022 00:02:14 +0000 (17:02 -0700)
commitb02e01822564e5bee3c31082e978419edea6a37c
treedfd2564e0671ead42378462ac16348757ff91f82
parente5a99e8ad893a788bb9107512182dc4ee5e310d3
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