[coreboot-gerrit] Change in coreboot[master]: Makefile.inc: Fix empty output when processing C struct files in CBFS

[Hung-Te Lin (Code Review)]

Hung-Te Lin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/48201 )

Change subject: Makefile.inc: Fix empty output when processing C struct files in CBFS
......................................................................

Patch Set 5:

(2 comments)

https://review.coreboot.org/c/coreboot/+/48201/5/Makefile.inc 
File Makefile.inc:

https://review.coreboot.org/c/coreboot/+/48201/5/Makefile.inc@336 
PS5, Line 336: <
> Yes.

So maybe just use $(1) instead? It may be more clear than $@ and $< given the order was reversed in the rule.

https://review.coreboot.org/c/coreboot/+/48201/5/Makefile.inc@336 
PS5, Line 336:
> Can't say that I follow. […]

I agree with Nico - having a space before \n is usually considered as wrong. Yes it may be ugly but that's how printf and debug messages already look like. So I'd also recommend removing the extra space.