undef/git-commit-vandalism
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

test: rename $satisfied to $satisfied_prereq

Browse Source 
All other shell variables that are used to globally keep track of
states related to prerequisite have "prereq" somewhere in their
names.  Be consistent and avoid potential name crashes with other
kinds of satisfaction in the future.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
This commit is contained in:
Junio C Hamano 2012-07-26 13:57:56 -07:00
parent cdd159b2f5
commit f3cfc3b271
1 changed files with 3 additions and 3 deletions

6
t/test-lib-functions.sh
View File

@ -221,9 +221,9 @@ write_script () {
# capital letters by convention).
test_set_prereq () {
satisfied="$satisfied$1 "
satisfied_prereq="$satisfied_prereq$1 "
}
satisfied=" "
satisfied_prereq=" "
test_have_prereq () {
# prerequisites can be concatenated with ','
@ -239,7 +239,7 @@ test_have_prereq () {
for prerequisite
do
total_prereq=$(($total_prereq + 1))
case $satisfied in
case "$satisfied_prereq" in
*" $prerequisite "*)
ok_prereq=$(($ok_prereq + 1))
;;