all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* bug#17873: 24.4.50; `desktop-save'
@ 2014-06-29 16:47 Drew Adams
  2014-06-29 23:59 ` Juri Linkov
  0 siblings, 1 reply; 4+ messages in thread
From: Drew Adams @ 2014-06-29 16:47 UTC (permalink / raw)
  To: 17873

1. The doc string says:

 If AUTO-SAVE is non-nil, compare the saved contents to the one last saved,
 and don't save the buffer if they are the same.

Saved contents of what?  One what last saved?  Saved contents cannot be
compared to "the one" anything.

I think you mean "compare the current desktop information to that in the
desktop file."  If you are avoiding saving (because the contents are the
same) then you cannot speak of comparing the "saved contents" - the
contents have not yet been saved.

And what buffer?  This is about saving a desktop in a desktop-file.
There is no notion of a buffer here - at the user level.  The code puts
the desktop info into a buffer (which then might get saved), but that is
implementation-level.

What `desktop-save' does needs to be described at a user level (what,
not how, unless that is important to users).  Just say that if the
desktop information has not changed since it was last saved then the
file is not rewritten.

2. I also have a question about the behavior: Why is writing the
file even when the content is unchanged the default behavior?  Why the
need to specify AUTO-SAVE instead of an optional SAVE-EVEN-IF-NO-CHANGE?
Is this just for backward compatibility?  (Before AUTO-SAVE was
introduced the behavior was to update the file even if the desktop
info was not changed.)

In GNU Emacs 24.4.50.1 (i686-pc-mingw32)
 of 2014-06-17 on ODIEONE
Bzr revision: 117359 monnier@iro.umontreal.ca-20140617193358-2t1nl1te9gc2mqrx
Windowing system distributor `Microsoft Corp.', version 6.1.7601
Configured using:
 `configure --prefix=/c/Devel/emacs/snapshot/trunk
 --enable-checking=yes,glyphs 'CFLAGS=-O0 -g3'
 LDFLAGS=-Lc:/Devel/emacs/lib 'CPPFLAGS=-DGC_MCHECK=1
 -Ic:/Devel/emacs/include''





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

end of thread, other threads:[~2014-07-02 23:47 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-06-29 16:47 bug#17873: 24.4.50; `desktop-save' Drew Adams
2014-06-29 23:59 ` Juri Linkov
2014-06-30  2:10   ` Drew Adams
2014-07-02 23:47     ` Juri Linkov

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.