From: Bruno Haible <bruno@clisp.org>
Date: Tue, 16 Aug 2005 16:07:59 +0000 (+0000)
Subject: Bug fix: make the comparison operations transitive.
X-Git-Tag: cvs-readonly~3136
X-Git-Url: http://erislabs.net/gitweb/?a=commitdiff_plain;h=ece57f1eec22076c27fcc9f524cde21a10aa2a3a;p=gnulib.git

Bug fix: make the comparison operations transitive.
---

diff --git a/lib/mbchar.h b/lib/mbchar.h
index 6bb531e79..4472aa020 100644
--- a/lib/mbchar.h
+++ b/lib/mbchar.h
@@ -182,21 +182,29 @@ typedef struct mbchar mbchar_t;
 #define mb_iseq(mbc, sc) ((mbc).wc_valid && (mbc).wc == (sc))
 #define mb_isnul(mbc) ((mbc).wc_valid && (mbc).wc == 0)
 #define mb_cmp(mbc1, mbc2) \
-  ((mbc1).wc_valid && (mbc2).wc_valid					\
-   ? (int) (mbc1).wc - (int) (mbc2).wc					\
-   : (mbc1).bytes == (mbc2).bytes					\
-     ? memcmp ((mbc1).ptr, (mbc2).ptr, (mbc1).bytes)			\
-     : (mbc1).bytes < (mbc2).bytes					\
-       ? (memcmp ((mbc1).ptr, (mbc2).ptr, (mbc1).bytes) > 0 ? 1 : -1)	\
-       : (memcmp ((mbc1).ptr, (mbc2).ptr, (mbc2).bytes) >= 0 ? 1 : -1))
+  ((mbc1).wc_valid							\
+   ? ((mbc2).wc_valid							\
+      ? (int) (mbc1).wc - (int) (mbc2).wc				\
+      : -1)								\
+   : ((mbc2).wc_valid							\
+      ? 1								\
+      : (mbc1).bytes == (mbc2).bytes					\
+        ? memcmp ((mbc1).ptr, (mbc2).ptr, (mbc1).bytes)			\
+        : (mbc1).bytes < (mbc2).bytes					\
+          ? (memcmp ((mbc1).ptr, (mbc2).ptr, (mbc1).bytes) > 0 ? 1 : -1) \
+          : (memcmp ((mbc1).ptr, (mbc2).ptr, (mbc2).bytes) >= 0 ? 1 : -1)))
 #define mb_casecmp(mbc1, mbc2) \
-  ((mbc1).wc_valid && (mbc2).wc_valid					\
-   ? (int) towlower ((mbc1).wc) - (int) towlower ((mbc2).wc)		\
-   : (mbc1).bytes == (mbc2).bytes					\
-     ? memcmp ((mbc1).ptr, (mbc2).ptr, (mbc1).bytes)			\
-     : (mbc1).bytes < (mbc2).bytes					\
-       ? (memcmp ((mbc1).ptr, (mbc2).ptr, (mbc1).bytes) > 0 ? 1 : -1)	\
-       : (memcmp ((mbc1).ptr, (mbc2).ptr, (mbc2).bytes) >= 0 ? 1 : -1))
+  ((mbc1).wc_valid							\
+   ? ((mbc2).wc_valid							\
+      ? (int) towlower ((mbc1).wc) - (int) towlower ((mbc2).wc)		\
+      : -1)								\
+   : ((mbc2).wc_valid							\
+      ? 1								\
+      : (mbc1).bytes == (mbc2).bytes					\
+        ? memcmp ((mbc1).ptr, (mbc2).ptr, (mbc1).bytes)			\
+        : (mbc1).bytes < (mbc2).bytes					\
+          ? (memcmp ((mbc1).ptr, (mbc2).ptr, (mbc1).bytes) > 0 ? 1 : -1) \
+          : (memcmp ((mbc1).ptr, (mbc2).ptr, (mbc2).bytes) >= 0 ? 1 : -1)))
 #define mb_equal(mbc1, mbc2) \
   ((mbc1).wc_valid && (mbc2).wc_valid					\
    ? (mbc1).wc == (mbc2).wc						\