emacs-orgmode
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [O] [PATCH] Add info when reference to remote table is not in the fi


From: Bastien
Subject: Re: [O] [PATCH] Add info when reference to remote table is not in the file
Date: Fri, 06 Apr 2012 09:43:28 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/24.0.94 (gnu/linux)

Hi Sébastien,

thanks for the patch.

I wonder if we should trigger an error instead -- why a message?

(I don't like using sit-for, it's often worse than getting an error...)

About the patch itself, thanks for the effort in adding a ChangeLog,
it's close to perfect: just think of starting sentences with a uppercase
letter, and of *including* the emacs ChangeLog in the patch.   See other
commit messages in the git log for reference.

Thanks!

-- 
 Bastien



reply via email to

[Prev in Thread] Current Thread [Next in Thread]