[MPlayer-dev-eng] [PATCH] RFC: CrystalHD decoder support
philipl at overt.org
Wed Dec 22 18:18:52 CET 2010
On Wed, 22 Dec 2010 14:04:44 +0100
Reimar Döffinger <Reimar.Doeffinger at gmx.de> wrote:
> If you are porting it to FFmpeg that is pretty much irrelevant, but I
> just realize that you are using the "videcodec" name to decide which
> decoder to use. That is a bad idea, since that is user-visible and we
> should be able to change that without changing program behaviour.
> You should instead change the "dll" as it is done for all FFmpeg
> decoders for example.
Yes, I guess it's irrelevant in the grand scheme of things, but I've
made the change in my local tree.
Jarod was nice enough to point me to the crystalhd-development list;
I'm going to be working with them to try and sort out the fundamental
problems and then I'll post an updated patch here; it'll probably be
past the weekend, but I wanted to make clear I'm not giving up :-)
I'll also address Diego's comments too.
More information about the MPlayer-dev-eng