Allan Caffee <allan.caffee@xxxxxxxxx> writes: > The problem with making it an index into the column_colors array is that > we don't have a convenient place to test whether the user actually wants > color. We can't do it in strbuf_write_column because AFAIK there's no > way to get the rev-info to test the options. I suppose we could define > GIT_NOT_A_COLOR to -1 and just set the color to that when we don't > intend to use color. (Either way I should probably change that to an > unsigned short.) What do you think? Sounds sensible. -- 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