[FFmpeg-devel] [PATCH v1 1/2] lavc/vp9: set yuvj pixel format for full range decode

[Thilo Borgmann]

Am 17.06.23 um 13:53 schrieb Paul B Mahol:
> On Sat, Jun 17, 2023 at 1:47 PM Thilo Borgmann <thilo.borgmann at mail.de>
> wrote:
> 
>> Am 17.06.23 um 13:40 schrieb Paul B Mahol:
>>> NAK, there is color_range that should not be ignored.
>>
>> In patch 1/2 color_range is used in the condition for the switch.
>>
>> if (avctx->color_range == AVCOL_RANGE_JPEG) {
>>
>> Or I don't understand. What shall be done with color_range?
>>
> 
> Actually used, and all YUVJ uses finally removed from codebase.

Complete removal shall be done, yes. (Hopefully very soon actually)
However, as long as we didn't and nobody can really guarantee anything about when it will be, having this is better than the current state.



>>> Fix finally scale filter.
>>
>> Hmm? Where what who?
>>
> 
> Perhaps just frame->color_range needs to be set?

Where does a scale filter correlate with this?
Do you mean setting this in 2/2? I can't see what you want me to do here...

-Thilo