[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Allow inhibiting 'auto-save-visited-mode' on a per-buffer ba
From: |
Štěpán Němec |
Subject: |
Re: [PATCH] Allow inhibiting 'auto-save-visited-mode' on a per-buffer basis. |
Date: |
Thu, 02 Apr 2020 11:59:47 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/28.0.50 (gnu/linux) |
On Thu, 02 Apr 2020 11:47:09 +0200
Robert Pluim wrote:
>>>>>> On Thu, 2 Apr 2020 11:40:45 +0200, Philipp Stephani <address@hidden>
>>>>>> said:
>
> Philipp> Am Do., 2. Apr. 2020 um 11:25 Uhr schrieb Robert Pluim
> <address@hidden>:
> >>
> >> >>>>> On Wed, 1 Apr 2020 19:57:36 +0200, Philipp Stephani
> <address@hidden> said:
> >>
> Philipp> * lisp/files.el (auto-save-visited-inhibit): New buffer-local
> Philipp> variable.
> Philipp> (auto-save-visited-mode): Use it.
> >>
> >> Could you expand on why this is needed? What's stopping you from just
> >> turning off auto-save-visited-mode in those buffers, either directly
> >> or via local variables?
>
> Philipp> auto-save-visited-mode is a global mode, it can't be turned off
> for
> Philipp> individual buffers.
>
> Youʼre right, I saw the 'define-minor-mode' bit and missed that itʼs a
> global minor mode.
Still, wouldn't it be better to add the rationale to the commit message?
IMO there should always be some commentary or explanation, unless it's
just an obvious or trivial fix. I often find myself wondering "why was
this change made?" when browsing the commit history, and I'm sure I'm
not the only one.
--
Štěpán
Re: [PATCH] Allow inhibiting 'auto-save-visited-mode' on a per-buffer basis., Eli Zaretskii, 2020/04/02
- Re: [PATCH] Allow inhibiting 'auto-save-visited-mode' on a per-buffer basis., Stefan Monnier, 2020/04/02
- Re: [PATCH] Allow inhibiting 'auto-save-visited-mode' on a per-buffer basis., Philipp Stephani, 2020/04/05
- Re: [PATCH] Allow inhibiting 'auto-save-visited-mode' on a per-buffer basis., Stefan Monnier, 2020/04/05
- Re: [PATCH] Allow inhibiting 'auto-save-visited-mode' on a per-buffer basis., Philipp Stephani, 2020/04/05
- Re: [PATCH] Allow inhibiting 'auto-save-visited-mode' on a per-buffer basis., Stefan Monnier, 2020/04/05
- Re: [PATCH] Allow inhibiting 'auto-save-visited-mode' on a per-buffer basis., Philipp Stephani, 2020/04/05
- Re: [PATCH] Allow inhibiting 'auto-save-visited-mode' on a per-buffer basis., Philipp Stephani, 2020/04/08