Kato Kazuyoshi <kato.kazuyoshi@xxxxxxxxx> writes: > On Mon, Oct 17, 2011 at 10:51 AM, Junio C Hamano <gitster@xxxxxxxxx> wrote: >> >> I think a better organization would be >> >> [1/2] change code so that $diff_class does not have leading SP >> optionally catch a case where $diff_class stays empty as an error? >> > > I think we don't have to treat empty $diff_class as an error because > $diff_class will be an empty when $line is around modification > (ex. foo or quux). > > foo > - bar > + baz > quuz Ok. then even though class="diff " is not any different from class="diff", I have to say generated html output becomes aesthetically less nice with this change; we may want to do something about it _iff_ it does not take too much change, but as I said I do not care too much about it. -- 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