[FFmpeg-devel] [PATCH] CDXL demuxer and decoder
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
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
Size: 198 bytes
Desc: Digital signature
More information about the ffmpeg-devel