Just spell it "struct rev_name"; it makes it more clear what is going on. Signed-off-by: Junio C Hamano <gitster@xxxxxxxxx> --- builtin/name-rev.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/builtin/name-rev.c b/builtin/name-rev.c index 1b37458..8af2cfa 100644 --- a/builtin/name-rev.c +++ b/builtin/name-rev.c @@ -7,11 +7,11 @@ #define CUTOFF_DATE_SLOP 86400 /* one day */ -typedef struct rev_name { +struct rev_name { const char *tip_name; int generation; int distance; -} rev_name; +}; static long cutoff = LONG_MAX; @@ -43,7 +43,7 @@ static void name_rev(struct commit *commit, } if (name == NULL) { - name = xmalloc(sizeof(rev_name)); + name = xmalloc(sizeof(struct rev_name)); commit->util = name; goto copy_data; } else if (name->distance > distance) { -- 1.7.12.286.g9df01f7 -- 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