On Thu, Oct 15, 2015 at 07:26:04PM -0400, Ganesh Ajjanagadde wrote:
> This one should not trigger any warnings, but will be useful for future
> robustness.
> 
> Signed-off-by: Ganesh Ajjanagadde <gajjanaga...@gmail.com>
> ---
>  libavutil/audio_fifo.h | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/libavutil/audio_fifo.h b/libavutil/audio_fifo.h
> index d93be2a..24f91da 100644
> --- a/libavutil/audio_fifo.h
> +++ b/libavutil/audio_fifo.h
> @@ -73,6 +73,7 @@ AVAudioFifo *av_audio_fifo_alloc(enum AVSampleFormat 
> sample_fmt, int channels,
>   * @param nb_samples  new allocation size, in samples
>   * @return            0 if OK, or negative AVERROR code on failure
>   */
> +av_warn_unused_result
>  int av_audio_fifo_realloc(AVAudioFifo *af, int nb_samples);

probably ok
strictly speaking one could check the size after the call instead of
the returb value i think

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

It is what and why we do it that matters, not just one of them.

Attachment: signature.asc
Description: Digital signature

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to