[vlc-devel] [PATCH 7/7] vout: display: pass the vout_display_priv_t directly in internal calls
remi at remlab.net
Thu Oct 10 08:29:45 CEST 2019
And I disagree with the other patch too, so...
This is totally against common practices for doing OOP-like code in non-OOP languages like C. You can't even do that sort of change in OOP...
Le 10 octobre 2019 09:15:00 GMT+03:00, Steve Lhomme <robux4 at ycbcr.xyz> a écrit :
>On 2019-10-09 18:19, Rémi Denis-Courmont wrote:
>> This feels very backward. We don't normally pass the private data.
>> compilers are perfectly capable to optimize the back-and-forth
>> the change is completely useless.
>container_of() is a glorified cast. One should not use cast unless
>there's no other option.
>This is exactly the same as was done in
>vlc-devel mailing list
>To unsubscribe or modify your subscription options:
Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the vlc-devel