[Bug-gnulib] suggestion

[Jim Meyering]

Hi Bruno,

I noticed you didn't apply parts of the recent stdbool patch.
Maybe you'd like some justification?

It is better to use $@ whenever possible,
to reduce duplication (ease maintenance).
It also shortens a long line, here.

sed's '<' is unnecessary.
Removing it also shortens the line.

And I've been taught that comments are best
written in the imperative.  In particular,
I try to avoid using the passive voice.

As for the `-t' vs. `t' suffix, I don't care much,
as long as we're consistent, but do have a slight preference
for the shorter one.

Jim

Index: stdbool
===================================================================
RCS file: /cvsroot/gnulib/gnulib/modules/stdbool,v
retrieving revision 1.7
diff -u -p -u -p -r1.7 stdbool
--- stdbool     6 Jun 2003 20:59:34 -0000       1.7
+++ stdbool     7 Jun 2003 07:01:00 -0000
@@ -14,17 +14,15 @@ AM_STDBOOL_H
 Makefile.am:
 EXTRA_DIST += stdbool_.h
 
-# The following is needed in order to create an <stdbool.h> when the system
-# doesn't have one that works.
+# Create stdbool.h on systems that lack a working one.
 all-local $(lib_OBJECTS): $(STDBOOL_H)
 stdbool.h: stdbool_.h
-       sed -e 's/@''HAVE__BOOL''@/$(HAVE__BOOL)/g' < $(srcdir)/stdbool_.h > 
stdbool.h-t
-       mv stdbool.h-t stdbool.h
-MOSTLYCLEANFILES += stdbool.h stdbool.h-t
+       sed -e 's/@''HAVE__BOOL''@/$(HAVE__BOOL)/g' $(srcdir)/stdbool_.h > 
address@hidden
+       mv address@hidden $@
+MOSTLYCLEANFILES += stdbool.h stdbool.ht
 
 Include:
 #include <stdbool.h>
 
 Maintainer:
 Bruno Haible
-