Re: [PATCH v2 6/7] color: support "italic" attribute

> Jeff King <peff@peff.net> writes:
> 
> > We already support bold, underline, and similar attributes.
> > Let's add italic to the mix.  According to the Wikipedia
> > page on ANSI colors, this attribute is "not widely
> > supported", but it does seem to work on my xterm.
> > ...
> > @@ -133,6 +133,7 @@ static int parse_attr(const char *name, size_t len)
> >  #define ATTR(x, val, neg) { (x), strlen(x), (val), (neg) }
> 
> I see this one was from an earlier reroll that did not use sizeof(x)-1;
> easy to fixup, though ;-)