[FFmpeg-cvslog] r25066 - trunk/libavcodec/imgconvert.c

[Måns Rullgård]

Reimar D?ffinger <Reimar.Doeffinger at gmx.de> writes:

> On Tue, Sep 21, 2010 at 03:05:08PM +0100, M?ns Rullg?rd wrote:
>> Stefano Sabatini <stefano.sabatini-lala at poste.it> writes:
>> > and I can barely remember the reason of the gray8=PAL thing.
>> 
>> It seems that if anyone is able to remember or understand this bizarre
>> state of affairs, they are unwilling to explain it to the rest of us.
>
> I did my best, if anyone did not understand it it's beyond my abilities
> to explain.

I got lost at the part about how doing a table lookup per pixel would
somehow be easier than not doing it.

>> Whatever the reason may be, we need to fix this mess properly once and
>> for all,
>
> The only "mess" is in the raw decoder which tries to avoid copying
> of which this is only one issue (demuxer packets magically changing
> context when the decoded image is modified is another for example).
> I sent a patch, however it really should be done a bit cleaner.
>
>> or else this bug will only resurface again at some later time
>> when the next person fails to imagine anything could be so absurd.
>
> This "bug" can only appear in codecs that do not use get_buffer,
> which limits the issue quite a lot IMO.

Since you apparently have some notion of how it should be fixed, could
you _please_ try to make a patch?

-- 
M?ns Rullg?rd
mans at mansr.com