[Mesa-dev] [PATCH 1/5] utils: automake: remove uncommon $()
emil.l.velikov at gmail.com
Fri Jul 17 11:21:20 PDT 2015
On 17 July 2015 at 19:11, Ilia Mirkin <imirkin at alum.mit.edu> wrote:
> On Fri, Jul 17, 2015 at 2:11 PM, Eric Anholt <eric at anholt.net> wrote:
>> Matt Turner <mattst88 at gmail.com> writes:
>>> On Fri, Jul 17, 2015 at 10:17 AM, Emil Velikov <emil.l.velikov at gmail.com> wrote:
>>>> Cc: Eric Anholt <eric at anholt.net>
>>>> Signed-off-by: Emil Velikov <emil.l.velikov at 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
>>> 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.
> Also makes diffs easier to read since you don't have spurious changes
> which just add a \ .
Indeed. In the case of the earlier egl series they would have improve
the readability of the diff.
Such patches are very rare though, so considering that we have only a
couple sentinels I would opt for consistency and nuke them.
More information about the mesa-dev