Eric Sunshine <sunshine@xxxxxxxxxxxxxx> writes: > On Sat, Jul 18, 2015 at 3:12 PM, Karthik Nayak <karthik.188@xxxxxxxxx> wrote: >> Add a new atom "align" and support %(align:X) where X is a number. >> This will align the preceeding atom value to the left followed by >> spaces for a total length of X characters. If X is less than the item >> size, the entire atom value is printed. >> >> Signed-off-by: Karthik Nayak <karthik.188@xxxxxxxxx> >> --- >> diff --git a/ref-filter.c b/ref-filter.c >> index 7561727..b81a08d 100644 >> --- a/ref-filter.c >> +++ b/ref-filter.c >> @@ -53,6 +55,7 @@ static struct { >> { "flag" }, >> { "HEAD" }, >> { "color" }, >> + { "align" }, > > Not a new issue, but some compilers (Solaris?) complain about the > trailing comma. Hmm, are you sure? I thought we avoid trailing comma for enum definitions, but not a list of values like this. -- 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