[MPlayer-dev-eng] Re: PATCH [2/12] CoreAVC Decoder Support

[Alan Nisota]

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.