[vlc-devel] [PATCH 01/15] display: make vd->owner a const pointer

Steve Lhomme robux4 at ycbcr.xyz
Sat Sep 5 07:53:28 CEST 2020


OK

For the nvdec_pool (better name to be found) I also used an owner. And 
in the end when the pool is released, it tells the owner it can be 
released too. So a const is not very valid here, even though from the 
point of view of the nvdec_pool, it should not be altered.

On 2020-09-04 15:32, Rémi Denis-Courmont wrote:
> Le perjantaina 4. syyskuuta 2020, 8.19.26 EEST Steve Lhomme a écrit :
>> I agree, but this is also for consistency with the rest of the patches.
> 
> 1) It is *not* consistent with the other existing owner structures.
> 
>> In the end only the parts that the display module is allowed to change
>> are not const.
> 
> 2) It cannot remain consistent within vout_display_t, as we could very well
> add a 'const vout_display_operations *ops;' member that would *not* be read-
> only for the module - just like we already have vout_window_t.ops.
> 
> -- 
> レミ・デニ-クールモン
> http://www.remlab.net/
> 
> 
> 
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel
> 


More information about the vlc-devel mailing list