unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* bug?  mm-display-external does delete-other-windows
@ 2016-02-28 21:35 Ingo Lohmar
  2016-02-29  1:07 ` Mike Kupfer
  2016-02-29  3:23 ` Lars Ingebrigtsen
  0 siblings, 2 replies; 6+ messages in thread
From: Ingo Lohmar @ 2016-02-28 21:35 UTC (permalink / raw)
  To: emacs-devel

Hi Emacs,

Due to some other changes, I am recently affected by the "aggressive"
behavior of gnus' mm-display-external function (in mm-decode.el): When
displaying a part in its own window (not inline), it calls
(delete-other-windows), destroying my window configuration.

Is this behavior wanted when used from inside gnus?  Are mm-decode.el's
functions meant to be used by other packages as well?  In my case the
notmuch Emacs client [1] uses mm-display-part which calls
mm-display-external under certain conditions.

If it's ok to use the function (I hope it is), I would consider
(delete-other-windows) a bug.  Otherwise, are there any plans to extract
the MIME functionality into a separate package?

[1] http://notmuchmail.org



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

* Re: bug? mm-display-external does delete-other-windows
  2016-02-28 21:35 bug? mm-display-external does delete-other-windows Ingo Lohmar
@ 2016-02-29  1:07 ` Mike Kupfer
  2016-02-29  3:23 ` Lars Ingebrigtsen
  1 sibling, 0 replies; 6+ messages in thread
From: Mike Kupfer @ 2016-02-29  1:07 UTC (permalink / raw)
  To: Ingo Lohmar; +Cc: emacs-devel

Ingo Lohmar wrote:

> Are mm-decode.el's
> functions meant to be used by other packages as well?  In my case the
> notmuch Emacs client [1] uses mm-display-part which calls
> mm-display-external under certain conditions.

Hi Ingo, yes, the mm functions are documented as general-purpose.
There's at least one other package that uses them (MH-E).

I don't see any problems with MH-E when I use an external viewer.  I
guess this is because MH-E is invoking mm-display-external directly in
response to a user keystroke, rather than going through mm-display-part.
And MH-E passes in a string, not a function, as the method.

I don't know how much the above helps you, but I wanted to reassure you
that the mm layer is supposed to be general-purpose.

regards,
mike



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

* Re: bug?  mm-display-external does delete-other-windows
  2016-02-28 21:35 bug? mm-display-external does delete-other-windows Ingo Lohmar
  2016-02-29  1:07 ` Mike Kupfer
@ 2016-02-29  3:23 ` Lars Ingebrigtsen
  2016-03-01 18:58   ` Ingo Lohmar
  1 sibling, 1 reply; 6+ messages in thread
From: Lars Ingebrigtsen @ 2016-02-29  3:23 UTC (permalink / raw)
  To: Ingo Lohmar; +Cc: emacs-devel

Ingo Lohmar <i.lohmar@gmail.com> writes:

> If it's ok to use the function (I hope it is), I would consider
> (delete-other-windows) a bug.

Yes, I think it's a bug.  Please report it with `report-emacs-bug'.

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



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

* Re: bug?  mm-display-external does delete-other-windows
  2016-02-29  3:23 ` Lars Ingebrigtsen
@ 2016-03-01 18:58   ` Ingo Lohmar
  2016-03-02 16:41     ` Lars Ingebrigtsen
  0 siblings, 1 reply; 6+ messages in thread
From: Ingo Lohmar @ 2016-03-01 18:58 UTC (permalink / raw)
  To: emacs-devel

On Mon, Feb 29 2016 14:23 (+1100), Lars Ingebrigtsen wrote:

> Ingo Lohmar <i.lohmar@gmail.com> writes:
>
>> If it's ok to use the function (I hope it is), I would consider
>> (delete-other-windows) a bug.
>
> Yes, I think it's a bug.  Please report it with `report-emacs-bug'.

[I erroneously sent this as a private mail at first, sorry.]

Reported as #22861.  I cannot think of a scenario where
(delete-other-windows) would be ok --- I will just remove it as the fix
(if nobody objects here).

I assume it's still correct to do this on emacs-25, or isn't it?



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

* Re: bug?  mm-display-external does delete-other-windows
  2016-03-01 18:58   ` Ingo Lohmar
@ 2016-03-02 16:41     ` Lars Ingebrigtsen
  2016-03-05 11:15       ` Ingo Lohmar
  0 siblings, 1 reply; 6+ messages in thread
From: Lars Ingebrigtsen @ 2016-03-02 16:41 UTC (permalink / raw)
  To: Ingo Lohmar; +Cc: emacs-devel

Ingo Lohmar <i.lohmar@gmail.com> writes:

> Reported as #22861.  I cannot think of a scenario where
> (delete-other-windows) would be ok --- I will just remove it as the fix
> (if nobody objects here).

It looks like the intention is to do the window manipulation if the
function is called from Gnus.  I don't know whether that's a good thing
to do from Gnus or not, though, but just removing it is probably the
wrong thing to do.

If this is something that should be done, then Gnus should pass
something in to make all that...  happen...

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



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

* Re: bug?  mm-display-external does delete-other-windows
  2016-03-02 16:41     ` Lars Ingebrigtsen
@ 2016-03-05 11:15       ` Ingo Lohmar
  0 siblings, 0 replies; 6+ messages in thread
From: Ingo Lohmar @ 2016-03-05 11:15 UTC (permalink / raw)
  To: Lars Ingebrigtsen; +Cc: emacs-devel

On Wed, Mar 02 2016 16:41 (+0000), Lars Ingebrigtsen wrote:

> It looks like the intention is to do the window manipulation if the
> function is called from Gnus.  I don't know whether that's a good thing
> to do from Gnus or not, though, but just removing it is probably the
> wrong thing to do.
>
> If this is something that should be done, then Gnus should pass
> something in to make all that...  happen...

Sooooo...  that would mean only somebody who uses Gnus, or better yet,
has intimate knowledge of its inner workings (wink, wink) can take care
of this bug?

I'll keep my private hack then, for the time being.

Thanks,
Ingo



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

end of thread, other threads:[~2016-03-05 11:15 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-02-28 21:35 bug? mm-display-external does delete-other-windows Ingo Lohmar
2016-02-29  1:07 ` Mike Kupfer
2016-02-29  3:23 ` Lars Ingebrigtsen
2016-03-01 18:58   ` Ingo Lohmar
2016-03-02 16:41     ` Lars Ingebrigtsen
2016-03-05 11:15       ` Ingo Lohmar

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).