Quoting foo86 (2016-01-03 18:50:13)
> This is initial version of libdcadec DCA decoder port I hacked together over
> the last week. This patch is of RFC/inquiry type and not meant for immediate
> inclusion.
> 
> Any comments about the code are appreciated, as well as suggestions concerning
> how should I proceed with this patch. Should I aim for including it as a
> separate decoder or a replacement for existing DCA decoder? Or it might be 
> that
> you prefer to continue improving existing decoder instead and not merge this 
> at
> all.
> 

Hmm, this is rather awkward. As you have noticed, Alexandra is currently
working on the native decoder. So either her or your work will be
wasted. I really wish you had let us know that you were planning to work
on this.

In any case, there is no point in having two decoders that do exactly
the same thing, so in the end there should be only one. I'm not familiar
enough with the current DCA decoder to tell whether it's less work to
finish and review this or add the missing stuff to the current one.
Perhaps Benjamin/Niels/Костя can judge that?

-- 
Anton Khirnov
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to