Thread (1 message) 1 message, 1 author, 2016-06-15

Re: [PATCH v2 19/19] mingw: do not bother to test funny file names

From: Junio C Hamano <hidden>
Date: 2016-06-15 23:07:54

Eric Sunshine [off-list ref] writes:
quoted
+if ! test_have_prereq MINGW && touch -- 'tab   embedded' 'newline
[...]
+test_have_prereq !MINGW &&
Where negation is concerned, is there a non-obvious reason that this
patch sometimes says:

    ! test_have_prereq MINGW

and sometimes:

    test_have_prereq !MINGW

? Is one form preferred over the other?
The latter may be preferrable as it can be extended more easily,
e.g.

	test_have_prereq !MINGW,SANITY

without having to cascade "&& test_have_prereq ANOTHER_PREREQ" in a
verbose way.

It is a minor thing that is not worth bothering to "fix" from one to
the other style once it is committed to my tree, but it may be worth
sticking to one style in new code.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help