unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#19062: 25.0.50; `Info-bookmark-jump' changes frames - see FIXME in code
@ 2014-11-15 15:51 Drew Adams
  2019-07-02  3:52 ` Stefan Kangas
  0 siblings, 1 reply; 5+ messages in thread
From: Drew Adams @ 2014-11-15 15:51 UTC (permalink / raw)
  To: 19062

See the FIXME in the source code.  My setup is too involved to present
here.  Suffice it to say that the `Info-bookark-jump' code messes with
the frame that was selected before popping up the Info frame (for
`Info-find-node').  (I get a separate frame for the Info buffer because
`special-display-regexps' is `("[ ]?[*][^*]+[*]")'.)


In GNU Emacs 25.0.50.1 (i686-pc-mingw32)
 of 2014-10-20 on LEG570
Bzr revision: 118168 rgm@gnu.org-20141020195941-icp42t8ttcnud09g
Windowing system distributor `Microsoft Corp.', version 6.1.7601
Configured using:
 `configure --enable-checking=yes,glyphs CPPFLAGS=-DGLYPH_DEBUG=1'





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

* bug#19062: 25.0.50; `Info-bookmark-jump' changes frames - see FIXME in code
  2014-11-15 15:51 bug#19062: 25.0.50; `Info-bookmark-jump' changes frames - see FIXME in code Drew Adams
@ 2019-07-02  3:52 ` Stefan Kangas
  2020-08-23  1:45   ` Stefan Kangas
  2020-08-23  4:40   ` Drew Adams
  0 siblings, 2 replies; 5+ messages in thread
From: Stefan Kangas @ 2019-07-02  3:52 UTC (permalink / raw)
  To: 19062

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

> See the FIXME in the source code.  My setup is too involved to present
> here.  Suffice it to say that the `Info-bookark-jump' code messes with
> the frame that was selected before popping up the Info frame (for
> `Info-find-node').  (I get a separate frame for the Info buffer because
> `special-display-regexps' is `("[ ]?[*][^*]+[*]")'.)

I don't understand the problem here.  I set the following, according to
your suggestion:

(setq special-display-regexps '("[ ]?[*][^*]+[*]"))

And I've experimented a bit with jumping to info bookmarks without
seeing anything unusual.

Thanks,
Stefan Kangas





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

* bug#19062: 25.0.50; `Info-bookmark-jump' changes frames - see FIXME in code
  2019-07-02  3:52 ` Stefan Kangas
@ 2020-08-23  1:45   ` Stefan Kangas
  2020-08-23  4:40   ` Drew Adams
  1 sibling, 0 replies; 5+ messages in thread
From: Stefan Kangas @ 2020-08-23  1:45 UTC (permalink / raw)
  To: Drew Adams; +Cc: 19062

Stefan Kangas <stefan@marxist.se> writes:

> Drew Adams <drew.adams@oracle.com> writes:
>
>> See the FIXME in the source code.  My setup is too involved to present
>> here.  Suffice it to say that the `Info-bookark-jump' code messes with
>> the frame that was selected before popping up the Info frame (for
>> `Info-find-node').  (I get a separate frame for the Info buffer because
>> `special-display-regexps' is `("[ ]?[*][^*]+[*]")'.)
>
> I don't understand the problem here.  I set the following, according to
> your suggestion:
>
> (setq special-display-regexps '("[ ]?[*][^*]+[*]"))
>
> And I've experimented a bit with jumping to info bookmarks without
> seeing anything unusual.

Drew, do you have a way to reproduce this error?  (It seems like you
weren't in the Cc for my last message sent over a year ago.)

Best regards,
Stefan Kangas





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

* bug#19062: 25.0.50; `Info-bookmark-jump' changes frames - see FIXME in code
  2019-07-02  3:52 ` Stefan Kangas
  2020-08-23  1:45   ` Stefan Kangas
@ 2020-08-23  4:40   ` Drew Adams
  2020-09-02 15:19     ` Stefan Kangas
  1 sibling, 1 reply; 5+ messages in thread
From: Drew Adams @ 2020-08-23  4:40 UTC (permalink / raw)
  To: Stefan Kangas, 19062

> > See the FIXME in the source code.  My setup is too involved to present
> > here.  Suffice it to say that the `Info-bookark-jump' code messes with
> > the frame that was selected before popping up the Info frame (for
> > `Info-find-node').  (I get a separate frame for the Info buffer because
> > `special-display-regexps' is `("[ ]?[*][^*]+[*]")'.)
> 
> I don't understand the problem here.  I set the following, according to
> your suggestion:
> 
> (setq special-display-regexps '("[ ]?[*][^*]+[*]"))
> 
> And I've experimented a bit with jumping to info bookmarks without
> seeing anything unusual.

Sorry, but a year later I don't recall the details.

The "FIXME" is maybe self-explanatory:
`save-window-excursion' can't restore the window config
of the frame that was selected before, if a different
frame is selected afterward.  It does this:

(let ((,c (current-window-configuration)))
   (unwind-protect (progn ,@body)
     (set-window-configuration ,c)))

If BODY selects a different frame then the
`set-window-configuration' is evaluated there,
which can't be right.

I don't know now whether the report was just
to point out the FIXME or whether it affected me.

I do get a separate frame for Info buffers, but
I don't know what to do to check whether the
window config in the original buffer doesn't
get restored correctly (I don't see how it could)
or whether trying to restore it in the newly
selected frame is problematic (I don't notice that
it is, but I don't fiddle with window configs, so
I don't think I'd notice, if it did).

I think a proper test would need to involve doing
something additional to the handler, to first
modify the original window config, then let the
new frame get selected, and see that the original
window config didn't get restored (it couldn't,
AFAICT).

I think the point of the bug report was probably
just to point to the FIXME.  Sorry I've forgotten.





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

* bug#19062: 25.0.50; `Info-bookmark-jump' changes frames - see FIXME in code
  2020-08-23  4:40   ` Drew Adams
@ 2020-09-02 15:19     ` Stefan Kangas
  0 siblings, 0 replies; 5+ messages in thread
From: Stefan Kangas @ 2020-09-02 15:19 UTC (permalink / raw)
  To: Drew Adams; +Cc: 19062-done

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

> I think the point of the bug report was probably
> just to point to the FIXME.  Sorry I've forgotten.

No worries, it happens to the best of us.  I think it's enough to track
that FIXME in the code, and someone will hopefully look into it at some
undefined point in the future.  That's generally what happens with
FIXME:s, I guess, until they become an actual issue.

I'm therefore closing this bug report now.  If you encounter any issues
with this functionality, feel free to reopen or open a new bug.  Thanks.





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

end of thread, other threads:[~2020-09-02 15:19 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-11-15 15:51 bug#19062: 25.0.50; `Info-bookmark-jump' changes frames - see FIXME in code Drew Adams
2019-07-02  3:52 ` Stefan Kangas
2020-08-23  1:45   ` Stefan Kangas
2020-08-23  4:40   ` Drew Adams
2020-09-02 15:19     ` Stefan Kangas

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).