Johannes Schindelin <[email protected]> writes:
> err = fsck_ident(&buffer, &commit->object, options);
> if (err)
> return err;
> + while (skip_prefix(buffer, "author ", &buffer)) {
> + err = report(options, &commit->object,
> FSCK_MSG_MULTIPLE_AUTHORS, "invalid format - multiple 'author' lines");
> + if (err)
> + return err;
> + err = fsck_ident(&buffer, &commit->object, options);
> + if (err)
> + return err;
> + }
Hmph, naively I would have expected that you wouldn't need an
extra call to fsck_ident() here, and instead would see something
like this:
author_count = 0;
while (skip_prefix("author ")) {
author_count++;
... do the existing check as-is ...
}
if (author_count < 1)
err |= report(missing author);
else if (author_count > 1)
err |= report(multiple authors);
--
To unsubscribe from this list: send the line "unsubscribe git" in
