Lars Ingebrigtsen writes: >> +(defcustom help-mode-revert-buffer-noconfirm nil > > In general, I think "negative" variables should be avoided, because > doing the "not not" logic confuses people. So I'd rather have > `help-mode-revert-buffer-confirm' (defaulting to t). I was trying to match the noconfirm argument of the revert function but I agree, `help-mode-revert-buffer-confirm' is less confusing. I've attached an updated patch. Kevin Foley