automake-ng
[Top][All Lists]
Advanced

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

[Automake-NG] [FYI] [ng] cosmetics: move a variable definition


From: Stefano Lattarini
Subject: [Automake-NG] [FYI] [ng] cosmetics: move a variable definition
Date: Tue, 24 Jul 2012 16:36:51 +0200

* lib/am/header-vars.am (am.chars.empty): This, move earlier.  This both
fixes the associated comments (which referred to a variable defined above
it as "defined below"), and ensures that this variable is not used before
being defined (not a big deal in practice, since it expands to empty; but
better be consistent).

Reported by Akim Demaille.

Signed-off-by: Stefano Lattarini <address@hidden>
---
 lib/am/header-vars.am | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/lib/am/header-vars.am b/lib/am/header-vars.am
index 16a0e40..251d152 100644
--- a/lib/am/header-vars.am
+++ b/lib/am/header-vars.am
@@ -50,6 +50,11 @@ am.chars.dquote := "
 am.chars.squote := '
 # "` # Fix font-lock.
 
+# An empty string.  It can be very useful to "fool" the make parser w.r.t.
+# whitespace handling, and allow us to obtain tricky semantics.  See the
+# definition of $(am.chars.newline) just below for a significant example.
+am.chars.empty :=
+
 # A literal newline character, that does not get stripped if used
 # at the end of the expansion of another macro.
 define am.chars.newline
@@ -143,11 +148,6 @@ am.make.dry-run := \
         $(findstring n,$v)))), \
     true, false))
 
-# An empty string.  It can be very useful to "fool" the make parser w.r.t.
-# whitespace handling, and allow us to obtain tricky semantics.  See the
-# definition of $(am.chars.newline) below for a significant example.
-am.chars.empty :=
-
 am.util.strip-first-word = $(wordlist 2,$(words $(1)),$(1))
 am.util.strip-last-word  = $(wordlist 2,$(words $(1)),dummy $(1))
 
-- 
1.7.12.rc0




reply via email to

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