[coreboot-gerrit] Change in ...coreboot[master]: commonlib: Add Bubble sort algorithm

[Julius Werner (Code Review)]

Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31544 )

Change subject: commonlib: Add Bubble sort algorithm
......................................................................

Patch Set 3:

(1 comment)

https://review.coreboot.org/#/c/31544/3/src/include/stdlib.h 
File src/include/stdlib.h:

https://review.coreboot.org/#/c/31544/3/src/include/stdlib.h@13 
PS3, Line 13: 			} while (0)
> Uhhh... not sure how I feel about this. Using lvalues in macros is commonly frowned upon. […]

edit: Actually, if you do decide to leave it like this, maybe call it SWAP() because then it's clearly not (and not treatable like) a function.