On Thu, Jun 29, 2017 at 2:01 PM, Stefan Beller wrote:
> On Tue, Jun 27, 2017 at 10:06 PM, Junio C Hamano wrote:
>> Junio C Hamano writes:
>> This probably deserves a bit more illustration of how I envision the
>> code should evolve.
>> ...
> It turns out that the code here is rather a very loose
On Tue, Jun 27, 2017 at 10:06 PM, Junio C Hamano wrote:
> Junio C Hamano writes:
>
>> Looking at the implementation of get_ws_cleaned_string() that is the
>> workhorse of emitted_symbol_cmp_no_ws(), it seems to be doing wrong
>> things for various "ignore whitespace" options (i.e. there is only
>
Stefan Beller writes:
> Originally I wanted to do that, see prep work in [1], but Jeff explained that
> the additional pointer in the compare function is **not** supposed to be
> a additional payload (such as the diff options specifying the white space
> options.)
>
> [1] https://public-inbox.org
On Tue, Jun 27, 2017 at 10:06 PM, Junio C Hamano wrote:
> Junio C Hamano writes:
>
>> Looking at the implementation of get_ws_cleaned_string() that is the
>> workhorse of emitted_symbol_cmp_no_ws(), it seems to be doing wrong
>> things for various "ignore whitespace" options (i.e. there is only
>
Junio C Hamano writes:
> Looking at the implementation of get_ws_cleaned_string() that is the
> workhorse of emitted_symbol_cmp_no_ws(), it seems to be doing wrong
> things for various "ignore whitespace" options (i.e. there is only
> one implementation, while "git diff" family takes things like
Stefan Beller writes:
> Reuse the compare function from the hash map instead of calling the
> compare function directly. Then we pick the correct compare function
> when told to compare ignoring white space.
>
> Signed-off-by: Stefan Beller
> ---
> diff.c | 3 +--
> t/t4015
Reuse the compare function from the hash map instead of calling the
compare function directly. Then we pick the correct compare function
when told to compare ignoring white space.
Signed-off-by: Stefan Beller
---
diff.c | 3 +--
t/t4015-diff-whitespace.sh | 65 ++
7 matches
Mail list logo