> > https://review.coreboot.org/c/coreboot/+/51825 proposes getting rid of the > > rule to make if-statement blocks (and the like) as short as possible. > > The rationale is to encourage a style that avoids subtle bugs which then > > need to be found by tooling such as Coverity Scan and fixed by commits like > > https://review.coreboot.org/51786.
My main case is really just that I think the whole premise is incorrect: these kinds of errors can be found by checkpatch, at upload time. There should be no risk of them slipping into the code and thus we don't need to add any inconvenience for humans to prevent something that can already be automatically be prevented by a computer. I've double-checked that now and realized that checkpatch currently doesn't look for this like I thought it did. But there is a fix, and it works very well (I've found no real false-positives in the whole coreboot tree): https://review.coreboot.org/c/coreboot/+/51838. I am also (re-)submitting this to Linux so hopefully we wouldn't be out of sync for too long if we want to apply it immediately. _______________________________________________ coreboot mailing list -- coreboot@coreboot.org To unsubscribe send an email to coreboot-le...@coreboot.org
