GNU bug report logs - #19328
[PATCH] Add mechanism to prompt about unsaved customizations

Previous Next

Full log

View this message in rfc822 format

From: Cameron Desautels <camdez <at> gmail.com>
To: Drew Adams <drew.adams <at> oracle.com>
Cc: 19328 <at> debbugs.gnu.org
Subject: bug#19328: [PATCH] Add mechanism to prompt about unsaved customizations
Date: Tue, 9 Dec 2014 13:47:56 -0600

> FWIW, These are the changes I made to `customize-unsaved':

> ;; 1. By default, ignores preferences in `customize-customized-ignore'.
> ;; 2. Added prefix arg to override `customize-customized-ignore'.
> ;; 3. When not interactive and there are changes, ask for confirmation.
> ;; 4. Always returns `t', so it can be used as a
> ;;    `kill-emacs-query-functions' hook.
> ;; 5. Wrap in `condition-case' and reissue error.

Thank you for sharing that!  I looked through it to see if I'm missing
anything important.  Your code is much more ambitious than the humble
patch I'm offering.  I definitely see the rationale behind your work
on `cus-edit+.el' but I think the important parts of it are orthogonal
to what I've done here.

WRT the five items above, 1 & 2 can be disregarded here because
`customize-customized-ignore' only exists in your library (and is only
necessary because of the elimination of the "changed outside of
Customize" state).

3-5 I cover differently--a new function rather than changing the
behavior of `customize-unsaved'--and this way has the benefit of being
minimally invasive (i.e. shouldn't break anyone else's code).

TLDR I still think it makes sense to merge the patch.

-- 
Cameron Desautels <camdez <at> gmail.com>

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.