]> git.ipfire.org Git - thirdparty/systemd.git/commitdiff
assert-util: drop message argument of assert_log()
authorYu Watanabe <watanabe.yu+github@gmail.com>
Fri, 15 Aug 2025 04:51:25 +0000 (13:51 +0900)
committerYu Watanabe <watanabe.yu+github@gmail.com>
Sat, 16 Aug 2025 15:14:42 +0000 (00:14 +0900)
As it is always equals to #expr.

src/basic/assert-util.h
src/fundamental/assert-fundamental.h
src/libsystemd/sd-bus/bus-internal.h

index f6bb7702ea69a13ba4c5ccefc9d3ac62316c7db1..beb52ba88494f1f7c32752c9e2fe6bb265c3df28 100644 (file)
@@ -10,19 +10,20 @@ bool log_get_assert_return_is_critical(void) _pure_;
 
 void log_assert_failed_return(const char *text, const char *file, int line, const char *func);
 
-#define assert_log(expr, message) ((_likely_(expr))                     \
-        ? (true)                                                        \
-        : (log_assert_failed_return(message, PROJECT_FILE, __LINE__, __func__), false))
+#define assert_log(expr)                                                \
+        (_likely_(expr) ?                                               \
+         true :                                                         \
+         (log_assert_failed_return(#expr, PROJECT_FILE, __LINE__, __func__), false))
 
 #define assert_return(expr, r)                                          \
         do {                                                            \
-                if (!assert_log(expr, #expr))                           \
+                if (!assert_log(expr))                                  \
                         return (r);                                     \
         } while (false)
 
 #define assert_return_errno(expr, r, err)                               \
         do {                                                            \
-                if (!assert_log(expr, #expr)) {                         \
+                if (!assert_log(expr)) {                                \
                         errno = err;                                    \
                         return (r);                                     \
                 }                                                       \
index 0b5c7f21de93b92fae52b311c0c9d9b72b6153af..3168e5699aa93c01779b035dacadd579d9e9b20e 100644 (file)
@@ -50,7 +50,7 @@ static inline int __coverity_check_and_return__(int condition) {
 
 #define assert_message_se(expr, message) __coverity_check__(!!(expr))
 
-#define assert_log(expr, message) __coverity_check_and_return__(!!(expr))
+#define assert_log(expr) __coverity_check_and_return__(!!(expr))
 
 #else  /* ! __COVERITY__ */
 
index 0a0dd4143b323c4952fbd66e572d600e59e5f2a9..e8145622de84416336472b261dfc2e2927e092d1 100644 (file)
@@ -399,7 +399,7 @@ int bus_maybe_reply_error(sd_bus_message *m, int r, const sd_bus_error *e);
 
 #define bus_assert_return(expr, r, error)                               \
         do {                                                            \
-                if (!assert_log(expr, #expr))                           \
+                if (!assert_log(expr))                                  \
                         return sd_bus_error_set_errno(error, r);        \
         } while (false)