groff-commit
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[groff] 03/07: m4/groff.m4: Fix code style nits.


From: G. Branden Robinson
Subject: [groff] 03/07: m4/groff.m4: Fix code style nits.
Date: Thu, 16 Feb 2023 01:00:00 -0500 (EST)

gbranden pushed a commit to branch master
in repository groff.

commit 37f7bf4fd2dbdbf6ff5dd1b76605d41082ffbba6
Author: G. Branden Robinson <g.branden.robinson@gmail.com>
AuthorDate: Wed Feb 15 13:26:25 2023 -0600

    m4/groff.m4: Fix code style nits.
    
    ...for maintainability and comprehensibility.
    
    * m4/groff.m4 (GROFF_GHOSTSCRIPT_VERSION_CHECK, GROFF_URW_FONTS_NOTICE):
      Add macro dependency on `GROFF_GHOSTSCRIPT_PATH`, which should have
      been there already.
    
      (GROFF_GROHTML_PROGRAM_NOTICE): Add macro dependency on
      `GROFF_CHECK_GROHTML_PROGRAMS`, which should have been there already.
    
      (GROFF_CHECK_GROHTML_PROGRAMS, GROFF_CHECK_GROPDF_PROGRAMS,
      GROFF_PNMTOOLS_CAN_BE_QUIET): Move `AC_REQUIRE` expansions to precede
      shell variable assignments (after checking that the required macros
      don't clobber the assignments we're making).
    
    Add dnl annotations to assist maintainers.
    
    Also stop uselessly quoting some literals.
---
 ChangeLog   | 14 ++++++++++++++
 m4/groff.m4 | 30 +++++++++++++++++++++++-------
 2 files changed, 37 insertions(+), 7 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index f121f59af..c6e7704b7 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,17 @@
+2023-02-15  G. Branden Robinson <g.branden.robinson@gmail.com>
+
+       * m4/groff.m4 (GROFF_GHOSTSCRIPT_VERSION_CHECK,
+       GROFF_URW_FONTS_NOTICE): Add macro dependency on
+       `GROFF_GHOSTSCRIPT_PATH`, which should have been there already.
+       (GROFF_GROHTML_PROGRAM_NOTICE): Add macro dependency on
+       `GROFF_CHECK_GROHTML_PROGRAMS`, which should have been there
+       already.
+
+       (GROFF_CHECK_GROHTML_PROGRAMS, GROFF_PNMTOOLS_CAN_BE_QUIET):
+       Move `AC_REQUIRE` expansions to precede shell variable
+       assignments (after checking that the required macros don't
+       clobber the assignments we're making).
+
 2023-02-15  G. Branden Robinson <g.branden.robinson@gmail.com>
 
        * font/devpdf/tests/basic-fonts-present.sh: Revise test again,
diff --git a/m4/groff.m4 b/m4/groff.m4
index fa1ec5a22..3a5627f20 100644
--- a/m4/groff.m4
+++ b/m4/groff.m4
@@ -174,9 +174,12 @@ AC_DEFUN([GROFF_USE_TEX_CHECK], [
 # grohtml needs the following programs to produce images from tbl(1)
 # tables and eqn(1) equations.
 
+dnl Any macro that tests $make_htmldoc should AC_REQUIRE this.
+
 AC_DEFUN([GROFF_CHECK_GROHTML_PROGRAMS], [
-  make_htmldoc=no
   AC_REQUIRE([GROFF_GHOSTSCRIPT_PATH])
+
+  make_htmldoc=no
   missing=
   m4_foreach([groff_prog],
     [[pnmcrop], [pnmcut], [pnmtopng], [pnmtops], [psselect]], [
@@ -222,12 +225,14 @@ AC_DEFUN([GROFF_CHECK_GROHTML_PROGRAMS], [
   properly.  It will not be possible to prepare or install
   groff-generated documentation in HTML format.
 "
-
    fi
    AC_SUBST([make_htmldoc])
 ])
 
+
 AC_DEFUN([GROFF_GROHTML_PROGRAM_NOTICE], [
+  AC_REQUIRE([GROFF_CHECK_GROHTML_PROGRAMS])
+
   if test "$make_htmldoc" = no
   then
     AC_MSG_NOTICE([$grohtml_notice])
@@ -238,10 +243,10 @@ AC_DEFUN([GROFF_GROHTML_PROGRAM_NOTICE], [
 # font description files.
 
 AC_DEFUN([GROFF_CHECK_GROPDF_PROGRAMS], [
-  use_gropdf=no
   AC_REQUIRE([GROFF_AWK_PATH])
   AC_REQUIRE([GROFF_GHOSTSCRIPT_PATH])
 
+  use_gropdf=no
   missing=
   test "$AWK" = missing && missing="'awk'"
   test "$GHOSTSCRIPT" = missing && missing="$missing 'gs'"
@@ -347,6 +352,8 @@ AC_DEFUN([GROFF_URW_FONTS_CHECK], [
 ])
 
 AC_DEFUN([GROFF_URW_FONTS_NOTICE], [
+  AC_REQUIRE([GROFF_GHOSTSCRIPT_PATH])
+
   if test "$GHOSTSCRIPT" != missing && test "$groff_have_urw_fonts" = no
   then
     AC_MSG_NOTICE([URW fonts in Type 1/PFB format were not found.
@@ -384,9 +391,13 @@ AC_DEFUN([GROFF_URW_FONTS_NOTICE], [
 
 # Check whether the pnm tools accept the -quiet option.
 
+dnl Any macro that tests $pnmtools_quiet should AC_REQUIRE this.
+
 AC_DEFUN([GROFF_PNMTOOLS_CAN_BE_QUIET], [
-  pnmtools_quiet=
   AC_REQUIRE([GROFF_CHECK_GROHTML_PROGRAMS])
+
+  pnmtools_quiet=
+
   if test "$make_htmldoc" = yes
   then
     AC_MSG_CHECKING([whether PNM tools accept the '-quiet' option])
@@ -408,8 +419,9 @@ AC_DEFUN([GROFF_PNMTOOLS_CAN_BE_QUIET], [
 # doc/gnu.eps from repository builds.
 
 AC_DEFUN([GROFF_PNMTOPS_NOSETPAGE], [
-  pnmtops_nosetpage="pnmtops $pnmtools_quiet"
   AC_REQUIRE([GROFF_PNMTOOLS_CAN_BE_QUIET])
+
+  pnmtops_nosetpage="pnmtops $pnmtools_quiet"
   AC_MSG_CHECKING([whether pnmtops accepts the '-nosetpage' option])
   if echo P2 2 2 255 0 1 2 0 | pnmtops -nosetpage > /dev/null 2>&1
   then
@@ -424,6 +436,8 @@ AC_DEFUN([GROFF_PNMTOPS_NOSETPAGE], [
 
 # Check location of 'gs'; allow '--with-gs=PROG' option to override.
 
+dnl Any macro that tests $GHOSTSCRIPT should AC_REQUIRE this.
+
 AC_DEFUN([GROFF_GHOSTSCRIPT_PATH],
   [AC_REQUIRE([GROFF_GHOSTSCRIPT_PREFS])
    AC_ARG_WITH([gs],
@@ -431,7 +445,7 @@ AC_DEFUN([GROFF_GHOSTSCRIPT_PATH],
        [actual [/path/]name of ghostscript executable])],
      [GHOSTSCRIPT=$withval],
      [AC_CHECK_TOOLS(GHOSTSCRIPT, [$ALT_GHOSTSCRIPT_PROGS], [missing])])
-   test "$GHOSTSCRIPT" = "no" && GHOSTSCRIPT=missing])
+   test "$GHOSTSCRIPT" = no && GHOSTSCRIPT=missing])
 
 # Preferences for choice of 'gs' program...
 # (allow --with-alt-gs="LIST" to override).
@@ -450,6 +464,8 @@ AC_DEFUN([GROFF_GHOSTSCRIPT_PREFS],
 #   <https://bugs.ghostscript.com/show_bug.cgi?id=703187>.
 
 AC_DEFUN([GROFF_GHOSTSCRIPT_VERSION_CHECK], [
+  AC_REQUIRE([GROFF_GHOSTSCRIPT_PATH])
+
   if test "$GHOSTSCRIPT" != missing
   then
     AC_MSG_CHECKING([for gs version with good left sidebearing handling])
@@ -529,7 +545,7 @@ AC_DEFUN([GROFF_AWK_PATH],
        [actual [/path/]name of awk executable])],
      [AWK=$withval],
      [AC_CHECK_TOOLS(AWK, [$ALT_AWK_PROGS], [missing])])
-   test "$AWK" = "no" && AWK=missing])
+   test "$AWK" = no && AWK=missing])
 
 
 # Preferences for choice of 'awk' program; allow --with-alt-awk="LIST"



reply via email to

[Prev in Thread] Current Thread [Next in Thread]