Michael J Gruber <g...@grubix.eu> writes:
>>What problem are you solving?
>
> Sorry, I forgot about that change and failed to mention it.
>
> It makes no difference in the non-debug case which cares about the
> Boolean only. In the debug case, I want to distinguish between
> annotated and lightweight tags, just like describe --debug
> does. By adding 1 via deref and passing this down, I know that an
> annotated tag gets the value 2, a lightweight tag 1 and everything
> else 0, just like describe --tags.
If you want to only affect debug display, perhaps you should start
with a patch like the attached, before adding any debug code as a
preparatory step. Then add your debugging thing, _WITHOUT_ the
increment in name_rev(), that uses from_tag to choose between
lightweight and annotated as a separate step.
When we decide that it would make sense to give precedence to
annotated ones over lightweight ones in is_better_name(), the
comparison can be further tweaked to actually compare values of the
from_tag thing in *name and the current candidate. That would have
to be a separate step, as it changes the semantics (I suspect it
would be a better change but it may not be).
How does that sound?
-- >8 --
Subject: name-rev: allow to tell annotated and lightweight tags apart
We do not use this feature yet, but from_tag that is passed around
and kept in the rev_name structure now takes three values, instead
of a boolean "did this come from refs/tags/ hierarchy?". A new
value '2' is "this is an annotated tag that came from refs/tags/
hierarchy".
---
builtin/name-rev.c | 9 ++++++---
1 file changed, 6 insertions(+), 3 deletions(-)
diff --git a/builtin/name-rev.c b/builtin/name-rev.c
index bf7ed015ae..fe2d306e7c 100644
--- a/builtin/name-rev.c
+++ b/builtin/name-rev.c
@@ -41,7 +41,7 @@ static int is_better_name(struct rev_name *name,
* We know that at least one of them is a non-tag at this point.
* favor a tag over a non-tag.
*/
- if (name->from_tag != from_tag)
+ if (!!name->from_tag != !!from_tag)
return from_tag;
/*
@@ -247,8 +247,11 @@ static int name_ref(const char *path, const struct
object_id *oid, int flags, vo
}
if (o && o->type == OBJ_COMMIT) {
struct commit *commit = (struct commit *)o;
- int from_tag = starts_with(path, "refs/tags/");
-
+ int from_tag;
+ if (starts_with(path, "refs/tags/"))
+ from_tag = 1 + deref;
+ else
+ from_tag = 0;
if (taggerdate == ULONG_MAX)
taggerdate = ((struct commit *)o)->date;
path = name_ref_abbrev(path, can_abbreviate_output);
