On Sun, Feb 14, 2010 at 08:36:36PM -0600, Jonathan Nieder wrote: > > --- a/builtin-var.c > > +++ b/builtin-var.c > > @@ -6,7 +6,7 @@ > > #include "cache.h" > > #include "exec_cmd.h" > > > > -static const char var_usage[] = "git var [-l | <variable>]"; > > +static const char var_usage[] = "git var (-l | <variable>)"; > > static const char *editor(int flag) > > { > > As Junio noticed, this doesn’t apply. The problem is a missing > blank line between the static const char var_usage and static const > char *editor lines. Worse still, I have no clue how it happened. I think it was a stray deletion in your editor. The patch is actually corrupt (the hunk claims 7 lines, but contains only 6). And you put text after the "---", so presumably you did so with an editor. So I think it is probably not a bug, but just a weird once-off user error. Or maybe cosmic rays ate your line feed. -Peff -- 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