Re: [PATCH 3/4] fast-import: add option command

From: Shawn O. Pearce <spearce@spearce.org>
Date: Aug 13, 2009, 14:45 UTC
Message-ID: <20090813144539.GL1033@spearce.org>
In-Reply-To: <1250140186-12363-4-git-send-email-srabbelier@gmail.com>

Sverre Rabbelier <srabbelier@gmail.com> wrote:

Show 6 quoted lines

> This allows the frontend to specify any of the supported options as
> long as no non-option command has been given. This way the
> user does not have to include any frontend-specific options, but
> instead she can rely on the frontend to tell fast-import what it
> needs.
> ---

Missing Signed-off-by.

Show 7 quoted lines

> @@ -2460,6 +2465,16 @@ static void parse_one_option(const char *option)
>      }
>  }
>  
> +static void parse_option(void)
> +{
> +    char* option = command_buf.buf + 7;

Git style is "char *option", isn't it?

> +
> +	if (seen_non_option_command)
> +		die("Got option command '%s' after non-option command", option);

Indentation is messed up here.  1 tab per level, please.

-- 
Shawn.