Re: [PATCH 1/2] diff: change semantics of "ignore whitespace" options

[Junio C Hamano <gitster@xxxxxxxxx>]

Jeff King <peff@xxxxxxxx> writes:

>> +	if ((XDF_IGNORE_WHITESPACE|
>> +	     XDF_IGNORE_WHITESPACE_CHANGE|
>> +	     XDF_IGNORE_WHITESPACE_AT_EOL) & options->xdl_opts) {
>> +		DIFF_OPT_SET(options, DIFF_FROM_CONTENTS);
>> +	} else {
>> +		DIFF_OPT_CLR(options, DIFF_FROM_CONTENTS);
>> +	}
>
> We have DIFF_XDL_TST these days;...

Heh, nobody seems to use it though.

Now the part reads

	if (DIFF_XDL_TST(options, IGNORE_WHITESPACE) ||
	    DIFF_XDL_TST(options, IGNORE_WHITESPACE_CHANGE) ||
	    DIFF_XDL_TST(options, IGNORE_WHITESPACE_AT_EOL))
		DIFF_OPT_SET(options, DIFF_FROM_CONTENTS);
	else
		DIFF_OPT_CLR(options, DIFF_FROM_CONTENTS);

Thanks.
--
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