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

Re: svn commit: r983222 - /subversion/trunk/subversion/svnrdump/load_editor.c

From: Ramkumar Ramachandra <artagnon_at_gmail.com>
Date: Sat, 7 Aug 2010 20:02:07 +0530

Hi Daniel,

Daniel Shahaf writes:
> artagnon_at_apache.org wrote on Sat, Aug 07, 2010 at 12:31:50 -0000:
> > Author: artagnon
> > Date: Sat Aug 7 12:31:50 2010
> > New Revision: 983222
> >
> > URL: http://svn.apache.org/viewvc?rev=983222&view=rev
> > Log:
> > svnrdump: Fix a bug in the load_editor; it was unable to handle
> > revisions without node information previously.
> >
> > * subversion/svnrdump/load_editor.c
> > (close_revision): Add a new if-branch; if the commit_editor doesn't
> > exist, create one, open_root and close_edit on it to indicate that
> > we've finished processing the revision. While at it, also fix indentation.
> ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
>
> I take it you haven't seen my previous commit review yet?

The trade-off is the creation of many trivial commits :)

-- Ram
Received on 2010-08-07 16:34:07 CEST

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