Re: [PATCH] quote: fuse into quotearg

[Akim Demaille]

Hi Bruno,

Le 7 mars 2012 à 11:40, Bruno Haible a écrit :

> 2012-03-07  Bruno Haible  <address@hidden>
> 
>       quote: Adhere to common module description layout.
>       * modules/quote (Makefile.am): Add back empty section.
> 
> --- modules/quote.orig        Wed Mar  7 11:36:14 2012
> +++ modules/quote     Wed Mar  7 11:33:17 2012
> @@ -11,6 +11,8 @@
> configure.ac:
> gl_QUOTE
> 
> +Makefile.am:
> +
> Include:
> "quote.h"

Thanks for fixing the style issue.  There is another issue
I overlooked in that patch, fixed below.

From a458ab72cd4838983c17955972c192af980fe419 Mon Sep 17 00:00:00 2001
From: Akim Demaille <address@hidden>
Date: Fri, 9 Mar 2012 08:52:12 +0100
Subject: [PATCH] quote: fix syntax-check.

* top/maint.mk (sc_prohibit_quote_without_use): quote.h
also exports quote_quoting_options.
---
 ChangeLog    |    6 ++++++
 top/maint.mk |    3 ++-
 2 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/ChangeLog b/ChangeLog
index 63b35f3..d85fe1d 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2012-03-09  Akim Demaille  <address@hidden>
+
+       quote: fix syntax-check.
+       * top/maint.mk (sc_prohibit_quote_without_use): quote.h
+       also exports quote_quoting_options.
+
 2012-03-08  Paul Eggert  <address@hidden>
 
        modechange: add notations +40, 00440, etc.
diff --git a/top/maint.mk b/top/maint.mk
index a97e0bd..951b358 100644
--- a/top/maint.mk
+++ b/top/maint.mk
@@ -455,7 +455,8 @@ sc_prohibit_quotearg_without_use:
 
 # Don't include quote.h unless you use one of its functions.
 sc_prohibit_quote_without_use:
-       @h='quote.h' re='\<quote(_n)? *\(' $(_sc_header_without_use)
+       @h='quote.h' re='\<quote((_n)? *\(|_quoting_options\>)' \
+         $(_sc_header_without_use)
 
 # Don't include this header unless you use one of its functions.
 sc_prohibit_long_options_without_use:
-- 
1.7.9.2