On Sat, May 01 2021, Jeff King wrote: > Many calls to report() in fsck_tree() are kept on a single line and are > quite long. Most were pretty big to begin with, but have gotten even > longer over the years as we've added more parameters. Let's accept the > churn of wrapping them in order to conform to our usual line limits. If we're going to have the churn I'd say just wrap the rest of the file as well, now it's mostly-consistent in having these long lines. FWIW I think having the long lines makes things more readable in this case, but the inconsistency is worse. If you do take me up on refactoring it more generally, there's also the one mis-alignment of an argument list to report() in verify_headers(). I wonder if this whole thing wouldn't be better by declaring the format in the msg_id_info struct. I.e. add this to fsck.h, but that's an even bigger change...
