Re: [PATCH] Correct indentation in svn_client_conflict_text_resolve_by_id
On 11/10/2016 23:37, Daniel Shahaf wrote:
> Stefan wrote on Thu, Nov 10, 2016 at 23:05:18 +0100:
>> On 11/10/2016 10:07, Stefan Sperling wrote:
>>> On Wed, Nov 09, 2016 at 11:21:42PM +0100, Stefan wrote:
>>>> attached is a patch to correct indentation in
>>>> svn_client_conflict_text_resolve_by_id() which was spotted by Evgeny
>>>> (related to the mail thread: "svn commit: r1764902 - in
>>>> /subversion/trunk/subversion: libsvn_client/conflicts.c
>>>> svn/conflict-callbacks.c tests/cmdline/resolve_tests.py").
>>>> Following up on r1764902: Corrected indentation.
>> Committed in r1769207 with the corrected name.
> A bit late: generally we prefer verbs to be in the imperative, not in
> the passive; i.e.: s/Corrected/Correct/.
> It's not too critical to edit this one; just noting this for future
Received on 2016-11-10 23:58:49 CET
This is an archived mail posted to the Subversion Dev