On Mon, Jul 20, 2015 at 12:12 PM, Junio C Hamano <[email protected]> wrote: > Eric Sunshine <[email protected]> writes: > >> On Sat, Jul 18, 2015 at 3:12 PM, Karthik Nayak <[email protected]> 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 <[email protected]> >>> --- >>> 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.
It's been years since I encountered such a compiler, so it's possible that my brain is conflating different cases of trailing commas... -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
