Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44105 )
Change subject: lib/gcov: Remove assert(0) ......................................................................
Patch Set 1: Code-Review+2
(1 comment)
https://review.coreboot.org/c/coreboot/+/44105/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/44105/1//COMMIT_MSG@10 PS1, Line 10: custom assert macro Should we stop hacking/using assert() and provide a standard implementation instead? It seems to be custom on all ends by now, it doesn't stop execution (well, that's configurable) and developers have adapted to that and use it as a short-hand check. We could rename the current implementation and all its callers.