[FFmpeg-devel] [PATCH] ffprobe: make writer_print_integer support to long int values

Stefano Sabatini stefasab at gmail.com
Wed Nov 16 18:59:32 CET 2011


On date Wednesday 2011-11-16 07:39:12 +0100, Clément Bœsch encoded:
> On Wed, Nov 16, 2011 at 12:43:59AM +0100, Stefano Sabatini wrote:
> > This makes possible to use writer_print_integer for printing int64_t
> > values.
> 
> Any reason not to use int64_t?
> It can cause various issues; AFAIK Windows' long is 32 bit long for
> instance, even in 64 bits. Maybe you wanted a "long long int"?

Yes I meant long long int, which is possibly more portable, but
there's no compelling reason to prefer it to int64_t AFAIK, and long
long int is required to contain int64_t (at least).

Up, I'll push it soon if no comments appear.
-- 
FFmpeg = Frenzy Free Mega Portentous Exuberant Guide
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-ffprobe-make-writer_print_integer-support-long-long-.patch
Type: text/x-diff
Size: 3650 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20111116/54ee0234/attachment.bin>


More information about the ffmpeg-devel mailing list