* doc/autoconf.texi (Coding Style): Some more constraints :).

diff --git a/ChangeLog b/ChangeLog
index 72347024df04e2f6a3de26dbefc643f752793ab9..abf0b0f404ad9dea52811088c21284e549835971 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,7 @@
+2000-07-10  Akim Demaille  <akim@epita.fr>
+
+       * doc/autoconf.texi (Coding Style): Some more constraints :).
+
 2000-07-10  Akim Demaille  <akim@epita.fr>
 
        * acspecific.m4 (AC_FUNC_STRERROR_R): Aaaaaaarg!  Don't leave the

diff --git a/acgeneral.m4 b/acgeneral.m4
index bf6144968e60bc8c85fa25846f1219c14585fdde..ef1def44214f6c988b85cdfd1a4386ddb3e288b7 100644 (file)
--- a/acgeneral.m4
+++ b/acgeneral.m4
@@ -2192,7 +2192,7 @@ AC_DEFUN_ONCE([AC_CANONICAL_BUILD],
 [AC_REQUIRE([AC_CONFIG_AUX_DIR_DEFAULT])dnl
 AC_DIVERT([HELP_CANON],
 [[
-Hosts type:
+System types:
   --build=BUILD     configure for building on BUILD [guessed]]])dnl
 # Make sure we can run config.sub.
 $ac_config_sub sun4 >/dev/null 2>&1 ||

diff --git a/doc/autoconf.texi b/doc/autoconf.texi
index 97f641c4b26f59b104ec8daaafb7255359bbeffe..e08c7e2e0386b832710f1a4684967016e4b37765 100644 (file)
--- a/doc/autoconf.texi
+++ b/doc/autoconf.texi
@@ -6697,6 +6697,12 @@ fi[]dnl
 ])# AC_PATH_X
 @end example
 
+If the macro is long, try to split it into logical chunks.  Typically,
+macros which check for a bug in a function and prepare its
+@code{AC_LIBOBJ} replacement should have an auxiliary macro to perform
+this setup.
+
+Do not hesitate to introduce auxiliary macros to factor your code.
 
 In order to highlight this coding style, here is a macro written the old
 way:

diff --git a/lib/autoconf/general.m4 b/lib/autoconf/general.m4
index bf6144968e60bc8c85fa25846f1219c14585fdde..ef1def44214f6c988b85cdfd1a4386ddb3e288b7 100644 (file)
--- a/lib/autoconf/general.m4
+++ b/lib/autoconf/general.m4
@@ -2192,7 +2192,7 @@ AC_DEFUN_ONCE([AC_CANONICAL_BUILD],
 [AC_REQUIRE([AC_CONFIG_AUX_DIR_DEFAULT])dnl
 AC_DIVERT([HELP_CANON],
 [[
-Hosts type:
+System types:
   --build=BUILD     configure for building on BUILD [guessed]]])dnl
 # Make sure we can run config.sub.
 $ac_config_sub sun4 >/dev/null 2>&1 ||