Re: [PATCH] notes: dry-run and verbose options for prune
From: Michael J Gruber <hidden>
Date: 2016-06-15 22:48:48
Stephen Boyd venit, vidit, dixit 16.05.2010 00:17:
On 05/14/2010 02:42 PM, Michael J Gruber wrote:quoted
@@ -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).
quoted hunk ↗ jump to hunk
---->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() };