[FFmpeg-devel] [PATCH] libavdevice/v4l2.c: av_log DEBUG cast

Reimar Döffinger Reimar.Doeffinger
Fri Feb 13 14:35:55 CET 2009


On Fri, Feb 13, 2009 at 02:18:56PM +0100, Patrik Kullman wrote:
> On Tue, 2009-02-10 at 19:41 +0000, M?ns Rullg?rd wrote:
> > Reimar D?ffinger <Reimar.Doeffinger at stud.uni-karlsruhe.de> writes:
> > 
> > > On Tue, Feb 10, 2009 at 06:28:52PM +0000, M?ns Rullg?rd wrote:
> > >> Luca Abeni <lucabe72 at email.it> writes:
> > >> > Looks like the correct patch is changing "PRIu64" in Lu (linux's __u64 
> > >> > is not uint64_t...)
> > >> 
> > >> %Lu is not a standard format specifier.  The standard modifier for
> > >> "long long" is ll, but I'm not sure what the actual underlying type
> > >> here is.
> > >
> > > Please do not use "ll", it does not work on MinGW.
> > 
> > That doesn't matter, since the entire file is Linux-specific.
> 
> Does this mean we can use %Lu?
> Since that seems to work on both 32 and 64 bit Linux?

My opinion: It will just give people bad ideas, a cast to uint64_t seems
more appropriate to me.




More information about the ffmpeg-devel mailing list