all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Stefan Monnier <monnier@iro.umontreal.ca>
To: Kelly Dean <kellydeanch@yahoo.com>
Cc: emacs-devel@gnu.org
Subject: Re: [PATCH] Desktop mode saves mark-ring too verbosely
Date: Sat, 23 Nov 2013 08:40:23 -0500	[thread overview]
Message-ID: <jwviovjz1ag.fsf-monnier+emacs@gnu.org> (raw)
In-Reply-To: <1369194255.41567.YahooMailClassic@web141101.mail.bf1.yahoo.com> (Kelly Dean's message of "Tue, 21 May 2013 20:44:15 -0700 (PDT)")

> You can (add-to-list 'desktop-locals-to-save 'mark-ring) and it works, so
> long as you have the fix for bug 13951 applied. But it's very verbose; for
> each element of mark-ring for each buffer, it writes an entry like
> (let ((mk (make-marker))) (add-hook 'desktop-delay-hook (list 'lambda '()
> (list 'set-marker mk 123 '(get-buffer "init.el")))) mk)

> @@ -694,7 +694,8 @@
>  	   (when (member (car locals) loclist)
>  	     (setq ll (cons (car locals) ll)))))
>         (setq locals (cdr locals)))
> -     ll)))
> +     ll)
> +   (mapcar 'marker-position mark-ring)))

Having special-case code to print&read some variables is not in itself
a bad idea, but I think we should do it via a table mapping variables to
ad-hoc massaging functions.


        Stefan



  reply	other threads:[~2013-11-23 13:40 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-05-22  3:44 [PATCH] Desktop mode saves mark-ring too verbosely Kelly Dean
2013-11-23 13:40 ` Stefan Monnier [this message]
2015-01-21 12:11   ` Kelly Dean
2015-01-21 15:04     ` Stefan Monnier
2015-01-22  5:43       ` Kelly Dean
2015-01-22  8:20         ` Ivan Shmakov
2015-01-23 13:20           ` Kelly Dean
2015-01-23 14:09             ` Ivan Shmakov
2015-01-24  3:08             ` Stephen J. Turnbull
2015-01-24 23:30               ` Elisp terminology (was: Re: [PATCH] Desktop mode saves mark-ring too verbosely) Kelly Dean
2015-01-25  9:49                 ` Elisp terminology David Kastrup
  -- strict thread matches above, loose matches on Subject: below --
2015-01-23  2:59 Rant - Elisp terminology is deceptive Kelly Dean
2015-01-23 20:15 ` Stefan Monnier
2015-01-24  0:41   ` Kelly Dean
2015-01-24  0:48     ` Óscar Fuentes
2015-01-24  3:28     ` Stephen J. Turnbull
2015-01-24  8:51       ` Eli Zaretskii
2015-01-24 10:32         ` Kelly Dean
2015-01-24 11:26           ` Eli Zaretskii
2015-01-24 10:30       ` Kelly Dean
2015-01-24 11:03         ` David Kastrup
2015-01-24 23:24           ` Kelly Dean
2015-01-25  9:16             ` David Kastrup
2015-01-26  3:52               ` Kelly Dean
2015-01-26  8:28                 ` Thien-Thi Nguyen

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=jwviovjz1ag.fsf-monnier+emacs@gnu.org \
    --to=monnier@iro.umontreal.ca \
    --cc=emacs-devel@gnu.org \
    --cc=kellydeanch@yahoo.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.