unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
From: Stefan Monnier <monnier@iro.umontreal.ca>
To: martin rudalics <rudalics@gmx.at>
Cc: Michael Bach <phaebz@gmail.com>, 10348@debbugs.gnu.org
Subject: bug#10348: 24.0.92; Save and load window states
Date: Fri, 23 Dec 2011 19:07:01 -0500	[thread overview]
Message-ID: <jwvk45myhj8.fsf-monnier+emacs@gnu.org> (raw)
In-Reply-To: <4EF45888.3030204@gmx.at> (martin rudalics's message of "Fri, 23 Dec 2011 11:31:36 +0100")

>>> (1) Do not save the `clone-of' parameter.  It's not yet useful anyway.
>>> This will not solve the more general problem mentioned above.
>> As mentioned, it's not a really convincing solution.
> It would allow us to silently ignore this problem until 24.2.  And it
> would obviously allow Michael to go on with his work.  His feedback
> would be overly welcome here.

No, it would address this particular bug-report, but similar problems
may reappear at any time.

> IIUC this is what desktop does.  The problem is rather that we would
> have to distinguish between values needed for intra-session purposes and
> those that make sense for inter-session purposes too.

I'm not sure distinguishing the two is needed (especially for
window-state-*).

>>> Unless you have a better suggestion I'll apply (1) for Emacs 24.1 and
>>> try to propose a combination of (3) and (4) for later releases.
>> I think the best course of action is to only save the window
>> parameters listed in some variable (window-state-saved-parameters?).
> You mean that anyone who misuses that variable (by including, for
> example, a parameter that actually stores a window object as value)
> would be on her own?

I don't see any harm in it.

> Doesn't look so attractive to me since the effect will only be seen in
> a new session, some time after the problematic save happened.

Not if we make this variable specify which parameters to include in
window-states, rather than only which parameters to write to a file.
Or maybe I don't understand the problem you're referring to.

>> Actually, it might even be good to filter which params to keep not just
>> when saving but already when constructing the window-state object (some
>> params may simply not belong in a window-state object because restoring
>> them would make no sense).
> Agreed.

>> After all the window-configurations don't save&restore
>> window parameters.
> Currently they do (unless you modify them destructively).  Otherwise,
> side windows and atomic windows won't work.

Oh, I see that's another change in Emacs-24.  It's actually problematic
because set-window-parameter does operate destructively, so it makes the
semantics rather irregular.


        Stefan





  parent reply	other threads:[~2011-12-24  0:07 UTC|newest]

Thread overview: 34+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-12-21 20:41 bug#10348: 24.0.92; Save and load window states Michael Bach
2011-12-22 17:04 ` martin rudalics
2011-12-22 17:14   ` Drew Adams
2011-12-22 21:57   ` Juri Linkov
2011-12-22 22:57     ` Juanma Barranquero
2011-12-22 23:34       ` Juri Linkov
2011-12-23 10:30         ` martin rudalics
2011-12-23 15:10           ` Drew Adams
2011-12-24 19:04             ` Juri Linkov
2011-12-23 10:30     ` martin rudalics
2011-12-23  1:03   ` Stefan Monnier
2011-12-23 10:31     ` martin rudalics
2011-12-23 11:30       ` Michael Bach
2011-12-23 21:14       ` Juri Linkov
2011-12-24  9:26         ` martin rudalics
2011-12-24 19:08           ` Juri Linkov
2011-12-25 13:57             ` martin rudalics
2011-12-24  0:07       ` Stefan Monnier [this message]
2011-12-24  9:27         ` martin rudalics
2011-12-25 11:00           ` Stefan Monnier
2011-12-25 13:58             ` martin rudalics
2011-12-25 21:36               ` Juri Linkov
2011-12-26 11:07                 ` martin rudalics
2011-12-26  0:22               ` Stefan Monnier
2011-12-26 11:07                 ` martin rudalics
2011-12-26 18:25                   ` martin rudalics
2011-12-27 23:23                     ` Stefan Monnier
2011-12-28 15:57                       ` martin rudalics
2011-12-28 23:09                         ` Stefan Monnier
2011-12-29 11:39                           ` martin rudalics
2012-01-16  9:42                       ` martin rudalics
2011-12-28  9:50                     ` martin rudalics
2011-12-24 19:17 ` Juri Linkov
2012-01-16  9:43 ` martin rudalics

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

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=jwvk45myhj8.fsf-monnier+emacs@gnu.org \
    --to=monnier@iro.umontreal.ca \
    --cc=10348@debbugs.gnu.org \
    --cc=phaebz@gmail.com \
    --cc=rudalics@gmx.at \
    /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 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).