[Ffmpeg-devel] fix some warnings

Michael Niedermayer michaelni
Wed Jan 25 12:18:42 CET 2006


Hi

On Wed, Jan 25, 2006 at 12:09:24PM +0100, Luca Abeni wrote:
> Hi all,
> 
> when compiling the current CVS version on debian unstable (gcc version
> 4.0.3 20060115 (prerelease) (Debian 4.0.2-7)), I get a lot of warings
> about pointers signedness. I attach the list of all the errors.
> 
> The attached patch (warn_fixes.diff) fixes the most obvious warnings (I
> am not sure about how to fix the other ones).
> 
> After applying the patch, "make codectest" and "make libavtest" show no
> regressions.

[...]

> @@ -6971,7 +6971,7 @@ static int decode_slice(H264Context *h){
>  
>  static int decode_unregistered_user_data(H264Context *h, int size){
>      MpegEncContext * const s = &h->s;
> -    uint8_t user_data[16+256];
> +    char user_data[16+256];
>      int e, build, i;

IMHO uint8_t is more correct here

the other stuff looks ok and can be applied ...

-- 
Michael





More information about the ffmpeg-devel mailing list