From: Jonathan Nieder <jrnieder@gmail.com>
Date: Sat, 20 Nov 2010 14:53:07 GMT
Subject: Re: [PATCH 10/15] vcs-svn: Reject path nodes without Node-action
Message-ID: <20101120145306.GA9804@burratino>
In-Reply-To: <20101120005259.GK17445@burratino>

Jonathan Nieder wrote:

> --- a/vcs-svn/svndump.c
> +++ b/vcs-svn/svndump.c
> @@ -174,7 +174,8 @@ static void handle_node(void)
>  
>  	if (node_ctx.srcRev) {
>  		repo_copy(node_ctx.srcRev, node_ctx.src, node_ctx.dst);
> -		node_ctx.action = NODEACT_CHANGE;
> +		if (node_ctx.action == NODEACT_ADD)
> +			  node_ctx.action = NODEACT_CHANGE;

Whitespace damage.  Here's a fixup.

Signed-off-by: Jonathan Nieder <jrnieder@gmail.com>
---
diff --git a/vcs-svn/svndump.c b/vcs-svn/svndump.c
index 78c2de0..0af8ac6 100644
--- a/vcs-svn/svndump.c
+++ b/vcs-svn/svndump.c
@@ -175,7 +175,7 @@ static void handle_node(void)
 	if (node_ctx.srcRev) {
 		repo_copy(node_ctx.srcRev, node_ctx.src, node_ctx.dst);
 		if (node_ctx.action == NODEACT_ADD)
-			  node_ctx.action = NODEACT_CHANGE;
+			node_ctx.action = NODEACT_CHANGE;
 	}
 
 	if (mark && type == REPO_MODE_DIR)