pw_assert: Simplify PW_CHECK implementation
PW_CHECK optionally accepts a message. Rather than checking if there is
a message and either passing "" or __VA_ARGS__, simply pass
"" __VA_ARGS__. If there are no arguments, this expands to "". If there
are arguments, this expands to "" "format string".
This change assumes PW_CHECK format strings are string literals, but
this is already required when using pw_assert_log and pw_log_tokenized.
Change-Id: If76e7995a81a74ea746d2fd0acc0ec15231e4671
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/70000
Commit-Queue: Wyatt Hepler <hepler@google.com>
Reviewed-by: Armando Montanez <amontanez@google.com>
diff --git a/pw_assert/docs.rst b/pw_assert/docs.rst
index 41ea1ca..d528a87 100644
--- a/pw_assert/docs.rst
+++ b/pw_assert/docs.rst
@@ -37,6 +37,9 @@
PW_CHECK_INT_LE(ItemCount(), 100);
PW_CHECK_INT_LE(ItemCount(), 100, "System state: %s", GetStateStr());
+ To ensure compatibility with :ref:`module-pw_assert_log` and
+ :ref:`module-pw_log_tokenized`, the message must be a string literal.
+
Example
=======