Better default for MBEDTLS_CHECK_RETURN in config.h
An empty expansion is possible, but as documented its effect is to disable
the feature, so that isn't a good example. Instead, use the GCC
implementation as the default: it's plausible that it could work even on
compilers that don't advertise themselves as sufficiently GCC-like to define
__GNUC__, and if not it gives users a concrete idea of what the macro is
supposed to do.
Signed-off-by: Gilles Peskine <Gilles.Peskine@arm.com>
diff --git a/include/mbedtls/mbedtls_config.h b/include/mbedtls/mbedtls_config.h
index 2110cc2..13c86ed 100644
--- a/include/mbedtls/mbedtls_config.h
+++ b/include/mbedtls/mbedtls_config.h
@@ -3080,7 +3080,7 @@
* If the implementation here is empty, this will effectively disable the
* checking of functions' return values.
*/
-//#define MBEDTLS_CHECK_RETURN
+//#define MBEDTLS_CHECK_RETURN __attribute__(__warn_unused_result__)
/* PSA options */
/**