all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
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".





  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.