[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#19690: [PATCH] Clarify mode switch messages to avoid potential confu
bug#19690: [PATCH] Clarify mode switch messages to avoid potential confusion
Mon, 26 Jan 2015 03:40:58 +0000
This is only a feature enhancement, not a bug report.
When first learning Emacs, I was often confused about whether a minor mode was
enabled globally or just in the current buffer. For example, show-paren-mode is
global, but whitespace-mode is buffer-local (unless you use
global-whitespace-mode). I still sometimes misinterpret this.
Sure, the user can find out, but the attached patch helps make things clear, by
indicating if the mode being toggled is buffer-local. It will be especially
helpful for alerting users in the relatively rare cases when they toggle a
normally-global minor mode that has been set buffer-locally.
BTW, in define-minor-mode, last-message is indirected, but I don't see any
reason not to just use it directly.
@@ -279,14 +279,23 @@
(if (called-interactively-p 'any)
,(if (and globalp (symbolp mode))
+ ;; Unnecessary but harmless if mode set buffer-locally
;; Avoid overwriting a message shown by the body,
;; but do overwrite previous messages.
(unless (and (current-message)
(not (equal ,last-message
- (message ,(format "%s %%sabled" pretty-name)
- (if ,mode "en" "dis")))))
+ (let ((local
+ ,(if globalp
+ (if (symbolp mode)
+ `(if (assq ',mode (buffer-local-variables))
+ " in current buffer"
+ " in current buffer")))
+ (message ,(format "%s %%sabled%%s" pretty-name)
+ (if ,mode "en" "dis") local)))))
,@(when after-hook `(,after-hook)))
;; Return the new setting.
|[Prev in Thread]
||[Next in Thread]|
- bug#19690: [PATCH] Clarify mode switch messages to avoid potential confusion,
Kelly Dean <=