Re: [FFmpeg-devel] [PATCH] avutil/internal: add av_warn_unused_result to avpriv_open

2015-10-16 Thread Ganesh Ajjanagadde
On Fri, Oct 16, 2015 at 11:29 AM, Michael Niedermayer wrote: > On Thu, Oct 15, 2015 at 05:50:53PM -0400, Ganesh Ajjanagadde wrote: >> The open syscall can obviously fail, and its return code needs to be >> checked. >> >> Signed-off-by: Ganesh Ajjanagadde >> --- >> libavutil/internal.h | 1 + >>

Re: [FFmpeg-devel] [PATCH] avutil/internal: add av_warn_unused_result to avpriv_open

2015-10-16 Thread Michael Niedermayer
On Thu, Oct 15, 2015 at 05:50:53PM -0400, Ganesh Ajjanagadde wrote: > The open syscall can obviously fail, and its return code needs to be > checked. > > Signed-off-by: Ganesh Ajjanagadde > --- > libavutil/internal.h | 1 + > 1 file changed, 1 insertion(+) LGTM thx [...] -- Michael GnuPG

[FFmpeg-devel] [PATCH] avutil/internal: add av_warn_unused_result to avpriv_open

2015-10-15 Thread Ganesh Ajjanagadde
The open syscall can obviously fail, and its return code needs to be checked. Signed-off-by: Ganesh Ajjanagadde --- libavutil/internal.h | 1 + 1 file changed, 1 insertion(+) diff --git a/libavutil/internal.h b/libavutil/internal.h index 13dbd3b..5c2cd99 100644 --- a/libavutil/internal.h +++ b/