all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* bug#26243: 25.1; `dired-pop-to-buffer' warning
@ 2017-03-24 18:21 Drew Adams
  2019-07-26 11:13 ` Lars Ingebrigtsen
  0 siblings, 1 reply; 7+ messages in thread
From: Drew Adams @ 2017-03-24 18:21 UTC (permalink / raw)
  To: 26243

If code uses function `dired-pop-to-buffer', this is the byte-compiler
warning you get:

`dired-pop-to-buffer' is an obsolete function (as of 24.3);
use `dired-mark-pop-up' instead.
    ^^^^^^^^^^^^^^^^^^^

That is a ridiculous warning.  `dired-mark-pop-up' is about acting on
each of the marked files and directories.  `dired-pop-to-buffer' pops
up a single buffer.

There is zero connection between `dired-mark-pop-up' and
`dired-pop-to-buffer'.


In GNU Emacs 25.1.1 (x86_64-w64-mingw32)
 of 2016-11-15
Windowing system distributor `Microsoft Corp.', version 6.1.7601
Configured using:
 `configure --without-dbus --without-compress-install 'CFLAGS=-O2
 -static -g3''





^ permalink raw reply	[flat|nested] 7+ messages in thread

* bug#26243: 25.1; `dired-pop-to-buffer' warning
  2017-03-24 18:21 bug#26243: 25.1; `dired-pop-to-buffer' warning Drew Adams
@ 2019-07-26 11:13 ` Lars Ingebrigtsen
  2019-07-29 15:16   ` martin rudalics
  0 siblings, 1 reply; 7+ messages in thread
From: Lars Ingebrigtsen @ 2019-07-26 11:13 UTC (permalink / raw)
  To: Drew Adams; +Cc: 26243

Drew Adams <drew.adams@oracle.com> writes:

> If code uses function `dired-pop-to-buffer', this is the byte-compiler
> warning you get:
>
> `dired-pop-to-buffer' is an obsolete function (as of 24.3);
> use `dired-mark-pop-up' instead.
>     ^^^^^^^^^^^^^^^^^^^
>
> That is a ridiculous warning.  `dired-mark-pop-up' is about acting on
> each of the marked files and directories.  `dired-pop-to-buffer' pops
> up a single buffer.
>
> There is zero connection between `dired-mark-pop-up' and
> `dired-pop-to-buffer'.

Those functions seem to have little to do with each other; yes.  So I've
now removed the reference.

It was added with a reference to bug 1806, which is a 170 message long
thread which I've not read.  I've Cc'd Juri (who made the change) in
case the reference makes sense anyway, in which case I'll revert my
change.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no





^ permalink raw reply	[flat|nested] 7+ messages in thread

* bug#26243: 25.1; `dired-pop-to-buffer' warning
  2019-07-26 11:13 ` Lars Ingebrigtsen
@ 2019-07-29 15:16   ` martin rudalics
  2019-07-29 22:41     ` Juri Linkov
  0 siblings, 1 reply; 7+ messages in thread
From: martin rudalics @ 2019-07-29 15:16 UTC (permalink / raw)
  To: Lars Ingebrigtsen, Drew Adams; +Cc: 26243

 > It was added with a reference to bug 1806, which is a 170 message long
 > thread which I've not read.  I've Cc'd Juri (who made the change) in
 > case the reference makes sense anyway, in which case I'll revert my
 > change.

I think the declaration should say to use 'pop-to-buffer' instead of
'dired-pop-to-buffer'.  But let's wait for Juri.

martin





^ permalink raw reply	[flat|nested] 7+ messages in thread

* bug#26243: 25.1; `dired-pop-to-buffer' warning
  2019-07-29 15:16   ` martin rudalics
@ 2019-07-29 22:41     ` Juri Linkov
  2019-07-30  0:53       ` Drew Adams
  2019-07-30 11:35       ` Lars Ingebrigtsen
  0 siblings, 2 replies; 7+ messages in thread
From: Juri Linkov @ 2019-07-29 22:41 UTC (permalink / raw)
  To: martin rudalics; +Cc: Lars Ingebrigtsen, 26243

>> It was added with a reference to bug 1806, which is a 170 message long
>> thread which I've not read.  I've Cc'd Juri (who made the change) in
>> case the reference makes sense anyway, in which case I'll revert my
>> change.
>
> I think the declaration should say to use 'pop-to-buffer' instead of
> 'dired-pop-to-buffer'.  But let's wait for Juri.

Yes, either 'pop-to-buffer' or 'dired-mark-pop-up'.  If 'dired-mark-pop-up'
doesn't suit the needs due to its narrow applicability, then a more general
'pop-to-buffer' should be used.  'dired-pop-to-buffer' was an ugly wrapper
around 'pop-to-buffer' with some additions that now should be specified
as ACTION args of 'display-buffer' (an example is in 'dired-mark-pop-up').
I don't understand why Drew still uses 'dired-pop-to-buffer'.





^ permalink raw reply	[flat|nested] 7+ messages in thread

* bug#26243: 25.1; `dired-pop-to-buffer' warning
  2019-07-29 22:41     ` Juri Linkov
@ 2019-07-30  0:53       ` Drew Adams
  2019-07-30 11:35       ` Lars Ingebrigtsen
  1 sibling, 0 replies; 7+ messages in thread
From: Drew Adams @ 2019-07-30  0:53 UTC (permalink / raw)
  To: Juri Linkov, martin rudalics; +Cc: Lars Ingebrigtsen, 26243

> Yes, either 'pop-to-buffer' or 'dired-mark-pop-up'.  If 'dired-mark-
> pop-up'
> doesn't suit the needs due to its narrow applicability, then a more
> general
> 'pop-to-buffer' should be used.  'dired-pop-to-buffer' was an ugly
> wrapper
> around 'pop-to-buffer' with some additions that now should be specified
> as ACTION args of 'display-buffer' (an example is in 'dired-mark-pop-
> up').
> I don't understand why Drew still uses 'dired-pop-to-buffer'.

I use `dired-pop-to-buffer' in more than one place
in my code (Dired+).  (I use a different definition
of it, but that's not really important here.  I also
use a different definition of `dired-mark-pop-up'.)

`dired-pop-to-buffer' shouldn't have been declared
"obsolete" in my opinion.  But in any case, surely
`dired-mark-pop-up' can't be used to do anything
like what it does.  (And no, `pop-to-buffer' isn't
an adequate replacement for it either.)

> some additions that now should be specified
> as ACTION args of 'display-buffer' (an example
> is in 'dired-mark-pop-up').

FWIW, we apparently disagree about whether using
`display-buffer' ACTION args is the way to do things.
We likely have different ideas about what is elegant
and what is ugly. ;-)  When a doc string resorts to
telling users things like this, I'd say we're already
veering off the elegant road:

 "use the Customization interface to add a new rule
  to `display-buffer-alist' where condition regexp is
  \"^ \\*Marked Files\\*$\", action argument symbol
  is `window-height' and its value is nil."






^ permalink raw reply	[flat|nested] 7+ messages in thread

* bug#26243: 25.1; `dired-pop-to-buffer' warning
  2019-07-29 22:41     ` Juri Linkov
  2019-07-30  0:53       ` Drew Adams
@ 2019-07-30 11:35       ` Lars Ingebrigtsen
  2019-07-30 19:08         ` Drew Adams
  1 sibling, 1 reply; 7+ messages in thread
From: Lars Ingebrigtsen @ 2019-07-30 11:35 UTC (permalink / raw)
  To: Juri Linkov; +Cc: 26243

Juri Linkov <juri@linkov.net> writes:

>> I think the declaration should say to use 'pop-to-buffer' instead of
>> 'dired-pop-to-buffer'.  But let's wait for Juri.
>
> Yes, either 'pop-to-buffer' or 'dired-mark-pop-up'. 

I've now done the former.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no





^ permalink raw reply	[flat|nested] 7+ messages in thread

* bug#26243: 25.1; `dired-pop-to-buffer' warning
  2019-07-30 11:35       ` Lars Ingebrigtsen
@ 2019-07-30 19:08         ` Drew Adams
  0 siblings, 0 replies; 7+ messages in thread
From: Drew Adams @ 2019-07-30 19:08 UTC (permalink / raw)
  To: Lars Ingebrigtsen, Juri Linkov; +Cc: 26243

> >> I think the declaration should say to use 'pop-to-buffer' instead of
> >> 'dired-pop-to-buffer'.  But let's wait for Juri.
> >
> > Yes, either 'pop-to-buffer' or 'dired-mark-pop-up'.
> 
> I've now done the former.

OK, so instead of a "done" that's pretty much
a "won't fix" now.

A fairer treatment would be:
(declare (obsolete nil "24.3"))

A better treatment would be to remove the
declaration (un-obsolete).






^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2019-07-30 19:08 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-03-24 18:21 bug#26243: 25.1; `dired-pop-to-buffer' warning Drew Adams
2019-07-26 11:13 ` Lars Ingebrigtsen
2019-07-29 15:16   ` martin rudalics
2019-07-29 22:41     ` Juri Linkov
2019-07-30  0:53       ` Drew Adams
2019-07-30 11:35       ` Lars Ingebrigtsen
2019-07-30 19:08         ` Drew Adams

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.