emacs-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Emacs-diffs] master 0245cc3: Improve accessibility of window divide


From: Stefan Monnier
Subject: Re: [Emacs-diffs] master 0245cc3: Improve accessibility of window dividers. (Bug#20183)
Date: Thu, 02 Jul 2015 09:55:49 -0400
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/25.0.50 (gnu/linux)

> +(defgroup window-divider nil
[...]
> +(defcustom window-divider-mode nil
[..]
> +  :group 'window-divider

This :group is redundant since it refers to the last created group.

> +(define-minor-mode window-divider-mode
[...]
> +  :group 'window-divider
> +  :global t
> +  :variable (window-divider-mode
> +             . (lambda (value)
> +                 (frame--window-divider-mode-set-and-apply
> +                  (and value
> +                       (or frame--window-divider-previous-mode
> +                           (default-value 'window-divider-mode)
> +                           'right-only))))))

I'd rather not abuse the (GETTER . SETTER) here.  This is only really
needed for those cases where the value of the mode is neither kept in
a global var nor a buffer-local var (e.g. it's kept in a window or frame
parameter instead).

So, we should drop :variable, which will also let us drop the
separate defcustom.


        Stefan



reply via email to

[Prev in Thread] Current Thread [Next in Thread]