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

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