GNU bug report logs -
#24675
26.0.50; Misleading suggestion in Ediff Info manual
Previous Next
Reported by: Philipp Stephani <p.stephani2 <at> gmail.com>
Date: Wed, 12 Oct 2016 15:56:01 UTC
Severity: minor
Found in version 26.0.50
Done: Philipp Stephani <p.stephani2 <at> gmail.com>
Bug is archived. No further changes may be made.
Full log
Message #16 received at 24675-done <at> debbugs.gnu.org (full text, mbox):
[Message part 1 (text/plain, inline)]
Philipp Stephani <p.stephani2 <at> gmail.com> schrieb am So., 18. Dez. 2016 um
22:25 Uhr:
> Glenn Morris <rgm <at> gnu.org> schrieb am Di., 13. Dez. 2016 um 19:03 Uhr:
>
> Philipp Stephani wrote:
>
> > The Ediff Info manual, in section 7.1 "Hooks", says:
> >
> > 'ediff-cleanup-hook'
> > This hook is run just before 'ediff-quit-hook'. This is a good
> > place to do various cleanups, such as deleting the variant buffers.
> > Ediff provides a function, 'ediff-janitor', as one such possible
> > hook, which you can add to 'ediff-cleanup-hook' with 'add-hook'.
> >
> > However, ediff-cleanup-hook is a normal hook, and ediff-janitor is a
> > function taking two required arguments, so it cannot be added to
> > ediff-cleanup-hook. The manual should e.g. state that a lambda wrapping
> > ediff-janitor can be added to ediff-cleanup-hook, but not ediff-janitor
> > itself.
>
> It seems the doc hasn't been updated in the past 14 years.
> Before 50a07e1, there was a single optional argument.
> You might be one of the few users of this functionality, so maybe you
> would like to fix the doc yourself?
>
>
> I don't use it myself, but I got a report from a colleague who interpreted
> the manual such that you could add ediff-janitor directly to the hook with
> add-hook. I've attached a patch to clarify this. I'll push it to emacs-25
> in a few days until there are complaints.
>
Pushed as f69bd7954f113719bb0e46e143231744c80bef34.
[Message part 2 (text/html, inline)]
This bug report was last modified 8 years and 202 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.