Carlo Arenas <carenas@xxxxxxxxx> writes: > Signed-off-by: Carlo Marcelo Arenas Belón <carenas@xxxxxxxxx> Do you mean Tested-by: (meaning, you actually saw the breakage with SunCC without the patch and also saw the patch fixed the breakage)? > clang with -Wpedantic also catch this (at least with Apple LLVM > version 10.0.0); recent versions of gcc also include that flag and at > least 8.2.0 shows a warning for it, so it might be worth adding it to > developer mode (maybe under the pedantic DEVOPTS). Nice to know. > ..., would this be something I should be adding?, what are the expectations around > warnings and compilers? It may be a worthwhile thing to discuss, and as a discussion starter, a patch would help ;-).