[MPlayer-dev-eng] [PATCH] RFC: CrystalHD decoder support

Philip Langdale 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 mailing list