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 seanjc@google.com Link: https://lore.kernel.org/all/20240409224256.1581292-1-seanjc@google.com Fixes: 378193eff339 ("selftests: kselftest_harness: let PASS / FAIL provide diagnostic") Signed-off-by: Jakub Kicinski kuba@kernel.org
Thanks Jakub!
Tested-by: Sean Christopherson seanjc@google.com