[FFmpeg-devel] [PATCH] avutil/bprint: make header work with C++, simplify

Reimar Döffinger Reimar.Doeffinger at gmx.de
Mon Nov 24 16:54:08 CET 2014


On 24.11.2014, at 15:27, Nicolas George <george at nsup.org> wrote:
> Le quartidi 4 frimaire, an CCXXIII, wm4 a écrit :
>> This change should be ABI-compatible. Since the size of the useable
>> internal buffer was actually different on 32 bit and 64 bit, so the code
>> contains a somewhat ugly kludge to maintain ABI-compatibility. It can be
>> removed on the next ABI-bump.
> 
> No. Please read the discussion that lead to the current implementation for
> the explanations.

Hm, I'd say that discussion should probably be captured in comments next to the code.


More information about the ffmpeg-devel mailing list