Re: [FFmpeg-devel] [PATCH] avutil/file: add av_warn_unused_result to av_file_map

2015-10-16 Thread Ganesh Ajjanagadde
On Fri, Oct 16, 2015 at 11:30 AM, Michael Niedermayer wrote: > On Thu, Oct 15, 2015 at 06:19:08PM -0400, Ganesh Ajjanagadde wrote: >> The return code here should be checked. >> >> Signed-off-by: Ganesh Ajjanagadde >> --- >> libavutil/file.h | 1 + >> 1 file changed, 1 insertion(+) > > LGTM > > t

Re: [FFmpeg-devel] [PATCH] avutil/file: add av_warn_unused_result to av_file_map

2015-10-16 Thread Michael Niedermayer
On Thu, Oct 15, 2015 at 06:19:08PM -0400, Ganesh Ajjanagadde wrote: > The return code here should be checked. > > Signed-off-by: Ganesh Ajjanagadde > --- > libavutil/file.h | 1 + > 1 file changed, 1 insertion(+) LGTM thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC7

[FFmpeg-devel] [PATCH] avutil/file: add av_warn_unused_result to av_file_map

2015-10-15 Thread Ganesh Ajjanagadde
The return code here should be checked. Signed-off-by: Ganesh Ajjanagadde --- libavutil/file.h | 1 + 1 file changed, 1 insertion(+) diff --git a/libavutil/file.h b/libavutil/file.h index 1cae295..e931be7 100644 --- a/libavutil/file.h +++ b/libavutil/file.h @@ -40,6 +40,7 @@ * @return a non n