From 8d5fa64778161a6bbccb8a80478849467003cabd Mon Sep 17 00:00:00 2001
From: Jim Meyering <meyering@fb.com>
Date: Sun, 5 Oct 2014 17:32:13 -0700
Subject: maint: enable sc_m4_quote_check syntax check

* cfg.mk (local-checks-to-skip): Remove its exemption.
* configure.ac: Fix the under-quoting problems thus exposed.
---
 cfg.mk       | 1 -
 configure.ac | 6 +++---
 2 files changed, 3 insertions(+), 4 deletions(-)

diff --git a/cfg.mk b/cfg.mk
index 7e2cac7..61a5ce5 100644
--- a/cfg.mk
+++ b/cfg.mk
@@ -28,7 +28,6 @@ local-checks-to-skip =			\
   sc_error_message_uppercase		\
   sc_file_system			\
   sc_long_lines				\
-  sc_m4_quote_check			\
   sc_prohibit_atoi_atof			\
   sc_prohibit_empty_lines_at_EOF	\
   sc_prohibit_magic_number_exit		\
diff --git a/configure.ac b/configure.ac
index 1801b73..58d6f76 100644
--- a/configure.ac
+++ b/configure.ac
@@ -11,7 +11,7 @@ AM_SILENT_RULES([yes]) # make --enable-silent-rules the default.
 AC_CONFIG_MACRO_DIRS([m4])
 
 SED_FEATURE_VERSION=4.2.2
-AC_DEFINE_UNQUOTED(SED_FEATURE_VERSION, "$SED_FEATURE_VERSION",
+AC_DEFINE_UNQUOTED([SED_FEATURE_VERSION], "$SED_FEATURE_VERSION",
   [Define to the version of GNU sed whose features are supported by this sed.])
 AC_SUBST(SED_FEATURE_VERSION)
 
@@ -82,7 +82,7 @@ AC_CHECK_FUNCS_ONCE(isatty isascii memcpy strchr strtoul lstat readlink
 
 # Check whether we are able to follow symlinks
 if test "$ac_cv_func_lstat:$ac_cv_func_readlink" = yes:yes; then
-   AC_DEFINE(ENABLE_FOLLOW_SYMLINKS, ,[Follow symlinks when processing in place])
+   AC_DEFINE([ENABLE_FOLLOW_SYMLINKS], ,[Follow symlinks when processing in place])
 fi
 
 AC_ARG_ENABLE(i18n,
@@ -101,7 +101,7 @@ enable_regex_tests=$with_included_regex)
 
 AM_CONDITIONAL(TEST_REGEX, test "x$enable_regex_tests" = xyes)
 if test "x$enable_regex_tests" = xyes; then
-  AC_DEFINE_UNQUOTED(_REGEX_RE_COMP, 1,
+  AC_DEFINE_UNQUOTED([_REGEX_RE_COMP], 1,
     [Include BSD functions in regex, used by the testsuite])
 fi
 
-- 
cgit v1.2.1