[Ffmpeg-devel] [PATCH] PRId64 / PRIx64 changes

Diego Biurrun diego
Wed Nov 1 23:40:31 CET 2006


On Wed, Nov 01, 2006 at 11:03:03PM +0100, Michael Niedermayer wrote:
> 
> On Wed, Nov 01, 2006 at 09:28:38PM +0100, Steve Lhomme wrote:
> > Michael Niedermayer wrote:
> > >
> > >On Mon, Oct 23, 2006 at 06:32:04PM +0200, Steve Lhomme wrote:
> > >>
> > >>Here is a patch to replace most of the %lld and %llx into their 
> > >>(cleaner) PRI*64 counterparts.
> > >
> > >[...]
> > >>-    printf("get_v %5lld / %llX in %s %s:%d\n", v, v, file, func, line);
> > >>+    printf("get_v %5lld / %"PRIx64" in %s %s:%d\n", v, v, file, func, 
> > >>line);
> > >
> > >wrong, should be PRIX64 and you missed the %5lld (and there are more of 
> > >these)
> > 
> > Here it is with PRIX64 and %(whatever)ll(whatever).
> 
> looks ok, i hope none where actual long long, but even if, the type
> missmatches with the patch applied would still decrease ...

Patch applied.

Diego




More information about the ffmpeg-devel mailing list