On Mon, Oct 22, 2012 at 03:39:01AM +0200, Felipe Contreras wrote: > By using print_comp as suggested by SZEDER Gábor. > > Signed-off-by: Felipe Contreras <felipe.contreras@xxxxxxxxx> > --- > t/t9902-completion.sh | 13 +++++-------- > 1 file changed, 5 insertions(+), 8 deletions(-) > > diff --git a/t/t9902-completion.sh b/t/t9902-completion.sh > index 1c6952a..2e7fc06 100755 > --- a/t/t9902-completion.sh > +++ b/t/t9902-completion.sh > @@ -74,15 +74,12 @@ newline=$'\n' This $newline variable was only used to set IFS to a newline inside SQ blocks. AFAICS after this change there are no such places left, because print_comp() takes care of IFS, so $newline is not necessary anymore. > test_gitcomp () > { > + local -a COMPREPLY && > sed -e 's/Z$//' > expected && > - ( > - local -a COMPREPLY && > - cur="$1" && > - shift && > - __gitcomp "$@" && > - IFS="$newline" && > - echo "${COMPREPLY[*]}" > out > - ) && > + cur="$1" && > + shift && > + __gitcomp "$@" && > + print_comp && > test_cmp expected out > } > > -- > 1.8.0 > -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html