[FFmpeg-devel] [PATCH] CrystalHD decoder support v3

[Reimar Döffinger]

On Sat, Feb 05, 2011 at 10:36:11AM -0800, Philip Langdale wrote:
> > > +static inline void print_frame_info(CHDContext *priv,
> > > BC_DTS_PROC_OUT *output) +{
> > > +    av_log(priv->avctx, AV_LOG_VERBOSE, "\tYBuffSz: %u\n",
> > > output->YbuffSz);
> > > +    av_log(priv->avctx, AV_LOG_VERBOSE, "\tYBuffDoneSz: %u\n",
> > > +           output->YBuffDoneSz);
> > > +    av_log(priv->avctx, AV_LOG_VERBOSE, "\tUVBuffDoneSz: %u\n",
> > > +           output->UVBuffDoneSz);
> > > +    av_log(priv->avctx, AV_LOG_VERBOSE, "\tTimestamp: %lu\n",
> > > +           output->PicInfo.timeStamp);
> > 
> > timestamp is an uint64_t
> 
> Yeah, and when compiling 64bit, %lu is correct. For 32bit, you need %llu
> which I assume if what you were referring to.

And (while we do not really support it) something completely different
for Windows (Microsoft C runtime).

> Is there a pre-existing
> macro substitution in the codebase anywhere?

C99, inttypes.h:
"%"PRIu64"\n"