From: Bruno Haible <bruno@clisp.org>
Date: Wed, 19 May 2010 21:30:18 +0000 (+0200)
Subject: Clean up dead code in recent commit.
X-Git-Tag: v0.1~4130
X-Git-Url: http://erislabs.net/gitweb/?a=commitdiff_plain;h=75dc5a3dbe3c65f57adc9533561100079922e180;hp=1fcaaedf40a31c2ea6c961ab4e720a5bbd841d37;p=gnulib.git

Clean up dead code in recent commit.
---

diff --git a/ChangeLog b/ChangeLog
index 5bc2c09a5..8fcce2671 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,13 @@
 2010-05-19  Bruno Haible  <bruno@clisp.org>
 
+	Clean up dead code in recent commit.
+	* m4/libunistring-base.m4 (gl_LIBUNISTRING_VERSION_CMP): Include the
+	body of gl_LIBUNISTRING_VERSION_CMP_ORIG as fallback.
+	(gl_LIBUNISTRING_VERSION_CMP_ORIG): Remove macro.
+	Suggested by Paolo Bonzini.
+
+2010-05-19  Bruno Haible  <bruno@clisp.org>
+
 	Avoid valgrind error reports from libunistring.
 	* lib/libunistring.valgrind: New file, based on lib/malloca.valgrind.
 	* modules/libunistring (Files): Add it.
diff --git a/m4/libunistring-base.m4 b/m4/libunistring-base.m4
index 355fe2f30..54aa521c7 100644
--- a/m4/libunistring-base.m4
+++ b/m4/libunistring-base.m4
@@ -1,4 +1,4 @@
-# libunistring-base.m4 serial 1
+# libunistring-base.m4 serial 2
 dnl Copyright (C) 2010 Free Software Foundation, Inc.
 dnl This file is free software; the Free Software Foundation
 dnl gives unlimited permission to copy and/or distribute it,
@@ -97,45 +97,43 @@ changequote([,])
 dnl gl_LIBUNISTRING_VERSION_CMP([VERSION])
 dnl Expands to a shell statement that evaluates to true if LIBUNISTRING_VERSION
 dnl is less than the VERSION argument.
-dnl This is the unoptimized variant:
-AC_DEFUN([gl_LIBUNISTRING_VERSION_CMP_ORIG],
-[ { test "$HAVE_LIBUNISTRING" != yes \
-    || {
-         requested_version_major=`echo '$1' | sed -n -e "$gl_libunistring_sed_extract_major"`
-         requested_version_minor=`echo '$1' | sed -n -e "$gl_libunistring_sed_extract_minor"`
-         requested_version_subminor=`echo '$1' | sed -n -e "$gl_libunistring_sed_extract_subminor"`
-         test $LIBUNISTRING_VERSION_MAJOR -lt $requested_version_major \
-         || { test $LIBUNISTRING_VERSION_MAJOR -eq $requested_version_major \
-              && { test $LIBUNISTRING_VERSION_MINOR -lt $requested_version_minor \
-                   || { test $LIBUNISTRING_VERSION_MINOR -eq $requested_version_minor \
-                        && test $LIBUNISTRING_VERSION_SUBMINOR -lt $requested_version_subminor
-                      }
-                 }
-            }
-       }
-  }
-])
-dnl This is the optimized variant, that assumes the argument is a literal:
 AC_DEFUN([gl_LIBUNISTRING_VERSION_CMP],
 [ { test "$HAVE_LIBUNISTRING" != yes \
     || {
-         m4_pushdef([requested_version_major],
-           [gl_LIBUNISTRING_ARG_OR_ZERO(m4_bpatsubst([$1], [^\([0-9]*\).*], [\1]), [])])
-         m4_pushdef([requested_version_minor],
-           [gl_LIBUNISTRING_ARG_OR_ZERO(m4_bpatsubst([$1], [^[0-9]*[.]\([0-9]*\).*], [\1]), [$1])])
-         m4_pushdef([requested_version_subminor],
-           [gl_LIBUNISTRING_ARG_OR_ZERO(m4_bpatsubst([$1], [^[0-9]*[.][0-9]*[.]\([0-9]*\).*], [\1]), [$1])])
-         test $LIBUNISTRING_VERSION_MAJOR -lt requested_version_major \
-         || { test $LIBUNISTRING_VERSION_MAJOR -eq requested_version_major \
-              && { test $LIBUNISTRING_VERSION_MINOR -lt requested_version_minor \
-                   || { test $LIBUNISTRING_VERSION_MINOR -eq requested_version_minor \
-                        && test $LIBUNISTRING_VERSION_SUBMINOR -lt requested_version_subminor
-                      }
-                 }
-            }
-         m4_popdef([requested_version_subminor])
-         m4_popdef([requested_version_minor])
-         m4_popdef([requested_version_major])
+         dnl AS_LITERAL_IF exists and works fine since autoconf-2.59 at least.
+         AS_LITERAL_IF([$1],
+           [dnl This is the optimized variant, that assumes the argument is a literal:
+            m4_pushdef([requested_version_major],
+              [gl_LIBUNISTRING_ARG_OR_ZERO(m4_bpatsubst([$1], [^\([0-9]*\).*], [\1]), [])])
+            m4_pushdef([requested_version_minor],
+              [gl_LIBUNISTRING_ARG_OR_ZERO(m4_bpatsubst([$1], [^[0-9]*[.]\([0-9]*\).*], [\1]), [$1])])
+            m4_pushdef([requested_version_subminor],
+              [gl_LIBUNISTRING_ARG_OR_ZERO(m4_bpatsubst([$1], [^[0-9]*[.][0-9]*[.]\([0-9]*\).*], [\1]), [$1])])
+            test $LIBUNISTRING_VERSION_MAJOR -lt requested_version_major \
+            || { test $LIBUNISTRING_VERSION_MAJOR -eq requested_version_major \
+                 && { test $LIBUNISTRING_VERSION_MINOR -lt requested_version_minor \
+                      || { test $LIBUNISTRING_VERSION_MINOR -eq requested_version_minor \
+                           && test $LIBUNISTRING_VERSION_SUBMINOR -lt requested_version_subminor
+                         }
+                    }
+               }
+            m4_popdef([requested_version_subminor])
+            m4_popdef([requested_version_minor])
+            m4_popdef([requested_version_major])
+           ],
+           [dnl This is the unoptimized variant:
+            requested_version_major=`echo '$1' | sed -n -e "$gl_libunistring_sed_extract_major"`
+            requested_version_minor=`echo '$1' | sed -n -e "$gl_libunistring_sed_extract_minor"`
+            requested_version_subminor=`echo '$1' | sed -n -e "$gl_libunistring_sed_extract_subminor"`
+            test $LIBUNISTRING_VERSION_MAJOR -lt $requested_version_major \
+            || { test $LIBUNISTRING_VERSION_MAJOR -eq $requested_version_major \
+                 && { test $LIBUNISTRING_VERSION_MINOR -lt $requested_version_minor \
+                      || { test $LIBUNISTRING_VERSION_MINOR -eq $requested_version_minor \
+                           && test $LIBUNISTRING_VERSION_SUBMINOR -lt $requested_version_subminor
+                         }
+                    }
+               }
+           ])
        }
   }
 ])