[FFmpeg-devel] [PATCH v2 00/16] Replace native DCA decoder with libdcadec based one
h.leppkes at gmail.com
Sat Jan 30 19:05:16 CET 2016
On Sat, Jan 30, 2016 at 1:43 AM, Andreas Cadhalpun
<andreas.cadhalpun at googlemail.com> wrote:
> On 25.01.2016 23:47, Hendrik Leppkes wrote:
>> The decoder in itself looks fine to me, short of the regression michael found.
>> If you could look at that, that would be great.
>> I can squash and re-shuffle the commits appropriately for pushing and
>> make sure all intermediate steps still build, so don't worry about
>> that as much.
>> I would love to get this in soon, we can do further improvements and
>> more FATE coverage after.
> I agree that it's time to apply these patches. Resending this huge
> patch set for tiny improvements is just not practical.
>> There doesn't seem to be much agreement yet of the order of pushing.
>> I would argue that since we're replacing the decoder entirely anyway,
>> a tiny period in between where we don't actually have a dca decoder
>> wouldn't break any bisect flow, since it would probably end there
>> So considering that, it feels cleaner to me to push the removal first,
>> and then the additions for the new decoder for "prettier" history.
> That seems fine to me, but I don't have a strong opinion about this.
I have started to rebase, update and squash it appropriately,
including the patch for Michaels issue I posted earlier.
Once its all ready for pushing, I'll post my GitHub link for a final
review if anyone wants to, and otherwise push it in the next day or
two, so we finally get this done.
Incremental improvements can then be made in tree afterwards.
More information about the ffmpeg-devel