[FFmpeg-devel] [PATCH] CDXL demuxer and decoder

Michael Niedermayer michaelni at gmx.at
Fri Dec 30 04:24:19 CET 2011

On Thu, Dec 29, 2011 at 08:10:20PM -0500, Derek Buitenhuis wrote:
> On 29/12/2011 8:08 PM, Reimar Döffinger wrote:
> >On 30 Dec 2011, at 02:05, Derek Buitenhuis<derek.buitenhuis at gmail.com>  wrote:
> >>On 29/12/2011 7:55 PM, Reimar Döffinger wrote:
> >>>give it some fixed value, like this it gets G2N which makes no sense.
> >>
> >>This would potentially break ABI compat with the fork
> >
> >Giving them a fixed values is exactly what was done to keep ABI compatibility.
> Perhaps Micheal could comment here...

if the fork adds it at the same place then its surely the best to add
it there too
If they put something else at the end of the array then its better to
give it a random fixed value and support both values. Otherwise ABI
would break

My attempts at communicating with them in the past about such issues
have all failed. If you have more luck then sure put it at the end
of the respective list

Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

If a bugfix only changes things apparently unrelated to the bug with no
further explanation, that is a good sign that the bugfix is wrong.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20111230/3c8687f1/attachment.asc>

More information about the ffmpeg-devel mailing list