On 11/29/10 19:15, Jonathan Nieder wrote:
> + {OPTION_LOWLEVEL_CALLBACK, 0, "cacheinfo", NULL,
> + "<mode> <object> <path>",
> + "add the specified entry to the index",
> + PARSE_OPT_NOARG | PARSE_OPT_NONEG | PARSE_OPT_LITERAL_ARGHELP,
> + (parse_opt_cb *) cacheinfo_callback},
Doesn't this take arguments and thus shouldn't be marked
PARSE_OPT_NOARG? Confused.
> @@ -602,151 +799,48 @@ int cmd_update_index(int argc, const char **argv, const char *prefix)
> if (entries < 0)
> die("cache corrupted");
>
> - for (i = 1 ; i < argc; i++) {
> - const char *path = argv[i];
> - const char *p;
> + /*
> + * Custom copy of parse_options() because we want to handle
> + * filename arguments as they come.
> + */
> + parse_options_start(&ctx, argc, argv, prefix,
> + PARSE_OPT_STOP_AT_NON_OPTION);
This will need to take options too, sorry.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html