On Mon, Jan 08, 2018 at 11:47:20PM +0100, Lars Schneider wrote:
>
> > On 08 Jan 2018, at 22:28, Junio C Hamano <gitster@xxxxxxxxx> wrote:
> >
> > lars.schneider@xxxxxxxxxxxx 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
I noticed the missing macro as well, while doing the rebase
to git.git/master, but forget to mention it here on the list
Lars, if you want, please have a look here:
https://github.com/tboegi/git/tree/180108-1858-For-lars-schneider-encode-V3C
