[FFmpeg-devel] [PATCH 03/11] avdevice/sndio: add av_warn_unused_result

Michael Niedermayer michael at niedermayer.cc
Thu Oct 29 22:53:54 CET 2015


Hi Brad

do you have a oppinion on the change below ?
you are the author of the code


On Thu, Oct 15, 2015 at 10:22:17PM -0400, Ganesh Ajjanagadde wrote:
> This does not trigger any warnings, but adds robustness.
> Untested, as my configure does not compile this file.
> 
> Signed-off-by: Ganesh Ajjanagadde <gajjanagadde at gmail.com>
> ---
>  libavdevice/sndio.h | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/libavdevice/sndio.h b/libavdevice/sndio.h
> index 54a5ec3..6571429 100644
> --- a/libavdevice/sndio.h
> +++ b/libavdevice/sndio.h
> @@ -42,6 +42,7 @@ typedef struct SndioData {
>      int sample_rate;
>  } SndioData;
>  
> +av_warn_unused_result
>  int ff_sndio_open(AVFormatContext *s1, int is_output, const char *audio_device);
>  int ff_sndio_close(SndioData *s);
>  
> -- 
> 2.6.1
> 
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

What does censorship reveal? It reveals fear. -- Julian Assange
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20151029/ba471de9/attachment.sig>


More information about the ffmpeg-devel mailing list