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

Clears 5 warnings from clang of the form:

RdBitF.c:141:32: warning: empty expression statement has no effect;
 remove unnecessary ';' to silence this warning [-Wextra-semi-stmt]
            RETURN (BitmapFileInvalid);
                                      ^

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

diff --git a/src/RdBitF.c b/src/RdBitF.c
index e5a8592e..ce370806 100644
--- a/src/RdBitF.c
+++ b/src/RdBitF.c
@@ -134,7 +134,7 @@ XReadBitmapFileData (
 
     /* error cleanup and return macro	*/
 #define	RETURN(code) \
-{ Xfree (bits); fclose (fstream); return code; }
+    do { Xfree (bits); fclose (fstream); return code; } while (0)
 
     while (fgets(line, MAX_SIZE, fstream)) {
 	if (strlen(line) == MAX_SIZE-1)