Move the description of MBEDTLS_TEST_DEFINES_ZEROIZE to before its use

Signed-off-by: Tom Cosgrove <tom.cosgrove@arm.com>
diff --git a/include/mbedtls/platform_util.h b/include/mbedtls/platform_util.h
index 70dc51a..62f6d70 100644
--- a/include/mbedtls/platform_util.h
+++ b/include/mbedtls/platform_util.h
@@ -221,6 +221,11 @@
 #define MBEDTLS_IGNORE_RETURN(result) ((void) !(result))
 #endif
 
+/* If the following macro is defined, the library is being built by the test
+ * framework, and the framework is going to provide a replacement
+ * mbedtls_platform_zeroize() using a preprocessor macro, so the function
+ * declaration should be omitted.  */
+#if !defined(MBEDTLS_TEST_DEFINES_ZEROIZE) //no-check-names
 /**
  * \brief       Securely zeroize a buffer
  *
@@ -243,18 +248,9 @@
  * \param len   Length of the buffer in bytes
  *
  */
-#if !defined(MBEDTLS_TEST_DEFINES_ZEROIZE)
 void mbedtls_platform_zeroize(void *buf, size_t len);
 #endif
 
-/* MBEDTLS_TEST_DEFINES_ZEROIZE
- *
- * Indicates that the library is being built by the test framework, and the
- * framework is going to provide a replacement mbedtls_platform_zeroize()
- * using a pre-processor macro, so the function declaration should be omitted.
- */
-//#define MBEDTLS_TEST_DEFINES_ZEROIZE
-
 #if defined(MBEDTLS_HAVE_TIME_DATE)
 /**
  * \brief      Platform-specific implementation of gmtime_r()