Re: [PATCHv2 0/8] Add color test for range-diff, simplify diff.c

2018-08-02 Thread Junio C Hamano
Stefan Beller writes: > On Wed, Aug 1, 2018 at 12:13 PM Junio C Hamano wrote: >> >> Stefan Beller writes: >> >> > Stefan Beller (8): >> > test_decode_color: understand FAINT and ITALIC >> > t3206: add color test for range-diff --dual-color >> > diff.c: simplify caller of emit_line_0 >> >

Re: [PATCHv2 0/8] Add color test for range-diff, simplify diff.c

2018-08-01 Thread Stefan Beller
On Wed, Aug 1, 2018 at 12:13 PM Junio C Hamano wrote: > > Stefan Beller writes: > > > Stefan Beller (8): > > test_decode_color: understand FAINT and ITALIC > > t3206: add color test for range-diff --dual-color > > diff.c: simplify caller of emit_line_0 > > diff.c: reorder arguments for em

Re: [PATCHv2 0/8] Add color test for range-diff, simplify diff.c

2018-08-01 Thread Junio C Hamano
Stefan Beller writes: > Stefan Beller (8): > test_decode_color: understand FAINT and ITALIC > t3206: add color test for range-diff --dual-color > diff.c: simplify caller of emit_line_0 > diff.c: reorder arguments for emit_line_ws_markup > diff.c: add set_sign to emit_line_0 > diff: us

[PATCHv2 0/8] Add color test for range-diff, simplify diff.c

2018-07-30 Thread Stefan Beller
addressed all of Erics feedback: * reworded commit messages * dropped q_to_tab and use cat instead * use -\EOF isntead of -EOF Thanks, Stefan Stefan Beller (8): test_decode_color: understand FAINT and ITALIC t3206: add color test for range-diff --dual-color diff.c: simplify caller of emit_l