Junio C Hamano writes:
> Alexey Shumkin writes:
>
>> -msg=$(printf "modify 2nd file (ge\303\244ndert)")
>> +msg="modify 2nd file (ge\303\244ndert)"
>> if test -n "$1"
>> then
>> -msg=$(echo $msg | iconv -f utf-8 -t $1)
>> +print "$msg" | iconv -f utf-8 -
Alexey Shumkin writes:
> - msg=$(printf "modify 2nd file (ge\303\244ndert)")
> + msg="modify 2nd file (ge\303\244ndert)"
> if test -n "$1"
> then
> - msg=$(echo $msg | iconv -f utf-8 -t $1)
> + print "$msg" | iconv -f utf-8 -t "$1"
> + else
> +
Function 'test_format' has become harder to read after its
change in de6029a2 (pretty: Add failing tests: --format output
should honor logOutputEncoding, 2013-06-26). Simplify it by
moving its "should we expect it to fail?" parameter to the end.
Note, current code does not use this last parameter
3 matches
Mail list logo