On 13.01.21 13:40, [ext] Christian Storm wrote:
> Hi Jan,
>
>> Lost a newline here. Fixing up while dropping that INFO tag.
>
> Hm, why did you remove the
> do { ... } while (0)
> enclosing in INFO() while your modifications of the patch?
>
> It now reads (see [1])
> #define INFO(fmt, ...) \
> PrintC(EFI_LIGHTGRAY, fmt, ##__VA_ARGS__);
> which means it effectively expands to double ;; and it breaks,
> e.g., for some if-else constructs such as
> if (condition)
> INFO(L"foo");
> else
> ERROR(L"bar");
> as the double ;; "eats" the else part so the compiler is unhappy.
>
> Is there a reason for this I'm missing?
>
Nope, I just missed the remove the additional ";" as well. Fixed.
Jan
--
Siemens AG, T RDA IOT
Corporate Competence Center Embedded Linux
--
You received this message because you are subscribed to the Google Groups "EFI
Boot Guard" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To view this discussion on the web visit
https://groups.google.com/d/msgid/efibootguard-dev/78e1eb35-7091-1d24-c015-a1340ee21620%40siemens.com.
