Re: [PATCH v13 7/8] history: create squashed commits without editing

Junio C Hamano <[email protected]> writes:

> Phillip Wood <[email protected]> writes:
>
>> It's piped into "tr" to check that
>>
>> 	target = lookup_commit_reference_by_name(s);
>> 	if (target && istarts_with(oid_to_hex(&target->object.oid), s))
>>
>> uses istarts_with() so that we accept uppercase object ids.
>
> You should check in with brian about istarts_with() that is used
> here, as I suspect he missed it in his effort to declare uppercase
> hexadecimal literals invalid.

Ah no, no, no.  This is a new code introduced by this series.

And as I said, I do not see why we want to be case insensitive with
this comparison.