Stephen Boyd venit, vidit, dixit 16.05.2010 00:17:
> On 05/14/2010 02:42 PM, Michael J Gruber wrote:
>> @@ -792,7 +792,10 @@ static int remove_cmd(int argc, const char **argv, const char *prefix)
>> static int prune(int argc, const char **argv, const char *prefix)
>> {
>> struct notes_tree *t;
>> + int show_only, verbose;
>> struct option options[] = {
>> + OPT_BOOLEAN('n', NULL, &show_only, "do not remove, show only"),
>> + OPT_BOOLEAN('v', NULL, &verbose, "report pruned notes"),
>> OPT_END()
>> };
>>
>
> Why not use the standard OPT__DRY_RUN and OPT__VERBOSE macros?
Because of the part you cut out: complete analogy with "git prune".
I don't mind using the OPT__ions, but suggest that unification as a
separate task (involving all occurences of -v and -n, not just this one).
>
> ---->8----
>
> diff --git a/Documentation/git-notes.txt b/Documentation/git-notes.txt
> index 0f32792..6e53c9a 100644
> --- a/Documentation/git-notes.txt
> +++ b/Documentation/git-notes.txt
> @@ -122,10 +122,12 @@ OPTIONS
> is taken to be in `refs/notes/` if it is not qualified.
>
> -n::
> +--dry-run::
> Do not remove anything; just report the object names whose notes
> would be removed.
>
> -v::
> +--verbose::
> Report all object names whose notes are removed.
>
>
> diff --git a/builtin/notes.c b/builtin/notes.c
> index 8c2ac51..0ae2561 100644
> --- a/builtin/notes.c
> +++ b/builtin/notes.c
> @@ -794,8 +794,8 @@ static int prune(int argc, const char **argv, const char *pr
> struct notes_tree *t;
> int show_only, verbose;
> struct option options[] = {
> - OPT_BOOLEAN('n', NULL, &show_only, "do not remove, show only"),
> - OPT_BOOLEAN('v', NULL, &verbose, "report pruned notes"),
> + OPT__DRY_RUN(&show_only),
> + OPT__VERBOSE(&verbose),
> OPT_END()
> };
>
>
>
--
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