Eli Zaretskii <
address@hidden> schrieb am Fr., 29. Dez. 2017 um 17:15 Uhr:
> From: Michael Heerdegen <address@hidden>
> Cc: address@hidden, address@hidden, address@hidden
> Date: Fri, 29 Dec 2017 15:08:41 +0100
>
> Eli Zaretskii <address@hidden> writes:
>
> > Does this warning pop up during bootstrap, and if so, how many times?
>
> It currently would pop up five times:
>
> | ../lisp/electric.el:Warning: Warning: obsolete gv-setter: ‘buffer-local-value’
> | --------------------
> | ../lisp/electric.el:Warning: Warning: obsolete gv-setter: ‘buffer-local-value’
> | --------------------
> | electric.el:350:40:Warning: Warning: obsolete gv-setter: ‘buffer-local-value’
> | --------------------
> | electric.el:580:39:Warning: Warning: obsolete gv-setter: ‘buffer-local-value’
> | --------------------
> | elec-pair.el:608:38:Warning: Warning: obsolete gv-setter: ‘buffer-local-value’
>
>
> Yes, these would need to be treated.
Can we treat them as part of fixing this issue?
Yes, but I think changing them should be a separate commit because it's not straightforward.
These are all modes that can be locally or globally enabled. I think typically such modes would be defined using `define-minor-mode` and `define-globalized-minor-mode`, but the electric modes are defined the other way round, i.e. the main modes are global, and then there are local modes that use `buffer-local-value` as mode variable. I'd suggest to turn this around to use `define-minor-mode` for the local modes and `define-globalized-minor-mode` for the global ones. Would that have any downsides?