[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH] Correct indentation in svn_client_conflict_text_resolve_by_id

From: Daniel Shahaf <d.s_at_daniel.shahaf.name>
Date: Thu, 10 Nov 2016 22:37:47 +0000

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:
> >> Hi,
> >>
> >> 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").
> >>
> >> Regards,
> >> Stefan
> >>
> >> [[[
> >> 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
reference.

Cheers,

Daniel
Received on 2016-11-10 23:40:20 CET

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.