From: Eli Zaretskii <eliz@gnu.org>
To: Philipp Stephani <p.stephani2@gmail.com>
Cc: jixiuf@qq.com, emacs-devel@gnu.org
Subject: Re: make pred customable in auto-save-visited-mode
Date: Sat, 31 Mar 2018 13:42:25 +0300 [thread overview]
Message-ID: <837eps8r1q.fsf@gnu.org> (raw)
In-Reply-To: <CAArVCkTmCuEumgqg3fR-r_Xsu8sWtFK89g489SYWV+JGMbV-qQ@mail.gmail.com> (message from Philipp Stephani on Sat, 31 Mar 2018 10:23:13 +0000)
> From: Philipp Stephani <p.stephani2@gmail.com>
> Date: Sat, 31 Mar 2018 10:23:13 +0000
> Cc: 纪秀峰 <jixiuf@qq.com>, emacs-devel@gnu.org
>
> Eli Zaretskii <eliz@gnu.org> schrieb am Sa., 31. März 2018 um 10:44 Uhr:
>
> > From: 纪秀峰 <jixiuf@qq.com>
> > Date: Sat, 31 Mar 2018 00:59:19 +0800
> > Feedback-ID: esmtp:qq.com:bgforeign:bgforeign1
> >
> > When I enable auto-save-visited-mode and start an ediff merge session
> > It always prompt me save "*ediff-merge*” to a file.
>
> Why is that a problem? *ediff-merge* is a buffer where you prepare
> the merge, and that merge will eventually be saved to some file,
> right? So it doesn't sound wrong to offer to save it.
>
> However, auto-saving should probably not prompt for filenames. That's at least the intention of the :noprompt
> argument here.
I think you misinterpret the question with which Emacs prompts, see
below.
> The issue might be in `save-some-buffers': Its docstring says "If PRED is t, then certain non-file buffers will
> also be considered.", but later it only checks whether PRED is non-nil:
The question with which Emacs prompts is not the one controlled by
PRED, it's a question about the file name in which to save, I think.
It comes from basic-save-buffer, and that happens because
*ediff-merge* doesn't have a buffer-file-name.
So if we make the change you suggest, then Emacs will not offer to
save *ediff-merge* at all when this minor mode is turned on. Is that
what we want?
Also, I don't think there's a problem with t and non-t non-nil values:
the doc string doesn't document such values that are not functions.
So there's no problem with checking that PRED is non-nil.
prev parent reply other threads:[~2018-03-31 10:42 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-03-30 16:59 make pred customable in auto-save-visited-mode 纪秀峰
2018-03-31 8:43 ` Eli Zaretskii
2018-03-31 10:23 ` Philipp Stephani
2018-03-31 10:28 ` Philipp Stephani
2018-03-31 10:45 ` Eli Zaretskii
2018-04-01 17:19 ` Philipp Stephani
2018-04-01 17:35 ` Eli Zaretskii
2018-04-01 19:51 ` Philipp Stephani
2018-04-02 4:44 ` Eli Zaretskii
2018-04-02 10:20 ` Philipp Stephani
2018-03-31 10:42 ` Eli Zaretskii [this message]
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=837eps8r1q.fsf@gnu.org \
--to=eliz@gnu.org \
--cc=emacs-devel@gnu.org \
--cc=jixiuf@qq.com \
--cc=p.stephani2@gmail.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.