[vlc-devel] [PATCH] picture: fix missing video_format_t attributes from the picture

Rémi Denis-Courmont remi at remlab.net
Fri Sep 28 14:23:59 CEST 2018


Le vendredi 28 septembre 2018, 15:04:33 EEST Filip Roséen a écrit :
> LGTM, as long as we are sure that we will not (once again) step into
> life-time issues related to the ownership of
> `video_format_t.p_palette` (but as it worked before, well.. it at
> least wouldn't get worse than what we had previously).

I have said it before and I will say it again: we should just kill p_palette 
so that video format is actually POD like so much code assumes. The troubles 
are not worth the benefits. It is just daft to treat paletized formats as 
decoded formats nowadays.

We should just have a common helper to dereference paletized pictures into YUV 
or RGB in the few affected video decoders.

-- 
Rémi Denis-Courmont




More information about the vlc-devel mailing list