diff options
| author | Junio C Hamano <junkio@cox.net> | 2006-01-25 01:03:18 -0800 |
|---|---|---|
| committer | Junio C Hamano <junkio@cox.net> | 2006-01-28 00:09:38 -0800 |
| commit | 46a6c2620ba421397eec627b8eb18eb530e694fc (patch) | |
| tree | 9222e809b0d77ad10e7304073d4c053c7edd9266 /describe.c | |
| parent | 6b94f1e404afc552e5139c4357331843f5be61ad (diff) | |
| download | git-46a6c2620ba421397eec627b8eb18eb530e694fc.tar.gz git-46a6c2620ba421397eec627b8eb18eb530e694fc.tar.xz | |
abbrev cleanup: use symbolic constants
The minimum length of abbreviated object name was hardcoded in
different places to be 4, risking inconsistencies in the future.
Also there were three different "default abbreviation
precision". Use two C preprocessor symbols to clean up this
mess.
Signed-off-by: Junio C Hamano <junkio@cox.net>
Diffstat (limited to 'describe.c')
| -rw-r--r-- | describe.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/describe.c b/describe.c index 4866510ea..ff6574261 100644 --- a/describe.c +++ b/describe.c @@ -11,7 +11,6 @@ static const char describe_usage[] = static int all = 0; /* Default to annotated tags only */ static int tags = 0; /* But allow any tags if --tags is specified */ -#define DEFAULT_ABBREV 8 /* maybe too many */ static int abbrev = DEFAULT_ABBREV; static int names = 0, allocs = 0; @@ -155,7 +154,7 @@ int main(int argc, char **argv) tags = 1; else if (!strncmp(arg, "--abbrev=", 9)) { abbrev = strtoul(arg + 9, NULL, 10); - if (abbrev < 4 || 40 <= abbrev) + if (abbrev < MINIMUM_ABBREV || 40 <= abbrev) abbrev = DEFAULT_ABBREV; } else |