Re: [FFmpeg-devel] [PATCH] tests: drop bc dependency

2015-02-10 Thread Clément Bœsch
On Tue, Feb 10, 2015 at 04:02:53AM +0100, Reimar Döffinger wrote: > On 08.02.2015, at 10:32, Clément Bœsch wrote: > >> --- a/tests/fate-run.sh > >> +++ b/tests/fate-run.sh > >> @@ -38,7 +38,7 @@ target_path(){ > >> # $1=value1, $2=value2, $3=threshold > >> # prints 0 if absolute difference between

Re: [FFmpeg-devel] [PATCH] tests: drop bc dependency

2015-02-09 Thread Reimar Döffinger
On 08.02.2015, at 10:32, Clément Bœsch wrote: >> --- a/tests/fate-run.sh >> +++ b/tests/fate-run.sh >> @@ -38,7 +38,7 @@ target_path(){ >> # $1=value1, $2=value2, $3=threshold >> # prints 0 if absolute difference between value1 and value2 is <= threshold >> compare(){ >> -echo "scale=2; v = $1

Re: [FFmpeg-devel] [PATCH] tests: drop bc dependency

2015-02-08 Thread Clément Bœsch
On Sun, Feb 08, 2015 at 10:27:14AM +0100, Clément Bœsch wrote: > On Tue, Jan 20, 2015 at 09:58:33PM +0100, Reimar Döffinger wrote: > > On Mon, Jan 19, 2015 at 10:58:02PM +0100, Clément Bœsch wrote: > > > -echo "scale=2; v = $1 - $2; if (v < 0) v = -v; if (v > $3) r = 1; r" > > > | bc > > > +

Re: [FFmpeg-devel] [PATCH] tests: drop bc dependency

2015-02-08 Thread Clément Bœsch
On Tue, Jan 20, 2015 at 09:58:33PM +0100, Reimar Döffinger wrote: > On Mon, Jan 19, 2015 at 10:58:02PM +0100, Clément Bœsch wrote: > > -echo "scale=2; v = $1 - $2; if (v < 0) v = -v; if (v > $3) r = 1; r" | > > bc > > +awk "BEGIN { v=$1-$2; printf \"%d\\n\", ((v<0?v:-v) > $3) ? 1 : 0 }" >

Re: [FFmpeg-devel] [PATCH] tests: drop bc dependency

2015-01-20 Thread Reimar Döffinger
On Mon, Jan 19, 2015 at 10:58:02PM +0100, Clément Bœsch wrote: > -echo "scale=2; v = $1 - $2; if (v < 0) v = -v; if (v > $3) r = 1; r" | bc > +awk "BEGIN { v=$1-$2; printf \"%d\\n\", ((v<0?v:-v) > $3) ? 1 : 0 }" I'd suggest single quotes for the outer ones, then you do not need to escape \

Re: [FFmpeg-devel] [PATCH] tests: drop bc dependency

2015-01-20 Thread Michael Niedermayer
On Mon, Jan 19, 2015 at 10:58:02PM +0100, Clément Bœsch wrote: > We already have a dependency on awk, bc is sometimes not found in the > base system, and we save a shell fork. > --- > tests/fate-run.sh | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) looks ok but iam no awk/bc/shell guru s

[FFmpeg-devel] [PATCH] tests: drop bc dependency

2015-01-19 Thread Clément Bœsch
We already have a dependency on awk, bc is sometimes not found in the base system, and we save a shell fork. --- tests/fate-run.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/fate-run.sh b/tests/fate-run.sh index f78e4fc..52d14fe 100755 --- a/tests/fate-run.sh +++ b/t