build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/40123 )
Change subject: payloads: Remove unneeded whitespaces ......................................................................
Patch Set 1:
(11 comments)
https://review.coreboot.org/c/coreboot/+/40123/1/payloads/libpayload/arch/ar... File payloads/libpayload/arch/arm/virtual.c:
https://review.coreboot.org/c/coreboot/+/40123/1/payloads/libpayload/arch/ar... PS1, Line 58: * fault when accessed for the first time */ \ Avoid unnecessary line continuations
https://review.coreboot.org/c/coreboot/+/40123/1/payloads/libpayload/curses/... File payloads/libpayload/curses/curses.h:
https://review.coreboot.org/c/coreboot/+/40123/1/payloads/libpayload/curses/... PS1, Line 914: #define getyx(win,y,x) (y = getcury(win), x = getcurx(win)) space required after that ',' (ctx:VxV)
https://review.coreboot.org/c/coreboot/+/40123/1/payloads/libpayload/curses/... PS1, Line 914: #define getyx(win,y,x) (y = getcury(win), x = getcurx(win)) space required after that ',' (ctx:VxV)
https://review.coreboot.org/c/coreboot/+/40123/1/payloads/libpayload/curses/... PS1, Line 955: #define wstandout(win) (wattrset(win,A_STANDOUT)) space required after that ',' (ctx:VxV)
https://review.coreboot.org/c/coreboot/+/40123/1/payloads/libpayload/curses/... PS1, Line 956: #define wstandend(win) (wattrset(win,A_NORMAL)) space required after that ',' (ctx:VxV)
https://review.coreboot.org/c/coreboot/+/40123/1/payloads/libpayload/curses/... File payloads/libpayload/curses/form/form.h:
https://review.coreboot.org/c/coreboot/+/40123/1/payloads/libpayload/curses/... PS1, Line 115: unsigned short status; /* flags */ please, no spaces at the start of a line
https://review.coreboot.org/c/coreboot/+/40123/1/payloads/libpayload/curses/... PS1, Line 153: void* (*copyarg)(const void *); /* copy fieldtype arg */ please, no spaces at the start of a line
https://review.coreboot.org/c/coreboot/+/40123/1/payloads/libpayload/curses/... PS1, Line 153: void* (*copyarg)(const void *); /* copy fieldtype arg */ function definition argument 'const void *' should also have an identifier name
https://review.coreboot.org/c/coreboot/+/40123/1/payloads/libpayload/curses/... PS1, Line 320: bool (* const prev_choice)(FIELD *,const void *)); space required after that ',' (ctx:OxV)
https://review.coreboot.org/c/coreboot/+/40123/1/payloads/libpayload/curses/... File payloads/libpayload/curses/form/fty_regex.c:
https://review.coreboot.org/c/coreboot/+/40123/1/payloads/libpayload/curses/... PS1, Line 67: #define INIT register char *sp = RegEx_Init(instring); macros should not use a trailing semicolon
https://review.coreboot.org/c/coreboot/+/40123/1/payloads/libpayload/curses/... File payloads/libpayload/curses/menu/m_global.c:
https://review.coreboot.org/c/coreboot/+/40123/1/payloads/libpayload/curses/... PS1, Line 72: ' ', /* Pad character */ please, no spaces at the start of a line