Re: [FFmpeg-devel] [PATCH 1/2] tests/tiny_psnr: do not ignore errors from run_psnr

2015-04-23 Thread Michael Niedermayer
On Thu, Apr 23, 2015 at 08:38:15PM +0200, Andreas Cadhalpun wrote: > On 23.04.2015 15:44, Michael Niedermayer wrote: > > On Wed, Apr 22, 2015 at 09:10:51PM -0700, Timothy Gu wrote: > >> On Thu, Apr 23, 2015 at 04:43:09AM +0200, Michael Niedermayer wrote: > >>> failure to calculate psnr should not r

Re: [FFmpeg-devel] [PATCH 1/2] tests/tiny_psnr: do not ignore errors from run_psnr

2015-04-23 Thread Andreas Cadhalpun
On 23.04.2015 15:44, Michael Niedermayer wrote: > On Wed, Apr 22, 2015 at 09:10:51PM -0700, Timothy Gu wrote: >> On Thu, Apr 23, 2015 at 04:43:09AM +0200, Michael Niedermayer wrote: >>> failure to calculate psnr should not result in tiny_psnr returning success >>> >>> Signed-off-by: Michael Niederm

Re: [FFmpeg-devel] [PATCH 1/2] tests/tiny_psnr: do not ignore errors from run_psnr

2015-04-23 Thread Michael Niedermayer
On Wed, Apr 22, 2015 at 09:10:51PM -0700, Timothy Gu wrote: > On Thu, Apr 23, 2015 at 04:43:09AM +0200, Michael Niedermayer wrote: > > failure to calculate psnr should not result in tiny_psnr returning success > > > > Signed-off-by: Michael Niedermayer > > --- > > tests/tiny_psnr.c |9 ++

Re: [FFmpeg-devel] [PATCH 1/2] tests/tiny_psnr: do not ignore errors from run_psnr

2015-04-22 Thread Timothy Gu
On Thu, Apr 23, 2015 at 04:43:09AM +0200, Michael Niedermayer wrote: > failure to calculate psnr should not result in tiny_psnr returning success > > Signed-off-by: Michael Niedermayer > --- > tests/tiny_psnr.c |9 ++--- > 1 file changed, 6 insertions(+), 3 deletions(-) Patchset LGTM. W

[FFmpeg-devel] [PATCH 1/2] tests/tiny_psnr: do not ignore errors from run_psnr

2015-04-22 Thread Michael Niedermayer
failure to calculate psnr should not result in tiny_psnr returning success Signed-off-by: Michael Niedermayer --- tests/tiny_psnr.c |9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/tests/tiny_psnr.c b/tests/tiny_psnr.c index e09f394..5f791c1 100644 --- a/tests/tiny