[FFmpeg-devel] [PATCH] avcodec: Switch bitrate to 64bit with the bump

Michael Niedermayer michaelni at gmx.at
Wed Sep 2 22:50:30 CEST 2015


On Wed, Sep 02, 2015 at 01:47:07PM -0700, Ganesh Ajjanagadde wrote:
> On Wed, Sep 2, 2015 at 1:20 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
> > From: Michael Niedermayer <michael at niedermayer.cc>
> >
> > a 32bit bitrate is insufficient for high resolution, high framerate material
> > an example would be rawvideo
> >
> > Not all changes are covered by #if as its easier to just push when the
> > bump is done instead of making it coditional and removig the conditionallity
> >  again
> 
> Not for this patch; but just a thought - I noticed in swresample the
> use of int for sample rate. Note that all the C spec guarantees is int
> >= 16 bits, and 2^15 < 33000 which is insufficient for a lot of audio.
> Your comment above seems to reflect the assumption that int >= 32
> bits. Maybe this assumption is made elsewhere in the codebase as well,
> but it is not safe as per the standard. Thus, I feel swresample rate
> should be an int32_t; and there may be more opportunities for cleanup
> elsewhere.

POSIX gurantees >=32bit for int

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

If you think the mosad wants you dead since a long time then you are either
wrong or dead since a long time.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150902/40a744a3/attachment.sig>


More information about the ffmpeg-devel mailing list