[MPlayer-dev-eng] Re: PATCH [2/12] CoreAVC Decoder Support
alannisota at gmail.com
Fri Feb 9 16:54:22 CET 2007
Reimar Doeffinger writes:
> Sorry I was looking at line 241 as the patch said, but that offset is
> due to your other changes, the patch actually starts around line 150.
> I don't like it in so far as this approach will always work only with
> one colourspace, if we now had another codec that behaves just as stupid
> but gives RGB output we could never both get to work.
> Such a situation seems unlikely so it's not critical but still
To be fair, getting this codec to work has been very tricky. It is quite
possible that I can rework this patch to be more flexible. We have a list of
known output formats in codecs.conf, perhaps we can choose one of those for the
initial dest fmt (or we can ask the codec to suggest one, that should work as
well and would be easier to implement.
More information about the MPlayer-dev-eng