GNU bug report logs -
#11520
24.1.50; delete-selection-mode conflicts with electric-pair-mode
Previous Next
Reported by: Simon Law <sfllaw <at> sfllaw.ca>
Date: Sat, 19 May 2012 20:10:01 UTC
Severity: normal
Found in version 24.1.50
Done: Stefan Monnier <monnier <at> iro.umontreal.ca>
Bug is archived. No further changes may be made.
Full log
Message #59 received at 11520-done <at> debbugs.gnu.org (full text, mbox):
> Yeah, I wasn't sure about delete-selection-pre-hook, but that's what
> the function was already called, so I wasn't going to change the API
> there.
Yes, the function name is better left as is (and really isn't bad), the
error was in the docstring (where you described the function as
a normal hook).
>> And the symbol symbol is named with "-hook" (rather than with
>> "-hooks") for normal hooks and "-functions" for abnormal hooks.
> Ah, I read that it was either one or the other in
> http://www.gnu.org/software/emacs/manual/html_node/elisp/Hooks.html.
Historically, "-hooks" was used, but they've (all?) been renamed to "-hook".
> Perhaps that documentation should discourage the -hooks convention?
Yes, thanks, the wording wasn't clear enough. Fixed.
>> - there's no self-insert-iso in Emacs, so I'd rather not touch it.
> delete-selection-mode.el used to touch self-insert-iso, as does
> cua-rect.el. Is this something to clean up, or is there something I'm
> missing?
I don't know where this self-insert-iso comes from, so we should just
remove it, but I'll keep this change for after the feature freeze.
For now, I simply left the setting as it used to be.
> Finally, I noticed that you created self-insert-uses-region-functions,
> but there is no defvar or docstring for that. For future reference, is
> it normal that packages have hidden hooks like that?
No, it's not normal. Thanks for catching this missing commit.
Stefan
This bug report was last modified 10 years and 359 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.