On Tue, Apr 16, 2024, Jakub Kicinski wrote:
> Apparently it's more legal to pass the format as NULL, than
> it is to use an empty string. Clang complains about empty
> formats:
> 
> ./../kselftest_harness.h:1207:30: warning: format string is empty
> [-Wformat-zero-length]
>  1207 |            diagnostic ? "%s" : "", diagnostic);
>       |                                 ^~
> 1 warning generated.
> 
> Reported-by: Sean Christopherson <[email protected]>
> Link: https://lore.kernel.org/all/[email protected]
> Fixes: 378193eff339 ("selftests: kselftest_harness: let PASS / FAIL provide 
> diagnostic")
> Signed-off-by: Jakub Kicinski <[email protected]>

Thanks Jakub!

Tested-by: Sean Christopherson <[email protected]>

Reply via email to