Re: [PATCH v6 04/19] fsck: Offer a function to demote fsck errors to warnings

2015-06-22 Thread Johannes Schindelin
Hi Junio, On 2015-06-19 21:26, Junio C Hamano wrote: > Johannes Schindelin writes: > >> +static inline int substrcmp(const char *string, int len, const char *match) >> +{ >> +int match_len = strlen(match); >> +if (match_len != len) >> +return -1; >> +return memcmp(string,

Re: [PATCH v6 04/19] fsck: Offer a function to demote fsck errors to warnings

2015-06-21 Thread Junio C Hamano
Johannes Schindelin writes: > How about I implement your suggestion tomorrow, then show the diff > between the two versions and we can assess what looks to be simpler > (i.e. more maintainable)? I'm indifferent at this point, partly because as we agree that what you have as long as it works is o

Re: [PATCH v6 04/19] fsck: Offer a function to demote fsck errors to warnings

2015-06-21 Thread Johannes Schindelin
Hi Junio, On 2015-06-21 19:36, Junio C Hamano wrote: > Johannes Schindelin writes: > >> Hmm. I really do not like that kind of thinking, i.e. having to >> duplicate, then modify data to be able to call the API, only to have >> to modify the data back afterwards, and eventually having to >> unall

Re: [PATCH v6 04/19] fsck: Offer a function to demote fsck errors to warnings

2015-06-21 Thread Junio C Hamano
Johannes Schindelin writes: > Hmm. I really do not like that kind of thinking, i.e. having to > duplicate, then modify data to be able to call the API, only to have > to modify the data back afterwards, and eventually having to > unallocate the data in all code paths. That feels just very inelega

Re: [PATCH v6 04/19] fsck: Offer a function to demote fsck errors to warnings

2015-06-21 Thread Johannes Schindelin
Hi Junio, On 2015-06-19 21:26, Junio C Hamano wrote: > Johannes Schindelin writes: > >> +static inline int substrcmp(const char *string, int len, const char *match) >> +{ >> +int match_len = strlen(match); >> +if (match_len != len) >> +return -1; >> +return memcmp(string,

Re: [PATCH v6 04/19] fsck: Offer a function to demote fsck errors to warnings

2015-06-19 Thread Junio C Hamano
Johannes Schindelin writes: > diff --git a/fsck.c b/fsck.c > index da5717c..8c3caff 100644 > --- a/fsck.c > +++ b/fsck.c > @@ -103,13 +103,85 @@ static int fsck_msg_type(enum fsck_msg_id msg_id, > { > int msg_type; > > - msg_type = msg_id_info[msg_id].msg_type; > - if (options->s

[PATCH v6 04/19] fsck: Offer a function to demote fsck errors to warnings

2015-06-19 Thread Johannes Schindelin
There are legacy repositories out there whose older commits and tags have issues that prevent pushing them when 'receive.fsckObjects' is set. One real-life example is a commit object that has been hand-crafted to list two authors. Often, it is not possible to fix those issues without disrupting th