maint.mk: match 0 or more whitespace-before-function-call '('

* top/maint.mk (sc_error_exit_success): Relax regexp to match uses
that have zero or two-and-more spaces between the function name
and the open parenthesis.
(sc_error_message_warn_fatal): Likewise.
(sc_error_message_uppercase): Likewise.
(sc_error_message_period): Likewise.

diff --git a/ChangeLog b/ChangeLog
index 82263a7c9de4b069caa64ada5f3d9d51bb505bb9..f1410545de3d98e2ddcbe93339882598585616c8 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -16,6 +16,14 @@
 
 2010-03-28  Jose E. Marchesi  <jemarch@gnu.org>
 
+       maint.mk: match 0 or more whitespace-before-function-call '('
+       * top/maint.mk (sc_error_exit_success): Relax regexp to match uses
+       that have zero or two-and-more spaces between the function name
+       and the open parenthesis.
+       (sc_error_message_warn_fatal): Likewise.
+       (sc_error_message_uppercase): Likewise.
+       (sc_error_message_period): Likewise.
+
        maint.mk: s/_header_without_use/_sc_header_without_use/
        * top/maint.mk (_sc_header_without_use): Rename from _header_without_use.
        (sc_prohibit_assert_without_use): Use the new name.

diff --git a/top/maint.mk b/top/maint.mk
index 8253e8bc5cb807a0c6e69cf85ab955963cc4e42e..389019fca2322ea060d3260ee1f317f673b346d0 100644 (file)
--- a/top/maint.mk
+++ b/top/maint.mk
@@ -217,21 +217,21 @@ sc_prohibit_magic_number_exit:
 # Using EXIT_SUCCESS as the first argument to error is misleading,
 # since when that parameter is 0, error does not exit.  Use `0' instead.
 sc_error_exit_success:
-       @grep -nE 'error \(EXIT_SUCCESS,'                               \
+       @grep -nE 'error *\(EXIT_SUCCESS,'                              \
            $$($(VC_LIST_EXCEPT) | grep -E '\.[chly]$$') &&             \
          { echo '$(ME): found error (EXIT_SUCCESS' 1>&2; exit 1; } || :
 
 # `FATAL:' should be fully upper-cased in error messages
 # `WARNING:' should be fully upper-cased, or fully lower-cased
 sc_error_message_warn_fatal:
-       @grep -nEA2 '[^rp]error \(' $$($(VC_LIST_EXCEPT))               \
+       @grep -nEA2 '[^rp]error *\(' $$($(VC_LIST_EXCEPT))              \
            | grep -E '"Warning|"Fatal|"fatal' &&                       \
          { echo '$(ME): use FATAL, WARNING or warning' 1>&2;           \
            exit 1; } || :
 
 # Error messages should not start with a capital letter
 sc_error_message_uppercase:
-       @grep -nEA2 '[^rp]error \(' $$($(VC_LIST_EXCEPT))               \
+       @grep -nEA2 '[^rp]error *\(' $$($(VC_LIST_EXCEPT))              \
            | grep -E '"[A-Z]'                                          \
            | grep -vE '"FATAL|"WARNING|"Java|"C#|PRIuMAX' &&           \
          { echo '$(ME): found capitalized error message' 1>&2;         \
@@ -239,7 +239,7 @@ sc_error_message_uppercase:
 
 # Error messages should not end with a period
 sc_error_message_period:
-       @grep -nEA2 '[^rp]error \(' $$($(VC_LIST_EXCEPT))               \
+       @grep -nEA2 '[^rp]error *\(' $$($(VC_LIST_EXCEPT))              \
            | grep -E '[^."]\."' &&                                     \
          { echo '$(ME): found error message ending in period' 1>&2;    \
            exit 1; } || :