[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