Re: [Emacs-diffs] scratch/bug19328/custom-prompt-customize-unsaved-options-rc2 e9e8f01: Provide custom-prompt-customize-unsaved-options.

[Ted Zlatanov <tzz@lifelogs.com>]

On Sun, 14 Dec 2014 09:36:00 -0500 Stefan Monnier <monnier@iro.umontreal.ca> wrote: 

SM> Looks good, with the following nitpicks
>> * lisp/cus-edit.el (custom-prompt-customize-unsaved-options): Add a
>> mechanism for prompting user about unsaved customizations.

SM> These two lines mean "change the custom-prompt-customize-unsaved-options
SM> by adding a mechanism for ...", which is not what the patch does.
SM> Instead, they should just say

SM>   * lisp/cus-edit.el (custom-prompt-customize-unsaved-options): New function.

SM> Additionally, this text fails to mention custom-unsaved-options or the
SM> fact that customize-unsaved has been modified, so I'd write:

SM>   * lisp/cus-edit.el (custom-unsaved-options): New function, extracted
SM>   from customize-unsaved.
SM>   (custom-unsaved): Use it.
SM>   (custom-prompt-customize-unsaved-options): New function.

OK, I installed it earlier and have now added a commit to just fix that
ChangeLog entry.

FWIW, I think your version does not explain what the change provides,
only that it was made (it basically restates the diff).  I don't know if
that's what you intended.

Ted