r...@remlab.net:
> From: Rémi Denis-Courmont <r...@remlab.net>
> 
> While this probably never overflows, we are better safe than sorry.
> 
> The callback prototype should probably also use ptrdiff_t or size_t but
> I  diggress.
> ---
>  libavcodec/vorbisdec.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libavcodec/vorbisdec.c b/libavcodec/vorbisdec.c
> index 38a5367be3..9728997370 100644
> --- a/libavcodec/vorbisdec.c
> +++ b/libavcodec/vorbisdec.c
> @@ -1581,7 +1581,7 @@ static inline int vorbis_residue_decode(vorbis_context 
> *vc, vorbis_residue *vr,
>  
>  void ff_vorbis_inverse_coupling(float *mag, float *ang, intptr_t blocksize)
>  {
> -    int i;
> +    ptrdiff_t i;
>      for (i = 0;  i < blocksize;  i++) {
>          if (mag[i] > 0.0) {
>              if (ang[i] > 0.0) {

We allow the C99 "for (ptrdiff_t i = 0;".

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

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Reply via email to