Re: [FFmpeg-devel] [PATCH] cmdutils: fix success path

2015-02-11 Thread Michael Niedermayer
On Wed, Feb 11, 2015 at 02:18:27PM +0100, Jean Delvare wrote: > Since commit 934f2d2f5c16df8aad9f401a9fd842b5d9a78b11, > cmdutils_read_file() prints a confusing message on success: > > IO error: Success > > This is because the error message is printed on the success path as > well. Add the missin

[FFmpeg-devel] [PATCH] cmdutils: fix success path

2015-02-11 Thread Jean Delvare
Since commit 934f2d2f5c16df8aad9f401a9fd842b5d9a78b11, cmdutils_read_file() prints a confusing message on success: IO error: Success This is because the error message is printed on the success path as well. Add the missing condition so that it is only printed on error. Signed-off-by: Jean Delvar