diff options
| author | Paul Eggert | 2025-12-02 16:32:04 -0800 |
|---|---|---|
| committer | Paul Eggert | 2025-12-02 17:46:56 -0800 |
| commit | 3930d12e4dcb2d79d70c3aa8a199aec0340d0b39 (patch) | |
| tree | eee16540f6eab12985246dcfe4952f1189d1c445 /src | |
| parent | d2ac3cfb9036752695c9d5a4b196e282dc785c4d (diff) | |
| download | emacs-3930d12e4dcb2d79d70c3aa8a199aec0340d0b39.tar.gz emacs-3930d12e4dcb2d79d70c3aa8a199aec0340d0b39.zip | |
Pacify gcc -Wdangling-else in dbusbind.c
* src/dbusbind.c (XD_DBUS_VALIDATE_BUS_NAME)
(XD_DBUS_VALIDATE_PATH, XD_DBUS_VALIDATE_INTERFACE)
(XD_DBUS_VALIDATE_MEMBER): Make these expand to single statements
when they are followed by ‘;’. All uses changed.
Diffstat (limited to 'src')
| -rw-r--r-- | src/dbusbind.c | 20 |
1 files changed, 10 insertions, 10 deletions
diff --git a/src/dbusbind.c b/src/dbusbind.c index 5f89122eaf7..979b313bc33 100644 --- a/src/dbusbind.c +++ b/src/dbusbind.c | |||
| @@ -328,34 +328,34 @@ XD_OBJECT_TO_STRING (Lisp_Object object) | |||
| 328 | 328 | ||
| 329 | #if HAVE_DBUS_VALIDATE_BUS_NAME | 329 | #if HAVE_DBUS_VALIDATE_BUS_NAME |
| 330 | #define XD_DBUS_VALIDATE_BUS_NAME(bus_name) \ | 330 | #define XD_DBUS_VALIDATE_BUS_NAME(bus_name) \ |
| 331 | XD_DBUS_VALIDATE_OBJECT(bus_name, dbus_validate_bus_name); | 331 | XD_DBUS_VALIDATE_OBJECT (bus_name, dbus_validate_bus_name) |
| 332 | #else | 332 | #else |
| 333 | #define XD_DBUS_VALIDATE_BUS_NAME(bus_name) \ | 333 | #define XD_DBUS_VALIDATE_BUS_NAME(bus_name) \ |
| 334 | if (!NILP (bus_name)) CHECK_STRING (bus_name); | 334 | do { if (!NILP (bus_name)) CHECK_STRING (bus_name); } while (false) |
| 335 | #endif | 335 | #endif |
| 336 | 336 | ||
| 337 | #if HAVE_DBUS_VALIDATE_PATH | 337 | #if HAVE_DBUS_VALIDATE_PATH |
| 338 | #define XD_DBUS_VALIDATE_PATH(path) \ | 338 | #define XD_DBUS_VALIDATE_PATH(path) \ |
| 339 | XD_DBUS_VALIDATE_OBJECT(path, dbus_validate_path); | 339 | XD_DBUS_VALIDATE_OBJECT (path, dbus_validate_path) |
| 340 | #else | 340 | #else |
| 341 | #define XD_DBUS_VALIDATE_PATH(path) \ | 341 | #define XD_DBUS_VALIDATE_PATH(path) \ |
| 342 | if (!NILP (path)) CHECK_STRING (path); | 342 | do { if (!NILP (path)) CHECK_STRING (path); } while (false) |
| 343 | #endif | 343 | #endif |
| 344 | 344 | ||
| 345 | #if HAVE_DBUS_VALIDATE_INTERFACE | 345 | #if HAVE_DBUS_VALIDATE_INTERFACE |
| 346 | #define XD_DBUS_VALIDATE_INTERFACE(interface) \ | 346 | #define XD_DBUS_VALIDATE_INTERFACE(interface) \ |
| 347 | XD_DBUS_VALIDATE_OBJECT(interface, dbus_validate_interface); | 347 | XD_DBUS_VALIDATE_OBJECT (interface, dbus_validate_interface) |
| 348 | #else | 348 | #else |
| 349 | #define XD_DBUS_VALIDATE_INTERFACE(interface) \ | 349 | #define XD_DBUS_VALIDATE_INTERFACE(interface) \ |
| 350 | if (!NILP (interface)) CHECK_STRING (interface); | 350 | do { if (!NILP (interface)) CHECK_STRING (interface); } while (false) |
| 351 | #endif | 351 | #endif |
| 352 | 352 | ||
| 353 | #if HAVE_DBUS_VALIDATE_MEMBER | 353 | #if HAVE_DBUS_VALIDATE_MEMBER |
| 354 | #define XD_DBUS_VALIDATE_MEMBER(member) \ | 354 | #define XD_DBUS_VALIDATE_MEMBER(member) \ |
| 355 | XD_DBUS_VALIDATE_OBJECT(member, dbus_validate_member); | 355 | XD_DBUS_VALIDATE_OBJECT (member, dbus_validate_member) |
| 356 | #else | 356 | #else |
| 357 | #define XD_DBUS_VALIDATE_MEMBER(member) \ | 357 | #define XD_DBUS_VALIDATE_MEMBER(member) \ |
| 358 | if (!NILP (member)) CHECK_STRING (member); | 358 | do { if (!NILP (member)) CHECK_STRING (member); } while (false) |
| 359 | #endif | 359 | #endif |
| 360 | 360 | ||
| 361 | /* Append to SIGNATURE a copy of X, making sure SIGNATURE does | 361 | /* Append to SIGNATURE a copy of X, making sure SIGNATURE does |
| @@ -425,7 +425,7 @@ xd_signature (char *signature, int dtype, int parent_type, Lisp_Object object) | |||
| 425 | /* We don't check the syntax of signature. This will be done by | 425 | /* We don't check the syntax of signature. This will be done by |
| 426 | libdbus. */ | 426 | libdbus. */ |
| 427 | if (dtype == DBUS_TYPE_OBJECT_PATH) | 427 | if (dtype == DBUS_TYPE_OBJECT_PATH) |
| 428 | XD_DBUS_VALIDATE_PATH (object) | 428 | XD_DBUS_VALIDATE_PATH (object); |
| 429 | else | 429 | else |
| 430 | CHECK_STRING (object); | 430 | CHECK_STRING (object); |
| 431 | sprintf (signature, "%c", dtype); | 431 | sprintf (signature, "%c", dtype); |
| @@ -749,7 +749,7 @@ xd_append_arg (int dtype, Lisp_Object object, DBusMessageIter *iter) | |||
| 749 | /* We don't check the syntax of signature. This will be done | 749 | /* We don't check the syntax of signature. This will be done |
| 750 | by libdbus. */ | 750 | by libdbus. */ |
| 751 | if (dtype == DBUS_TYPE_OBJECT_PATH) | 751 | if (dtype == DBUS_TYPE_OBJECT_PATH) |
| 752 | XD_DBUS_VALIDATE_PATH (object) | 752 | XD_DBUS_VALIDATE_PATH (object); |
| 753 | else | 753 | else |
| 754 | CHECK_STRING (object); | 754 | CHECK_STRING (object); |
| 755 | { | 755 | { |