[FFmpeg-devel] [PATCH]lavd/decklink_dec: Make bitrate calculation 64bit

Michael Niedermayer michael at niedermayer.cc
Thu Apr 28 21:43:54 CEST 2016


On Thu, Apr 28, 2016 at 05:30:36PM +0200, Nicolas George wrote:
> Le decadi 10 floréal, an CCXXIV, Carl Eugen Hoyos a écrit :
> > I hope attached patch fixes an issue seen in a decklink user report where 
> > the bit_rate was negative. Or should the division be changed to 1ULL/...?
> 
> This does not make the computation 64 bits, it makes them whatever the size
> happens to be for long long on this compiler and architecture. To make the
> computations 64 bits, a cast is in order: "(uint64_t)".

It would make them 64 bit or larger if this wasnt double from
av_q2d() already

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The real ebay dictionary, page 1
"Used only once"    - "Some unspecified defect prevented a second use"
"In good condition" - "Can be repaird by experienced expert"
"As is" - "You wouldnt want it even if you were payed for it, if you knew ..."
-------------- 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/20160428/41e3cfea/attachment.sig>


More information about the ffmpeg-devel mailing list