Re: [PATCH 0/9] saner xdiff hunk callbacks

2018-11-02 Thread Junio C Hamano
Jeff King writes: > ... it seems rather silly that we have xdiff generate a string > just so that we can parse it from within the same process. So instead I > improved the xdiff interface to pass the actual integers out, made use > of it as appropriate, and then in the final patch we can just

[PATCH 0/9] saner xdiff hunk callbacks

2018-11-02 Thread Jeff King
As part of my -Wunused-parameter hunt, I noticed that parse_hunk_header() can read out-of-bounds in the array it is given. But it turns out not to be a big problem because we only ever use it to parse lines that xdiff has just generated. This is fixable, and I'll show my patch to do so at the end