From 41c4aa408c3488e920e1f08eb946442faa051e6c Mon Sep 17 00:00:00 2001
From: Alan Coopersmith <alan.coopersmith@oracle.com>
Date: Sun, 10 Nov 2024 11:52:19 -0800
Subject: [PATCH] OneDataCard32: add do ... while (0) to avoid
 -Wextra-semi-stmt warnings

Clears 13 warnings from clang of the form:

Backgnd.c:44:72: warning: empty expression statement has no effect;
 remove unnecessary ';' to silence this warning [-Wextra-semi-stmt]
    OneDataCard32 (dpy, NEXTPTR(req,xChangeWindowAttributesReq), pixel);
                                                                       ^

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Part-of: <https://gitlab.freedesktop.org/xorg/lib/libx11/-/merge_requests/272>
---
 include/X11/Xlibint.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/X11/Xlibint.h b/include/X11/Xlibint.h
index 9f27b229..2768b4cd 100644
--- a/include/X11/Xlibint.h
+++ b/include/X11/Xlibint.h
@@ -765,7 +765,7 @@ extern void _XRead32(
 
 /* srcvar must be a variable for large architecture version */
 #define OneDataCard32(dpy,dstaddr,srcvar) \
-  { *(CARD32 *)(dstaddr) = (srcvar); }
+    do { *(CARD32 *)(dstaddr) = (srcvar); } while (0)
 
 
 typedef struct _XInternalAsync {