[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