On 02/10/15 12:43, Vittorio Giovara wrote:
> Fix a rather lenghty warning from clang:
> rangecoder.c:58:34: warning: passing
> 'uint8_t **' (aka 'unsigned char **') to parameter of type
> 'const uint8_t **' (aka 'const unsigned char **') discards qualifiers in
> nested pointer types [-Wincompatible-pointer-types-discards-qualifiers]
> ---
> libavcodec/rangecoder.c | 6 ++++--
> 1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/libavcodec/rangecoder.c b/libavcodec/rangecoder.c
> index af0a8c0..9897f58 100644
> --- a/libavcodec/rangecoder.c
> +++ b/libavcodec/rangecoder.c
> @@ -34,9 +34,10 @@
> #include <string.h>
>
> #include "libavutil/attributes.h"
> +#include "libavutil/intreadwrite.h"
> +
> #include "avcodec.h"
> #include "rangecoder.h"
> -#include "bytestream.h"
>
> av_cold void ff_init_range_encoder(RangeCoder *c, uint8_t *buf, int buf_size)
> {
> @@ -55,7 +56,8 @@ av_cold void ff_init_range_decoder(RangeCoder *c, const
> uint8_t *buf,
> /* cast to avoid compiler warning */
> ff_init_range_encoder(c, (uint8_t *)buf, buf_size);
>
> - c->low = bytestream_get_be16(&c->bytestream);
> + c->low = AV_RB16(c->bytestream);
> + c->bytestream += 2;
> }
>
> void ff_build_rac_states(RangeCoder *c, int factor, int max_p)
>
Probably Ok, it isn't any unsafer anyway.
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
