Matt Turner <matts...@gmail.com> writes: > On Fri, Jul 17, 2015 at 10:17 AM, Emil Velikov <emil.l.veli...@gmail.com> > wrote: >> Cc: Eric Anholt <e...@anholt.net> >> Signed-off-by: Emil Velikov <emil.l.veli...@gmail.com> >> --- >> src/util/tests/hash_table/Makefile.am | 3 +-- >> 1 file changed, 1 insertion(+), 2 deletions(-) >> >> diff --git a/src/util/tests/hash_table/Makefile.am >> b/src/util/tests/hash_table/Makefile.am >> index 04a77e3..0c99e7b 100644 >> --- a/src/util/tests/hash_table/Makefile.am >> +++ b/src/util/tests/hash_table/Makefile.am >> @@ -38,7 +38,6 @@ TESTS = \ >> null_destroy \ >> random_entry \ >> remove_null \ >> - replacement \ >> - $() >> + replacement > > To get the benefit of $() without some unknown incompatibility, pixman > uses $(NULL) which of course relies on not having a variable named > NULL. > > I might suggest that instead of removing them, but I'm not much > opposed to removing them either.
I do really like having a terminator on these lists. I find that without them, I'll end up copy-and-pasting the wrong thing and missing the trailing backslash on a line.
signature.asc
Description: PGP signature
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
