build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43717 )
Change subject: payloads/libpayload/libc: upgrade "qsort.c" to v1.15 ......................................................................
Patch Set 6:
(1 comment)
https://review.coreboot.org/c/coreboot/+/43717/6/payloads/libpayload/libc/qs... File payloads/libpayload/libc/qsort.c:
https://review.coreboot.org/c/coreboot/+/43717/6/payloads/libpayload/libc/qs... PS6, Line 73: #define vecswap(a, b, n) if ((n) > 0) swapfunc(a, b, n, swaptype) Macros starting with if should be enclosed by a do - while loop to avoid possible if/else logic defects