Re: [PATCH v3 5/7] convert_to_git(): safe_crlf/checksafe becomes int conv_flags
- From
Lars Schneider <larsxschneider@gmail.com>
- Date
- Jan 8, 2018, 22:47 UTC
- Message-ID
- <FFFF054A-6049-49A6-9B31-3E2D6585B21A@gmail.com>
- In-Reply-To
- <xmqq4lnw82hi.fsf@gitster.mtv.corp.google.com>
Show 24 quoted lines
> On 08 Jan 2018, at 22:28, Junio C Hamano <gitster@pobox.com> wrote:
>
> lars.schneider@autodesk.com writes:
>
>> diff --git a/sha1_file.c b/sha1_file.c
>> index afe4b90f6e..dcb02e9ffd 100644
>> --- a/sha1_file.c
>> +++ b/sha1_file.c
>> @@ -75,14 +75,14 @@ static struct cached_object *find_cached_object(const unsigned char *sha1)
>> }
>>
>>
>> -static enum safe_crlf get_safe_crlf(unsigned flags)
>> +static int get_conv_flags(unsigned flags)
>> {
>> if (flags & HASH_RENORMALIZE)
>> - return SAFE_CRLF_RENORMALIZE;
>> + return CONV_EOL_RENORMALIZE;
>> else if (flags & HASH_WRITE_OBJECT)
>> - return safe_crlf;
>> + return global_conv_flags_eol | CONV_WRITE_OBJECT;
>
> This macro has not yet introduced at this point (it appears in 6/7
> if I am not mistaken).Nice catch. I'll fix that in the next iteration.
Is it OK if I send the next iteration soon or would you prefer it if I wait until after 2.16 release?
Plus, is it ok to keep the base of the series or would you prefer it if I rebase it to the latest master (because of a minor conflict)?
Thanks, Lars