[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [O] [PATCH] org-capture: Add a custom to control save target file or
From: |
Nicolas Goaziou |
Subject: |
Re: [O] [PATCH] org-capture: Add a custom to control save target file or not. |
Date: |
Thu, 13 Dec 2018 17:25:29 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.1 (gnu/linux) |
Hello,
Kaushal Modi <address@hidden> writes:
> I have formatted 2 patches to address this (attached). If they look fine, I
> can commit them to master.
Thank you.
> + (unless (org-capture-get :no-save)
> + ;; Save the target file.
> + (save-buffer))
The comment does not look terribly useful. I would put the whole
`unless' in a single line.
Barring that nitpicking of mine, LGTM!
Regards,
--
Nicolas Goaziou