2007-10-15 01:35:37 +02:00
|
|
|
#include "git-compat-util.h"
|
|
|
|
#include "parse-options.h"
|
2008-06-23 22:55:11 +02:00
|
|
|
#include "cache.h"
|
2009-01-26 15:13:23 +01:00
|
|
|
#include "commit.h"
|
Add an optional argument for --color options
Make git-branch, git-show-branch, git-grep, and all the diff-based
programs accept an optional argument <when> for --color. The argument
is a colorbool: "always", "never", or "auto". If no argument is given,
"always" is used; --no-color is an alias for --color=never. This makes
the command-line interface consistent with other GNU tools, such as `ls'
and `grep', and with the git-config color options. Note that, without
an argument, --color and --no-color work exactly as before.
To implement this, two internal changes were made:
1. Allow the first argument of git_config_colorbool() to be NULL,
in which case it returns -1 if the argument isn't "always", "never",
or "auto".
2. Add OPT_COLOR_FLAG(), OPT__COLOR(), and parse_opt_color_flag_cb()
to the option parsing library. The callback uses
git_config_colorbool(), so color.h is now a dependency
of parse-options.c.
Signed-off-by: Mark Lodato <lodatom@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2010-02-17 05:55:58 +01:00
|
|
|
#include "color.h"
|
2013-02-09 07:31:09 +01:00
|
|
|
#include "utf8.h"
|
2007-10-15 01:35:37 +02:00
|
|
|
|
2010-06-12 14:57:39 +02:00
|
|
|
static int parse_options_usage(struct parse_opt_ctx_t *ctx,
|
|
|
|
const char * const *usagestr,
|
2010-05-17 17:34:41 +02:00
|
|
|
const struct option *opts, int err);
|
2010-01-12 07:28:45 +01:00
|
|
|
|
2007-10-15 01:35:37 +02:00
|
|
|
#define OPT_SHORT 1
|
|
|
|
#define OPT_UNSET 2
|
|
|
|
|
2011-08-11 11:15:37 +02:00
|
|
|
int optbug(const struct option *opt, const char *reason)
|
2010-12-02 07:01:18 +01:00
|
|
|
{
|
|
|
|
if (opt->long_name)
|
|
|
|
return error("BUG: option '%s' %s", opt->long_name, reason);
|
|
|
|
return error("BUG: switch '%c' %s", opt->short_name, reason);
|
|
|
|
}
|
|
|
|
|
2011-08-11 11:15:37 +02:00
|
|
|
int opterror(const struct option *opt, const char *reason, int flags)
|
2007-10-15 01:35:37 +02:00
|
|
|
{
|
|
|
|
if (flags & OPT_SHORT)
|
|
|
|
return error("switch `%c' %s", opt->short_name, reason);
|
|
|
|
if (flags & OPT_UNSET)
|
|
|
|
return error("option `no-%s' %s", opt->long_name, reason);
|
|
|
|
return error("option `%s' %s", opt->long_name, reason);
|
|
|
|
}
|
|
|
|
|
2008-07-08 12:34:08 +02:00
|
|
|
static int get_arg(struct parse_opt_ctx_t *p, const struct option *opt,
|
|
|
|
int flags, const char **arg)
|
|
|
|
{
|
|
|
|
if (p->opt) {
|
|
|
|
*arg = p->opt;
|
|
|
|
p->opt = NULL;
|
|
|
|
} else if (p->argc == 1 && (opt->flags & PARSE_OPT_LASTARG_DEFAULT)) {
|
|
|
|
*arg = (const char *)opt->defval;
|
2008-07-21 20:30:36 +02:00
|
|
|
} else if (p->argc > 1) {
|
2008-07-08 12:34:08 +02:00
|
|
|
p->argc--;
|
|
|
|
*arg = *++p->argv;
|
|
|
|
} else
|
|
|
|
return opterror(opt, "requires a value", flags);
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2009-05-23 20:53:13 +02:00
|
|
|
static void fix_filename(const char *prefix, const char **file)
|
|
|
|
{
|
|
|
|
if (!file || !*file || !prefix || is_absolute_path(*file)
|
|
|
|
|| !strcmp("-", *file))
|
|
|
|
return;
|
|
|
|
*file = xstrdup(prefix_filename(prefix, strlen(prefix), *file));
|
|
|
|
}
|
|
|
|
|
2008-06-23 21:59:37 +02:00
|
|
|
static int get_value(struct parse_opt_ctx_t *p,
|
2008-07-08 12:34:08 +02:00
|
|
|
const struct option *opt, int flags)
|
2007-10-15 01:35:37 +02:00
|
|
|
{
|
2007-10-15 01:45:45 +02:00
|
|
|
const char *s, *arg;
|
2007-11-07 11:20:27 +01:00
|
|
|
const int unset = flags & OPT_UNSET;
|
2009-05-23 20:53:13 +02:00
|
|
|
int err;
|
2007-10-15 01:35:37 +02:00
|
|
|
|
2007-11-07 11:20:27 +01:00
|
|
|
if (unset && p->opt)
|
2007-10-15 01:35:37 +02:00
|
|
|
return opterror(opt, "takes no value", flags);
|
2007-11-07 11:20:27 +01:00
|
|
|
if (unset && (opt->flags & PARSE_OPT_NONEG))
|
|
|
|
return opterror(opt, "isn't available", flags);
|
2010-12-02 00:30:40 +01:00
|
|
|
if (!(flags & OPT_SHORT) && p->opt && (opt->flags & PARSE_OPT_NOARG))
|
|
|
|
return opterror(opt, "takes no value", flags);
|
2007-11-07 11:20:27 +01:00
|
|
|
|
|
|
|
switch (opt->type) {
|
parse-options: allow git commands to invent new option types
parse-options provides a variety of option behaviors, including
OPTION_CALLBACK, which should take care of just about any sane
behavior. All supported behaviors obey the following constraint:
A --foo option can only accept (and base its behavior on)
one argument, which would be the following command-line
argument in the "unsticked" form.
Alas, some existing git commands have options that do not obey that
constraint. For example, update-index --cacheinfo takes three
arguments, and update-index --resolve takes all later parameters as
arguments.
Introduces an OPTION_LOWLEVEL_CALLBACK backdoor to parse-options so
such option types can be supported without tempting inventors of other
commands through mention in the public API. Commands can set the
callback field to a function accepting three arguments: the option
parsing context, the option itself, and a flag indicating whether the
the option was negated. When the option is encountered, that function
is called to take over from get_value(). The return value should be
zero for success, -1 for usage errors.
Thanks to Stephen Boyd for API guidance.
Improved-by: Stephen Boyd <bebarino@gmail.com>
Signed-off-by: Jonathan Nieder <jrnieder@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2010-12-02 00:32:16 +01:00
|
|
|
case OPTION_LOWLEVEL_CALLBACK:
|
|
|
|
return (*(parse_opt_ll_cb *)opt->callback)(p, opt, unset);
|
|
|
|
|
2007-11-07 11:20:27 +01:00
|
|
|
case OPTION_BIT:
|
|
|
|
if (unset)
|
|
|
|
*(int *)opt->value &= ~opt->defval;
|
2007-10-15 01:35:37 +02:00
|
|
|
else
|
2007-11-07 11:20:27 +01:00
|
|
|
*(int *)opt->value |= opt->defval;
|
|
|
|
return 0;
|
|
|
|
|
2009-05-07 21:44:17 +02:00
|
|
|
case OPTION_NEGBIT:
|
|
|
|
if (unset)
|
|
|
|
*(int *)opt->value |= opt->defval;
|
|
|
|
else
|
|
|
|
*(int *)opt->value &= ~opt->defval;
|
|
|
|
return 0;
|
|
|
|
|
parse-options: deprecate OPT_BOOLEAN
It is natural to expect that an option defined with OPT_BOOLEAN() could be
used in this way:
int option = -1; /* unspecified */
struct option options[] = {
OPT_BOOLEAN(0, "option", &option, "set option"),
OPT_END()
};
parse_options(ac, av, prefix, options, usage, 0);
if (option < 0)
... do the default thing ...
else if (!option)
... --no-option was given ...
else
... --option was given ...
to easily tell three cases apart:
- There is no mention of the `--option` on the command line;
- The variable is positively set with `--option`; or
- The variable is explicitly negated with `--no-option`.
Unfortunately, this is not the case. OPT_BOOLEAN() increments the variable
every time `--option` is given, and resets it to zero when `--no-option`
is given.
As a first step to remedy this, introduce a true boolean OPT_BOOL(), and
rename OPT_BOOLEAN() to OPT_COUNTUP(). To help transitioning, OPT_BOOLEAN
and OPTION_BOOLEAN are defined as deprecated synonyms to OPT_COUNTUP and
OPTION_COUNTUP respectively.
This is what db7244b (parse-options new features., 2007-11-07) from four
years ago started by marking OPTION_BOOLEAN as "INCR would have been a
better name".
Some existing users do depend on the count-up semantics; for example,
users of OPT__VERBOSE() could use it to raise the verbosity level with
repeated use of `-v` on the command line, but they probably should be
rewritten to use OPT__VERBOSITY() instead these days. I suspect that some
users of OPT__FORCE() may also use it to implement different level of
forcibleness but I didn't check.
On top of this patch, here are the remaining clean-up tasks that other
people can help:
- Look at each hit in "git grep -e OPT_BOOLEAN"; trace all uses of the
value that is set to the underlying variable, and if it can proven that
the variable is only used as a boolean, replace it with OPT_BOOL(). If
the caller does depend on the count-up semantics, replace it with
OPT_COUNTUP() instead.
- Same for OPTION_BOOLEAN; replace it with OPTION_SET_INT and arrange to
set 1 to the variable for a true boolean, and otherwise replace it with
OPTION_COUNTUP.
- Look at each hit in "git grep -e OPT__VERBOSE -e OPT__QUIET" and see if
they can be replaced with OPT__VERBOSITY().
I'll follow this message up with a separate patch as an example.
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2011-09-28 01:56:49 +02:00
|
|
|
case OPTION_COUNTUP:
|
2007-11-07 11:20:27 +01:00
|
|
|
*(int *)opt->value = unset ? 0 : *(int *)opt->value + 1;
|
|
|
|
return 0;
|
|
|
|
|
|
|
|
case OPTION_SET_INT:
|
|
|
|
*(int *)opt->value = unset ? 0 : opt->defval;
|
|
|
|
return 0;
|
|
|
|
|
|
|
|
case OPTION_SET_PTR:
|
|
|
|
*(void **)opt->value = unset ? NULL : (void *)opt->defval;
|
2007-10-15 01:35:37 +02:00
|
|
|
return 0;
|
|
|
|
|
|
|
|
case OPTION_STRING:
|
2008-07-08 12:34:08 +02:00
|
|
|
if (unset)
|
2007-11-07 11:20:27 +01:00
|
|
|
*(const char **)opt->value = NULL;
|
2008-07-08 12:34:08 +02:00
|
|
|
else if (opt->flags & PARSE_OPT_OPTARG && !p->opt)
|
2007-10-15 01:45:45 +02:00
|
|
|
*(const char **)opt->value = (const char *)opt->defval;
|
2008-07-08 12:34:08 +02:00
|
|
|
else
|
|
|
|
return get_arg(p, opt, flags, (const char **)opt->value);
|
2007-10-15 01:35:37 +02:00
|
|
|
return 0;
|
|
|
|
|
2009-05-23 20:53:13 +02:00
|
|
|
case OPTION_FILENAME:
|
|
|
|
err = 0;
|
|
|
|
if (unset)
|
|
|
|
*(const char **)opt->value = NULL;
|
|
|
|
else if (opt->flags & PARSE_OPT_OPTARG && !p->opt)
|
|
|
|
*(const char **)opt->value = (const char *)opt->defval;
|
|
|
|
else
|
|
|
|
err = get_arg(p, opt, flags, (const char **)opt->value);
|
|
|
|
|
|
|
|
if (!err)
|
|
|
|
fix_filename(p->prefix, (const char **)opt->value);
|
|
|
|
return err;
|
|
|
|
|
2007-10-15 01:45:45 +02:00
|
|
|
case OPTION_CALLBACK:
|
2007-11-07 11:20:27 +01:00
|
|
|
if (unset)
|
2008-06-23 22:46:36 +02:00
|
|
|
return (*opt->callback)(opt, NULL, 1) ? (-1) : 0;
|
2007-11-07 11:20:27 +01:00
|
|
|
if (opt->flags & PARSE_OPT_NOARG)
|
2008-06-23 22:46:36 +02:00
|
|
|
return (*opt->callback)(opt, NULL, 0) ? (-1) : 0;
|
2007-12-21 11:41:41 +01:00
|
|
|
if (opt->flags & PARSE_OPT_OPTARG && !p->opt)
|
2008-06-23 22:46:36 +02:00
|
|
|
return (*opt->callback)(opt, NULL, 0) ? (-1) : 0;
|
2008-07-08 12:34:08 +02:00
|
|
|
if (get_arg(p, opt, flags, &arg))
|
|
|
|
return -1;
|
|
|
|
return (*opt->callback)(opt, arg, 0) ? (-1) : 0;
|
2007-10-15 01:45:45 +02:00
|
|
|
|
2007-10-15 01:35:37 +02:00
|
|
|
case OPTION_INTEGER:
|
2007-11-07 11:20:27 +01:00
|
|
|
if (unset) {
|
2007-10-15 01:35:37 +02:00
|
|
|
*(int *)opt->value = 0;
|
|
|
|
return 0;
|
|
|
|
}
|
2007-12-21 11:41:41 +01:00
|
|
|
if (opt->flags & PARSE_OPT_OPTARG && !p->opt) {
|
2007-10-15 01:45:45 +02:00
|
|
|
*(int *)opt->value = opt->defval;
|
|
|
|
return 0;
|
|
|
|
}
|
2008-07-08 12:34:08 +02:00
|
|
|
if (get_arg(p, opt, flags, &arg))
|
|
|
|
return -1;
|
|
|
|
*(int *)opt->value = strtol(arg, (char **)&s, 10);
|
2007-10-15 01:35:37 +02:00
|
|
|
if (*s)
|
|
|
|
return opterror(opt, "expects a numerical value", flags);
|
|
|
|
return 0;
|
|
|
|
|
|
|
|
default:
|
|
|
|
die("should not happen, someone must be hit on the forehead");
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2008-06-23 21:59:37 +02:00
|
|
|
static int parse_short_opt(struct parse_opt_ctx_t *p, const struct option *options)
|
2007-10-15 01:35:37 +02:00
|
|
|
{
|
2009-05-07 21:45:08 +02:00
|
|
|
const struct option *numopt = NULL;
|
|
|
|
|
2007-10-15 01:35:37 +02:00
|
|
|
for (; options->type != OPTION_END; options++) {
|
|
|
|
if (options->short_name == *p->opt) {
|
|
|
|
p->opt = p->opt[1] ? p->opt + 1 : NULL;
|
|
|
|
return get_value(p, options, OPT_SHORT);
|
|
|
|
}
|
2009-05-07 21:45:08 +02:00
|
|
|
|
|
|
|
/*
|
|
|
|
* Handle the numerical option later, explicit one-digit
|
|
|
|
* options take precedence over it.
|
|
|
|
*/
|
|
|
|
if (options->type == OPTION_NUMBER)
|
|
|
|
numopt = options;
|
|
|
|
}
|
|
|
|
if (numopt && isdigit(*p->opt)) {
|
|
|
|
size_t len = 1;
|
|
|
|
char *arg;
|
|
|
|
int rc;
|
|
|
|
|
|
|
|
while (isdigit(p->opt[len]))
|
|
|
|
len++;
|
|
|
|
arg = xmemdupz(p->opt, len);
|
|
|
|
p->opt = p->opt[len] ? p->opt + len : NULL;
|
|
|
|
rc = (*numopt->callback)(numopt, arg, 0) ? (-1) : 0;
|
|
|
|
free(arg);
|
|
|
|
return rc;
|
2007-10-15 01:35:37 +02:00
|
|
|
}
|
2008-06-23 22:46:36 +02:00
|
|
|
return -2;
|
2007-10-15 01:35:37 +02:00
|
|
|
}
|
|
|
|
|
2008-06-23 21:59:37 +02:00
|
|
|
static int parse_long_opt(struct parse_opt_ctx_t *p, const char *arg,
|
2007-10-15 01:35:37 +02:00
|
|
|
const struct option *options)
|
|
|
|
{
|
2007-10-14 18:54:06 +02:00
|
|
|
const char *arg_end = strchr(arg, '=');
|
2007-11-05 14:15:21 +01:00
|
|
|
const struct option *abbrev_option = NULL, *ambiguous_option = NULL;
|
|
|
|
int abbrev_flags = 0, ambiguous_flags = 0;
|
2007-10-14 18:54:06 +02:00
|
|
|
|
|
|
|
if (!arg_end)
|
|
|
|
arg_end = arg + strlen(arg);
|
|
|
|
|
2007-10-15 01:35:37 +02:00
|
|
|
for (; options->type != OPTION_END; options++) {
|
2012-02-25 20:14:54 +01:00
|
|
|
const char *rest, *long_name = options->long_name;
|
|
|
|
int flags = 0, opt_flags = 0;
|
2007-10-15 01:35:37 +02:00
|
|
|
|
2012-02-25 20:14:54 +01:00
|
|
|
if (!long_name)
|
2007-10-15 01:35:37 +02:00
|
|
|
continue;
|
|
|
|
|
2012-02-25 20:14:54 +01:00
|
|
|
again:
|
|
|
|
rest = skip_prefix(arg, long_name);
|
2008-03-02 11:35:56 +01:00
|
|
|
if (options->type == OPTION_ARGUMENT) {
|
|
|
|
if (!rest)
|
|
|
|
continue;
|
|
|
|
if (*rest == '=')
|
|
|
|
return opterror(options, "takes no value", flags);
|
|
|
|
if (*rest)
|
|
|
|
continue;
|
|
|
|
p->out[p->cpidx++] = arg - 2;
|
|
|
|
return 0;
|
|
|
|
}
|
2007-10-15 01:35:37 +02:00
|
|
|
if (!rest) {
|
2007-10-14 18:54:06 +02:00
|
|
|
/* abbreviated? */
|
2012-02-25 20:14:54 +01:00
|
|
|
if (!strncmp(long_name, arg, arg_end - arg)) {
|
2007-10-14 18:54:06 +02:00
|
|
|
is_abbreviated:
|
2007-11-05 14:15:21 +01:00
|
|
|
if (abbrev_option) {
|
|
|
|
/*
|
|
|
|
* If this is abbreviated, it is
|
|
|
|
* ambiguous. So when there is no
|
|
|
|
* exact match later, we need to
|
|
|
|
* error out.
|
|
|
|
*/
|
|
|
|
ambiguous_option = abbrev_option;
|
|
|
|
ambiguous_flags = abbrev_flags;
|
|
|
|
}
|
2007-10-14 18:54:06 +02:00
|
|
|
if (!(flags & OPT_UNSET) && *arg_end)
|
|
|
|
p->opt = arg_end + 1;
|
|
|
|
abbrev_option = options;
|
2012-02-25 20:14:54 +01:00
|
|
|
abbrev_flags = flags ^ opt_flags;
|
2007-10-14 18:54:06 +02:00
|
|
|
continue;
|
|
|
|
}
|
2009-09-25 20:44:44 +02:00
|
|
|
/* negation allowed? */
|
|
|
|
if (options->flags & PARSE_OPT_NONEG)
|
|
|
|
continue;
|
2007-10-14 18:54:06 +02:00
|
|
|
/* negated and abbreviated very much? */
|
|
|
|
if (!prefixcmp("no-", arg)) {
|
|
|
|
flags |= OPT_UNSET;
|
|
|
|
goto is_abbreviated;
|
|
|
|
}
|
|
|
|
/* negated? */
|
2012-02-25 20:14:54 +01:00
|
|
|
if (prefixcmp(arg, "no-")) {
|
|
|
|
if (!prefixcmp(long_name, "no-")) {
|
|
|
|
long_name += 3;
|
|
|
|
opt_flags |= OPT_UNSET;
|
|
|
|
goto again;
|
|
|
|
}
|
2007-10-15 01:35:37 +02:00
|
|
|
continue;
|
2012-02-25 20:14:54 +01:00
|
|
|
}
|
2007-10-15 01:35:37 +02:00
|
|
|
flags |= OPT_UNSET;
|
2012-02-25 20:14:54 +01:00
|
|
|
rest = skip_prefix(arg + 3, long_name);
|
2007-10-14 18:54:06 +02:00
|
|
|
/* abbreviated and negated? */
|
2012-02-25 20:14:54 +01:00
|
|
|
if (!rest && !prefixcmp(long_name, arg + 3))
|
2007-10-14 18:54:06 +02:00
|
|
|
goto is_abbreviated;
|
2007-10-15 01:35:37 +02:00
|
|
|
if (!rest)
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
if (*rest) {
|
|
|
|
if (*rest != '=')
|
|
|
|
continue;
|
|
|
|
p->opt = rest + 1;
|
|
|
|
}
|
2012-02-25 20:14:54 +01:00
|
|
|
return get_value(p, options, flags ^ opt_flags);
|
2007-10-15 01:35:37 +02:00
|
|
|
}
|
2007-11-05 14:15:21 +01:00
|
|
|
|
|
|
|
if (ambiguous_option)
|
|
|
|
return error("Ambiguous option: %s "
|
|
|
|
"(could be --%s%s or --%s%s)",
|
|
|
|
arg,
|
|
|
|
(ambiguous_flags & OPT_UNSET) ? "no-" : "",
|
|
|
|
ambiguous_option->long_name,
|
|
|
|
(abbrev_flags & OPT_UNSET) ? "no-" : "",
|
|
|
|
abbrev_option->long_name);
|
2007-10-14 18:54:06 +02:00
|
|
|
if (abbrev_option)
|
|
|
|
return get_value(p, abbrev_option, abbrev_flags);
|
2008-06-23 22:46:36 +02:00
|
|
|
return -2;
|
2007-10-15 01:35:37 +02:00
|
|
|
}
|
|
|
|
|
2009-05-07 21:45:42 +02:00
|
|
|
static int parse_nodash_opt(struct parse_opt_ctx_t *p, const char *arg,
|
|
|
|
const struct option *options)
|
|
|
|
{
|
|
|
|
for (; options->type != OPTION_END; options++) {
|
|
|
|
if (!(options->flags & PARSE_OPT_NODASH))
|
|
|
|
continue;
|
|
|
|
if (options->short_name == arg[0] && arg[1] == '\0')
|
|
|
|
return get_value(p, options, OPT_SHORT);
|
|
|
|
}
|
|
|
|
return -2;
|
|
|
|
}
|
|
|
|
|
2008-07-16 12:42:18 +02:00
|
|
|
static void check_typos(const char *arg, const struct option *options)
|
2008-01-26 12:26:57 +01:00
|
|
|
{
|
|
|
|
if (strlen(arg) < 3)
|
|
|
|
return;
|
|
|
|
|
|
|
|
if (!prefixcmp(arg, "no-")) {
|
|
|
|
error ("did you mean `--%s` (with two dashes ?)", arg);
|
|
|
|
exit(129);
|
|
|
|
}
|
|
|
|
|
|
|
|
for (; options->type != OPTION_END; options++) {
|
|
|
|
if (!options->long_name)
|
|
|
|
continue;
|
|
|
|
if (!prefixcmp(options->long_name, arg)) {
|
|
|
|
error ("did you mean `--%s` (with two dashes ?)", arg);
|
|
|
|
exit(129);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2009-06-09 10:23:44 +02:00
|
|
|
static void parse_options_check(const struct option *opts)
|
|
|
|
{
|
|
|
|
int err = 0;
|
|
|
|
|
|
|
|
for (; opts->type != OPTION_END; opts++) {
|
|
|
|
if ((opts->flags & PARSE_OPT_LASTARG_DEFAULT) &&
|
2010-12-02 07:01:18 +01:00
|
|
|
(opts->flags & PARSE_OPT_OPTARG))
|
|
|
|
err |= optbug(opts, "uses incompatible flags "
|
|
|
|
"LASTARG_DEFAULT and OPTARG");
|
2010-12-02 07:05:05 +01:00
|
|
|
if (opts->flags & PARSE_OPT_NODASH &&
|
|
|
|
((opts->flags & PARSE_OPT_OPTARG) ||
|
|
|
|
!(opts->flags & PARSE_OPT_NOARG) ||
|
|
|
|
!(opts->flags & PARSE_OPT_NONEG) ||
|
|
|
|
opts->long_name))
|
|
|
|
err |= optbug(opts, "uses feature "
|
|
|
|
"not supported for dashless options");
|
2010-12-02 07:08:57 +01:00
|
|
|
switch (opts->type) {
|
parse-options: deprecate OPT_BOOLEAN
It is natural to expect that an option defined with OPT_BOOLEAN() could be
used in this way:
int option = -1; /* unspecified */
struct option options[] = {
OPT_BOOLEAN(0, "option", &option, "set option"),
OPT_END()
};
parse_options(ac, av, prefix, options, usage, 0);
if (option < 0)
... do the default thing ...
else if (!option)
... --no-option was given ...
else
... --option was given ...
to easily tell three cases apart:
- There is no mention of the `--option` on the command line;
- The variable is positively set with `--option`; or
- The variable is explicitly negated with `--no-option`.
Unfortunately, this is not the case. OPT_BOOLEAN() increments the variable
every time `--option` is given, and resets it to zero when `--no-option`
is given.
As a first step to remedy this, introduce a true boolean OPT_BOOL(), and
rename OPT_BOOLEAN() to OPT_COUNTUP(). To help transitioning, OPT_BOOLEAN
and OPTION_BOOLEAN are defined as deprecated synonyms to OPT_COUNTUP and
OPTION_COUNTUP respectively.
This is what db7244b (parse-options new features., 2007-11-07) from four
years ago started by marking OPTION_BOOLEAN as "INCR would have been a
better name".
Some existing users do depend on the count-up semantics; for example,
users of OPT__VERBOSE() could use it to raise the verbosity level with
repeated use of `-v` on the command line, but they probably should be
rewritten to use OPT__VERBOSITY() instead these days. I suspect that some
users of OPT__FORCE() may also use it to implement different level of
forcibleness but I didn't check.
On top of this patch, here are the remaining clean-up tasks that other
people can help:
- Look at each hit in "git grep -e OPT_BOOLEAN"; trace all uses of the
value that is set to the underlying variable, and if it can proven that
the variable is only used as a boolean, replace it with OPT_BOOL(). If
the caller does depend on the count-up semantics, replace it with
OPT_COUNTUP() instead.
- Same for OPTION_BOOLEAN; replace it with OPTION_SET_INT and arrange to
set 1 to the variable for a true boolean, and otherwise replace it with
OPTION_COUNTUP.
- Look at each hit in "git grep -e OPT__VERBOSE -e OPT__QUIET" and see if
they can be replaced with OPT__VERBOSITY().
I'll follow this message up with a separate patch as an example.
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2011-09-28 01:56:49 +02:00
|
|
|
case OPTION_COUNTUP:
|
2010-12-02 07:08:57 +01:00
|
|
|
case OPTION_BIT:
|
|
|
|
case OPTION_NEGBIT:
|
|
|
|
case OPTION_SET_INT:
|
|
|
|
case OPTION_SET_PTR:
|
|
|
|
case OPTION_NUMBER:
|
|
|
|
if ((opts->flags & PARSE_OPT_OPTARG) ||
|
|
|
|
!(opts->flags & PARSE_OPT_NOARG))
|
|
|
|
err |= optbug(opts, "should not accept an argument");
|
|
|
|
default:
|
|
|
|
; /* ok. (usually accepts an argument) */
|
|
|
|
}
|
2009-06-09 10:23:44 +02:00
|
|
|
}
|
|
|
|
if (err)
|
2010-12-02 07:01:18 +01:00
|
|
|
exit(128);
|
2009-06-09 10:23:44 +02:00
|
|
|
}
|
|
|
|
|
2008-06-23 21:59:37 +02:00
|
|
|
void parse_options_start(struct parse_opt_ctx_t *ctx,
|
2009-05-23 20:53:12 +02:00
|
|
|
int argc, const char **argv, const char *prefix,
|
2010-12-06 08:57:42 +01:00
|
|
|
const struct option *options, int flags)
|
2008-06-23 21:59:37 +02:00
|
|
|
{
|
|
|
|
memset(ctx, 0, sizeof(*ctx));
|
|
|
|
ctx->argc = argc - 1;
|
|
|
|
ctx->argv = argv + 1;
|
|
|
|
ctx->out = argv;
|
2009-05-23 20:53:12 +02:00
|
|
|
ctx->prefix = prefix;
|
2008-06-24 00:31:31 +02:00
|
|
|
ctx->cpidx = ((flags & PARSE_OPT_KEEP_ARGV0) != 0);
|
2008-06-23 21:59:37 +02:00
|
|
|
ctx->flags = flags;
|
2009-03-09 21:57:38 +01:00
|
|
|
if ((flags & PARSE_OPT_KEEP_UNKNOWN) &&
|
|
|
|
(flags & PARSE_OPT_STOP_AT_NON_OPTION))
|
|
|
|
die("STOP_AT_NON_OPTION and KEEP_UNKNOWN don't go together");
|
2010-12-06 08:57:42 +01:00
|
|
|
parse_options_check(options);
|
2008-06-23 21:59:37 +02:00
|
|
|
}
|
|
|
|
|
2010-06-12 14:57:39 +02:00
|
|
|
static int usage_with_options_internal(struct parse_opt_ctx_t *,
|
|
|
|
const char * const *,
|
2010-05-17 17:34:41 +02:00
|
|
|
const struct option *, int, int);
|
2007-11-19 10:21:44 +01:00
|
|
|
|
2008-06-23 22:38:58 +02:00
|
|
|
int parse_options_step(struct parse_opt_ctx_t *ctx,
|
|
|
|
const struct option *options,
|
|
|
|
const char * const usagestr[])
|
2007-10-15 01:35:37 +02:00
|
|
|
{
|
2009-03-08 19:15:08 +01:00
|
|
|
int internal_help = !(ctx->flags & PARSE_OPT_NO_INTERNAL_HELP);
|
|
|
|
|
2008-06-23 22:55:11 +02:00
|
|
|
/* we must reset ->opt, unknown short option leave it dangling */
|
|
|
|
ctx->opt = NULL;
|
|
|
|
|
2008-06-23 22:38:58 +02:00
|
|
|
for (; ctx->argc; ctx->argc--, ctx->argv++) {
|
|
|
|
const char *arg = ctx->argv[0];
|
2007-10-15 01:35:37 +02:00
|
|
|
|
|
|
|
if (*arg != '-' || !arg[1]) {
|
2009-05-07 21:45:42 +02:00
|
|
|
if (parse_nodash_opt(ctx, arg, options) == 0)
|
|
|
|
continue;
|
2008-06-23 22:38:58 +02:00
|
|
|
if (ctx->flags & PARSE_OPT_STOP_AT_NON_OPTION)
|
2010-12-02 00:32:55 +01:00
|
|
|
return PARSE_OPT_NON_OPTION;
|
2008-06-23 22:38:58 +02:00
|
|
|
ctx->out[ctx->cpidx++] = ctx->argv[0];
|
2007-10-15 01:35:37 +02:00
|
|
|
continue;
|
|
|
|
}
|
|
|
|
|
|
|
|
if (arg[1] != '-') {
|
2008-06-23 22:38:58 +02:00
|
|
|
ctx->opt = arg + 1;
|
2009-03-08 19:15:08 +01:00
|
|
|
if (internal_help && *ctx->opt == 'h')
|
2010-06-12 14:57:39 +02:00
|
|
|
return parse_options_usage(ctx, usagestr, options, 0);
|
2008-06-23 22:46:36 +02:00
|
|
|
switch (parse_short_opt(ctx, options)) {
|
|
|
|
case -1:
|
2010-06-12 14:57:39 +02:00
|
|
|
return parse_options_usage(ctx, usagestr, options, 1);
|
2008-06-23 22:46:36 +02:00
|
|
|
case -2:
|
2012-03-03 12:00:29 +01:00
|
|
|
if (ctx->opt)
|
|
|
|
check_typos(arg + 1, options);
|
2009-03-08 19:12:47 +01:00
|
|
|
goto unknown;
|
2008-06-23 22:46:36 +02:00
|
|
|
}
|
2008-06-23 22:38:58 +02:00
|
|
|
if (ctx->opt)
|
2008-01-26 12:26:57 +01:00
|
|
|
check_typos(arg + 1, options);
|
2008-06-23 22:38:58 +02:00
|
|
|
while (ctx->opt) {
|
2009-03-08 19:15:08 +01:00
|
|
|
if (internal_help && *ctx->opt == 'h')
|
2010-06-12 14:57:39 +02:00
|
|
|
return parse_options_usage(ctx, usagestr, options, 0);
|
2008-06-23 22:46:36 +02:00
|
|
|
switch (parse_short_opt(ctx, options)) {
|
|
|
|
case -1:
|
2010-06-12 14:57:39 +02:00
|
|
|
return parse_options_usage(ctx, usagestr, options, 1);
|
2008-06-23 22:46:36 +02:00
|
|
|
case -2:
|
2008-06-23 22:55:11 +02:00
|
|
|
/* fake a short option thing to hide the fact that we may have
|
|
|
|
* started to parse aggregated stuff
|
|
|
|
*
|
|
|
|
* This is leaky, too bad.
|
|
|
|
*/
|
|
|
|
ctx->argv[0] = xstrdup(ctx->opt - 1);
|
|
|
|
*(char *)ctx->argv[0] = '-';
|
2009-03-08 19:12:47 +01:00
|
|
|
goto unknown;
|
2008-06-23 22:46:36 +02:00
|
|
|
}
|
2008-01-26 12:26:57 +01:00
|
|
|
}
|
2007-10-15 01:35:37 +02:00
|
|
|
continue;
|
|
|
|
}
|
|
|
|
|
|
|
|
if (!arg[2]) { /* "--" */
|
2008-06-23 22:38:58 +02:00
|
|
|
if (!(ctx->flags & PARSE_OPT_KEEP_DASHDASH)) {
|
|
|
|
ctx->argc--;
|
|
|
|
ctx->argv++;
|
2007-10-15 01:35:37 +02:00
|
|
|
}
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
|
2009-03-08 19:15:08 +01:00
|
|
|
if (internal_help && !strcmp(arg + 2, "help-all"))
|
2010-06-12 14:57:39 +02:00
|
|
|
return usage_with_options_internal(ctx, usagestr, options, 1, 0);
|
2009-03-08 19:15:08 +01:00
|
|
|
if (internal_help && !strcmp(arg + 2, "help"))
|
2010-06-12 14:57:39 +02:00
|
|
|
return parse_options_usage(ctx, usagestr, options, 0);
|
2008-06-23 22:46:36 +02:00
|
|
|
switch (parse_long_opt(ctx, arg + 2, options)) {
|
|
|
|
case -1:
|
2010-06-12 14:57:39 +02:00
|
|
|
return parse_options_usage(ctx, usagestr, options, 1);
|
2008-06-23 22:46:36 +02:00
|
|
|
case -2:
|
2009-03-08 19:12:47 +01:00
|
|
|
goto unknown;
|
2008-06-23 22:46:36 +02:00
|
|
|
}
|
2009-03-08 19:12:47 +01:00
|
|
|
continue;
|
|
|
|
unknown:
|
|
|
|
if (!(ctx->flags & PARSE_OPT_KEEP_UNKNOWN))
|
|
|
|
return PARSE_OPT_UNKNOWN;
|
|
|
|
ctx->out[ctx->cpidx++] = ctx->argv[0];
|
|
|
|
ctx->opt = NULL;
|
2008-06-23 22:38:58 +02:00
|
|
|
}
|
|
|
|
return PARSE_OPT_DONE;
|
|
|
|
}
|
|
|
|
|
|
|
|
int parse_options_end(struct parse_opt_ctx_t *ctx)
|
|
|
|
{
|
|
|
|
memmove(ctx->out + ctx->cpidx, ctx->argv, ctx->argc * sizeof(*ctx->out));
|
|
|
|
ctx->out[ctx->cpidx + ctx->argc] = NULL;
|
|
|
|
return ctx->cpidx + ctx->argc;
|
|
|
|
}
|
|
|
|
|
2009-05-23 20:53:12 +02:00
|
|
|
int parse_options(int argc, const char **argv, const char *prefix,
|
|
|
|
const struct option *options, const char * const usagestr[],
|
|
|
|
int flags)
|
2008-06-23 22:38:58 +02:00
|
|
|
{
|
|
|
|
struct parse_opt_ctx_t ctx;
|
|
|
|
|
2010-12-06 08:57:42 +01:00
|
|
|
parse_options_start(&ctx, argc, argv, prefix, options, flags);
|
2008-06-23 22:38:58 +02:00
|
|
|
switch (parse_options_step(&ctx, options, usagestr)) {
|
|
|
|
case PARSE_OPT_HELP:
|
|
|
|
exit(129);
|
2010-12-02 00:32:55 +01:00
|
|
|
case PARSE_OPT_NON_OPTION:
|
2008-06-23 22:38:58 +02:00
|
|
|
case PARSE_OPT_DONE:
|
|
|
|
break;
|
|
|
|
default: /* PARSE_OPT_UNKNOWN */
|
2008-06-23 22:46:36 +02:00
|
|
|
if (ctx.argv[0][1] == '-') {
|
|
|
|
error("unknown option `%s'", ctx.argv[0] + 2);
|
2013-02-12 00:13:48 +01:00
|
|
|
} else if (isascii(*ctx.opt)) {
|
2008-06-23 22:46:36 +02:00
|
|
|
error("unknown switch `%c'", *ctx.opt);
|
2013-02-12 00:13:48 +01:00
|
|
|
} else {
|
|
|
|
error("unknown non-ascii option in string: `%s'",
|
|
|
|
ctx.argv[0]);
|
2008-06-23 22:46:36 +02:00
|
|
|
}
|
|
|
|
usage_with_options(usagestr, options);
|
2007-10-15 01:35:37 +02:00
|
|
|
}
|
|
|
|
|
git on Mac OS and precomposed unicode
Mac OS X mangles file names containing unicode on file systems HFS+,
VFAT or SAMBA. When a file using unicode code points outside ASCII
is created on a HFS+ drive, the file name is converted into
decomposed unicode and written to disk. No conversion is done if
the file name is already decomposed unicode.
Calling open("\xc3\x84", ...) with a precomposed "Ä" yields the same
result as open("\x41\xcc\x88",...) with a decomposed "Ä".
As a consequence, readdir() returns the file names in decomposed
unicode, even if the user expects precomposed unicode. Unlike on
HFS+, Mac OS X stores files on a VFAT drive (e.g. an USB drive) in
precomposed unicode, but readdir() still returns file names in
decomposed unicode. When a git repository is stored on a network
share using SAMBA, file names are send over the wire and written to
disk on the remote system in precomposed unicode, but Mac OS X
readdir() returns decomposed unicode to be compatible with its
behaviour on HFS+ and VFAT.
The unicode decomposition causes many problems:
- The names "git add" and other commands get from the end user may
often be precomposed form (the decomposed form is not easily input
from the keyboard), but when the commands read from the filesystem
to see what it is going to update the index with already is on the
filesystem, readdir() will give decomposed form, which is different.
- Similarly "git log", "git mv" and all other commands that need to
compare pathnames found on the command line (often but not always
precomposed form; a command line input resulting from globbing may
be in decomposed) with pathnames found in the tree objects (should
be precomposed form to be compatible with other systems and for
consistency in general).
- The same for names stored in the index, which should be
precomposed, that may need to be compared with the names read from
readdir().
NFS mounted from Linux is fully transparent and does not suffer from
the above.
As Mac OS X treats precomposed and decomposed file names as equal,
we can
- wrap readdir() on Mac OS X to return the precomposed form, and
- normalize decomposed form given from the command line also to the
precomposed form,
to ensure that all pathnames used in Git are always in the
precomposed form. This behaviour can be requested by setting
"core.precomposedunicode" configuration variable to true.
The code in compat/precomposed_utf8.c implements basically 4 new
functions: precomposed_utf8_opendir(), precomposed_utf8_readdir(),
precomposed_utf8_closedir() and precompose_argv(). The first three
are to wrap opendir(3), readdir(3), and closedir(3) functions.
The argv[] conversion allows to use the TAB filename completion done
by the shell on command line. It tolerates other tools which use
readdir() to feed decomposed file names into git.
When creating a new git repository with "git init" or "git clone",
"core.precomposedunicode" will be set "false".
The user needs to activate this feature manually. She typically
sets core.precomposedunicode to "true" on HFS and VFAT, or file
systems mounted via SAMBA.
Helped-by: Junio C Hamano <gitster@pobox.com>
Signed-off-by: Torsten Bögershausen <tboegi@web.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2012-07-08 15:50:25 +02:00
|
|
|
precompose_argv(argc, argv);
|
2008-06-23 21:59:37 +02:00
|
|
|
return parse_options_end(&ctx);
|
2007-10-15 01:35:37 +02:00
|
|
|
}
|
2007-10-15 01:38:30 +02:00
|
|
|
|
2010-05-17 17:34:41 +02:00
|
|
|
static int usage_argh(const struct option *opts, FILE *outfile)
|
2009-05-21 09:33:17 +02:00
|
|
|
{
|
|
|
|
const char *s;
|
2009-06-05 01:43:57 +02:00
|
|
|
int literal = (opts->flags & PARSE_OPT_LITERAL_ARGHELP) || !opts->argh;
|
2009-05-21 09:33:17 +02:00
|
|
|
if (opts->flags & PARSE_OPT_OPTARG)
|
|
|
|
if (opts->long_name)
|
|
|
|
s = literal ? "[=%s]" : "[=<%s>]";
|
|
|
|
else
|
|
|
|
s = literal ? "[%s]" : "[<%s>]";
|
|
|
|
else
|
|
|
|
s = literal ? " %s" : " <%s>";
|
2013-02-09 07:31:09 +01:00
|
|
|
return utf8_fprintf(outfile, s, opts->argh ? _(opts->argh) : _("..."));
|
2009-05-21 09:33:17 +02:00
|
|
|
}
|
|
|
|
|
2007-10-15 01:38:30 +02:00
|
|
|
#define USAGE_OPTS_WIDTH 24
|
|
|
|
#define USAGE_GAP 2
|
|
|
|
|
2010-06-12 14:57:39 +02:00
|
|
|
static int usage_with_options_internal(struct parse_opt_ctx_t *ctx,
|
|
|
|
const char * const *usagestr,
|
|
|
|
const struct option *opts, int full, int err)
|
2007-10-15 01:38:30 +02:00
|
|
|
{
|
2010-05-17 17:34:41 +02:00
|
|
|
FILE *outfile = err ? stderr : stdout;
|
|
|
|
|
2009-03-08 19:16:58 +01:00
|
|
|
if (!usagestr)
|
|
|
|
return PARSE_OPT_HELP;
|
|
|
|
|
2010-06-12 14:57:39 +02:00
|
|
|
if (!err && ctx && ctx->flags & PARSE_OPT_SHELL_EVAL)
|
|
|
|
fprintf(outfile, "cat <<\\EOF\n");
|
|
|
|
|
2012-05-06 16:23:51 +02:00
|
|
|
fprintf_ln(outfile, _("usage: %s"), _(*usagestr++));
|
2007-10-14 00:10:51 +02:00
|
|
|
while (*usagestr && **usagestr)
|
2012-05-06 16:23:51 +02:00
|
|
|
/* TRANSLATORS: the colon here should align with the
|
|
|
|
one in "usage: %s" translation */
|
|
|
|
fprintf_ln(outfile, _(" or: %s"), _(*usagestr++));
|
2008-06-14 09:27:21 +02:00
|
|
|
while (*usagestr) {
|
2012-05-06 16:23:51 +02:00
|
|
|
if (**usagestr)
|
|
|
|
fprintf_ln(outfile, _(" %s"), _(*usagestr));
|
|
|
|
else
|
|
|
|
putchar('\n');
|
2008-06-14 09:27:21 +02:00
|
|
|
usagestr++;
|
|
|
|
}
|
2007-10-15 01:38:30 +02:00
|
|
|
|
|
|
|
if (opts->type != OPTION_GROUP)
|
2010-05-17 17:34:41 +02:00
|
|
|
fputc('\n', outfile);
|
2007-10-15 01:38:30 +02:00
|
|
|
|
|
|
|
for (; opts->type != OPTION_END; opts++) {
|
|
|
|
size_t pos;
|
|
|
|
int pad;
|
|
|
|
|
|
|
|
if (opts->type == OPTION_GROUP) {
|
2010-05-17 17:34:41 +02:00
|
|
|
fputc('\n', outfile);
|
2007-10-15 01:38:30 +02:00
|
|
|
if (*opts->help)
|
2012-05-06 16:23:51 +02:00
|
|
|
fprintf(outfile, "%s\n", _(opts->help));
|
2007-10-15 01:38:30 +02:00
|
|
|
continue;
|
|
|
|
}
|
2007-11-19 10:21:44 +01:00
|
|
|
if (!full && (opts->flags & PARSE_OPT_HIDDEN))
|
|
|
|
continue;
|
2007-10-15 01:38:30 +02:00
|
|
|
|
2010-05-17 17:34:41 +02:00
|
|
|
pos = fprintf(outfile, " ");
|
2012-02-28 20:06:09 +01:00
|
|
|
if (opts->short_name) {
|
2009-05-07 21:45:42 +02:00
|
|
|
if (opts->flags & PARSE_OPT_NODASH)
|
2010-05-17 17:34:41 +02:00
|
|
|
pos += fprintf(outfile, "%c", opts->short_name);
|
2009-05-07 21:45:42 +02:00
|
|
|
else
|
2010-05-17 17:34:41 +02:00
|
|
|
pos += fprintf(outfile, "-%c", opts->short_name);
|
2009-05-07 21:45:42 +02:00
|
|
|
}
|
2007-10-15 01:38:30 +02:00
|
|
|
if (opts->long_name && opts->short_name)
|
2010-05-17 17:34:41 +02:00
|
|
|
pos += fprintf(outfile, ", ");
|
2007-10-15 01:38:30 +02:00
|
|
|
if (opts->long_name)
|
2012-02-28 20:06:09 +01:00
|
|
|
pos += fprintf(outfile, "--%s", opts->long_name);
|
2009-05-07 21:45:08 +02:00
|
|
|
if (opts->type == OPTION_NUMBER)
|
2013-02-09 07:31:09 +01:00
|
|
|
pos += utf8_fprintf(outfile, _("-NUM"));
|
2007-10-15 01:38:30 +02:00
|
|
|
|
2010-12-02 00:31:36 +01:00
|
|
|
if ((opts->flags & PARSE_OPT_LITERAL_ARGHELP) ||
|
|
|
|
!(opts->flags & PARSE_OPT_NOARG))
|
2010-05-17 17:34:41 +02:00
|
|
|
pos += usage_argh(opts, outfile);
|
2007-10-15 01:38:30 +02:00
|
|
|
|
2007-10-14 00:10:51 +02:00
|
|
|
if (pos <= USAGE_OPTS_WIDTH)
|
|
|
|
pad = USAGE_OPTS_WIDTH - pos;
|
2007-10-15 01:38:30 +02:00
|
|
|
else {
|
2010-05-17 17:34:41 +02:00
|
|
|
fputc('\n', outfile);
|
2007-10-15 01:38:30 +02:00
|
|
|
pad = USAGE_OPTS_WIDTH;
|
|
|
|
}
|
2012-05-06 16:23:51 +02:00
|
|
|
fprintf(outfile, "%*s%s\n", pad + USAGE_GAP, "", _(opts->help));
|
2007-10-15 01:38:30 +02:00
|
|
|
}
|
2010-05-17 17:34:41 +02:00
|
|
|
fputc('\n', outfile);
|
2007-10-14 00:10:51 +02:00
|
|
|
|
2010-06-12 14:57:39 +02:00
|
|
|
if (!err && ctx && ctx->flags & PARSE_OPT_SHELL_EVAL)
|
|
|
|
fputs("EOF\n", outfile);
|
|
|
|
|
2008-06-23 22:28:04 +02:00
|
|
|
return PARSE_OPT_HELP;
|
2007-10-15 01:38:30 +02:00
|
|
|
}
|
2007-10-14 11:05:12 +02:00
|
|
|
|
Fix sparse warnings
Fix warnings from 'make check'.
- These files don't include 'builtin.h' causing sparse to complain that
cmd_* isn't declared:
builtin/clone.c:364, builtin/fetch-pack.c:797,
builtin/fmt-merge-msg.c:34, builtin/hash-object.c:78,
builtin/merge-index.c:69, builtin/merge-recursive.c:22
builtin/merge-tree.c:341, builtin/mktag.c:156, builtin/notes.c:426
builtin/notes.c:822, builtin/pack-redundant.c:596,
builtin/pack-refs.c:10, builtin/patch-id.c:60, builtin/patch-id.c:149,
builtin/remote.c:1512, builtin/remote-ext.c:240,
builtin/remote-fd.c:53, builtin/reset.c:236, builtin/send-pack.c:384,
builtin/unpack-file.c:25, builtin/var.c:75
- These files have symbols which should be marked static since they're
only file scope:
submodule.c:12, diff.c:631, replace_object.c:92, submodule.c:13,
submodule.c:14, trace.c:78, transport.c:195, transport-helper.c:79,
unpack-trees.c:19, url.c:3, url.c:18, url.c:104, url.c:117, url.c:123,
url.c:129, url.c:136, thread-utils.c:21, thread-utils.c:48
- These files redeclare symbols to be different types:
builtin/index-pack.c:210, parse-options.c:564, parse-options.c:571,
usage.c:49, usage.c:58, usage.c:63, usage.c:72
- These files use a literal integer 0 when they really should use a NULL
pointer:
daemon.c:663, fast-import.c:2942, imap-send.c:1072, notes-merge.c:362
While we're in the area, clean up some unused #includes in builtin files
(mostly exec_cmd.h).
Signed-off-by: Stephen Boyd <bebarino@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2011-03-22 08:51:05 +01:00
|
|
|
void NORETURN usage_with_options(const char * const *usagestr,
|
2008-06-23 22:38:58 +02:00
|
|
|
const struct option *opts)
|
2007-11-19 10:21:44 +01:00
|
|
|
{
|
2010-06-12 14:57:39 +02:00
|
|
|
usage_with_options_internal(NULL, usagestr, opts, 0, 1);
|
2008-06-23 22:38:58 +02:00
|
|
|
exit(129);
|
2007-11-19 10:21:44 +01:00
|
|
|
}
|
|
|
|
|
Fix sparse warnings
Fix warnings from 'make check'.
- These files don't include 'builtin.h' causing sparse to complain that
cmd_* isn't declared:
builtin/clone.c:364, builtin/fetch-pack.c:797,
builtin/fmt-merge-msg.c:34, builtin/hash-object.c:78,
builtin/merge-index.c:69, builtin/merge-recursive.c:22
builtin/merge-tree.c:341, builtin/mktag.c:156, builtin/notes.c:426
builtin/notes.c:822, builtin/pack-redundant.c:596,
builtin/pack-refs.c:10, builtin/patch-id.c:60, builtin/patch-id.c:149,
builtin/remote.c:1512, builtin/remote-ext.c:240,
builtin/remote-fd.c:53, builtin/reset.c:236, builtin/send-pack.c:384,
builtin/unpack-file.c:25, builtin/var.c:75
- These files have symbols which should be marked static since they're
only file scope:
submodule.c:12, diff.c:631, replace_object.c:92, submodule.c:13,
submodule.c:14, trace.c:78, transport.c:195, transport-helper.c:79,
unpack-trees.c:19, url.c:3, url.c:18, url.c:104, url.c:117, url.c:123,
url.c:129, url.c:136, thread-utils.c:21, thread-utils.c:48
- These files redeclare symbols to be different types:
builtin/index-pack.c:210, parse-options.c:564, parse-options.c:571,
usage.c:49, usage.c:58, usage.c:63, usage.c:72
- These files use a literal integer 0 when they really should use a NULL
pointer:
daemon.c:663, fast-import.c:2942, imap-send.c:1072, notes-merge.c:362
While we're in the area, clean up some unused #includes in builtin files
(mostly exec_cmd.h).
Signed-off-by: Stephen Boyd <bebarino@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2011-03-22 08:51:05 +01:00
|
|
|
void NORETURN usage_msg_opt(const char *msg,
|
2009-02-02 06:12:58 +01:00
|
|
|
const char * const *usagestr,
|
|
|
|
const struct option *options)
|
|
|
|
{
|
|
|
|
fprintf(stderr, "%s\n\n", msg);
|
|
|
|
usage_with_options(usagestr, options);
|
|
|
|
}
|
|
|
|
|
2010-06-12 14:57:39 +02:00
|
|
|
static int parse_options_usage(struct parse_opt_ctx_t *ctx,
|
|
|
|
const char * const *usagestr,
|
2010-05-17 17:34:41 +02:00
|
|
|
const struct option *opts, int err)
|
2008-06-23 22:28:04 +02:00
|
|
|
{
|
2010-06-12 14:57:39 +02:00
|
|
|
return usage_with_options_internal(ctx, usagestr, opts, 0, err);
|
2008-06-23 22:28:04 +02:00
|
|
|
}
|
|
|
|
|