From: Drew Adams <drew.adams@oracle.com>
To: martin rudalics <rudalics@gmx.at>
Cc: Leo Liu <sdl.web@gmail.com>, 16038@debbugs.gnu.org
Subject: bug#16038: 24.3; latest change to with-help-window makes temp-buffer-browse useless
Date: Sun, 26 Jan 2014 09:52:08 -0800 (PST) [thread overview]
Message-ID: <ab3e0250-a794-41c2-996a-db673099131c@default> (raw)
In-Reply-To: <52E4F050.9010502@gmx.at>
> > And just what problem is that? You say there is a problem to
> > fix in my code. What is the problem?
>
> The one from http://debbugs.gnu.org/cgi/bugreport.cgi?bug=8368#32.
That demo example was not from my code, as you know. It
demonstrates bug #8368, showing why it needs to be fixed.
And it is still pertinent, AFAIK.
> > (And what is the fix you have in mind?)
And the fix is?
> >> > If bug #8368 is now fixed, great: What code to replace
> >> > with what code, concretely?
> >>
> >> Your code that uses `with-output-to-temp-buffer'.
> >
> > Why should that be replaced? And if it should, just what
> > should it to be replaced with?
>
> With `with-temp-buffer-window'.
How so? Is it a one-one replacement? What about the hooks?
If `with-temp-buffer-window' is supposed to be the replacement for
`with-output-to-temp-buffer' then that needs to be stated clearly
in the NEWS.
Including a spec of what the replacement should be for different
`with-output-to-temp-buffer' input patterns (formal parameters).
And including hook use (correspondences). With any significant
differences and limitations pointed out.
That is how to help users transition from the old to the new.
I imagine that you are well aware of that, but it's perhaps
better not to guess.
Consider by contrast this NEWS entry, which helps users
understand how to use `cl-flet' by pointing out how it differs
from (Emacs Lisp) `flet':
*** `cl-flet' is not like `flet' (which is deprecated).
Instead it obeys the behavior of Common-Lisp's `flet'.
In particular, in cl-flet function definitions are lexically
scoped, whereas in flet the scoping is dynamic.
> > That is part of deprecating something:
> > tell users what they should change to what in existing
> > code. E.g.,
> > (old-foo this that) ==>
> > (new-bar something-else that 42 (1- this))
> > or whatever.
>
> I don't propose to deprecate something.
It seems to me that that is part of what bug #16038, and this
discussion, are about. It is also part of what bug #8368 is
about.
You ask us to reread #8368, to guess how your new macro responds
to that bug. That bug speaks specifically about deprecating
`with-output-to-temp-buffer'.
And in that context the deprecation would involve a simple
renaming (one-one).
> I offer a solution to a problem.
What is the solution, beyond chanting the mantra
`with-temp-buffer-window'? What `with-output-to-temp-buffer'
patterns map to what `with-temp-buffer-window' patterns?
What about the various hooks?
> >> Maybe Leo would have started to do that if you had not
> >> interfered. I suggest you apologize and consult with him
> >> on how to proceed on this matter.
>
> > Suggest all you want, if it makes you feel better. But I
> > do not feel that I interfered in any way. Nor do I think
> > I have anything to apologize for.
>
> Then better don't ask for suggestions.
No one asked for ad hominem, off-the-wall "suggestions".
Of course, you elided your actual suggestion (restored here),
as well as what I said about it (restored here), giving the
impression that it might have actually been a helpful, technical
suggestion that I objected to when I said, "Suggest all you want".
next prev parent reply other threads:[~2014-01-26 17:52 UTC|newest]
Thread overview: 39+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-12-03 14:34 bug#16038: 24.3; latest change to with-help-window makes temp-buffer-browse useless Leo Liu
2014-01-11 14:01 ` martin rudalics
2014-01-11 14:32 ` Leo Liu
2014-01-11 15:41 ` martin rudalics
2014-01-11 16:31 ` Leo Liu
2014-01-12 9:53 ` martin rudalics
2014-01-14 0:23 ` Leo Liu
2014-01-14 1:47 ` Stefan Monnier
2014-01-14 4:42 ` Leo Liu
2014-01-14 21:38 ` Stefan Monnier
2014-01-14 21:55 ` Drew Adams
2014-01-14 23:46 ` Leo Liu
2014-01-22 2:14 ` Leo Liu
2014-01-22 3:02 ` Stefan Monnier
2014-01-22 3:32 ` Leo Liu
2014-01-22 13:44 ` Stefan Monnier
2014-01-23 10:03 ` Leo Liu
2014-01-23 17:48 ` Drew Adams
2014-01-24 7:30 ` Leo Liu
2014-01-24 15:06 ` Drew Adams
2014-01-24 15:45 ` martin rudalics
2014-01-24 16:29 ` Drew Adams
2014-01-25 9:21 ` martin rudalics
2014-01-25 16:34 ` Drew Adams
2014-01-25 16:48 ` martin rudalics
2014-01-25 17:33 ` Drew Adams
2014-01-25 18:25 ` martin rudalics
2014-01-25 20:19 ` Drew Adams
2014-01-26 11:24 ` martin rudalics
2014-01-26 17:52 ` Drew Adams [this message]
2014-01-27 8:21 ` martin rudalics
2014-01-27 15:14 ` Drew Adams
2014-01-22 3:07 ` Glenn Morris
2014-07-08 1:54 ` Glenn Morris
2014-07-08 2:04 ` Glenn Morris
2014-07-08 5:59 ` Glenn Morris
2014-07-08 13:57 ` Drew Adams
2014-07-08 22:24 ` Leo Liu
2014-07-10 7:20 ` Glenn Morris
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=ab3e0250-a794-41c2-996a-db673099131c@default \
--to=drew.adams@oracle.com \
--cc=16038@debbugs.gnu.org \
--cc=rudalics@gmx.at \
--cc=sdl.web@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.