On Mon, Sep 19, 2011 at 03:30:57PM +0200, David Goldwich wrote:
> On 9/19/11 12:33 PM, Anton Khirnov wrote:
> > 
> > On Sun, 18 Sep 2011 18:24:53 +0200, David Goldwich 
> > <[email protected]> wrote:
> >>
> >> +    geob_data = av_mallocz(sizeof(ID3v2ExtraMetaGEOB));
> >> +    if (!geob_data) {
> >> +        av_log(s, AV_LOG_ERROR, "Failed to alloc %"PRIuPTR" bytes\n", 
> >> sizeof(ID3v2ExtraMetaGEOB));
> > 
> > PRIuPTR? sizeof(foo) should be printfed with %zd IIRC. Same below.
> 
> Fixed. Is that portable?

Yes, but please use %zu like we do in other places.

Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to