[vlc-devel] [PATCH 1/9] vlc_list: add a typedef vlc_list_node to clarify which is the list and which is the node
robux4 at ycbcr.xyz
Mon Aug 20 09:16:39 CEST 2018
On 17/08/2018 19:15, Rémi Denis-Courmont wrote:
> By design, you cannot distinguish list nodes and list heads types in
> circular list implementations.
I was under the impression vlc_list is an implementation of "intrusive
lists" not that it's a circular list. The names "intrusive" or
"intrusion" are not anywhere in the code so I might be wrong.
As an intrusive list (forget what follows if that's not the case) the
intrusion needs to be documented and taken care of very carefully.
Having a field in a structure "vlc_list node" does not say anything
about it. If you receive pointers to such a struct are you allowed to
list them using a vlc_list ? The answer is probably no. So it should
explicitly say who is allowed to modify the "node" field. In most cases
that's one particular vlc_list list (currently all cases).
This is no different that using pointers and knowing who is allowed to
set it and free it. One would not reuse the same pointer field for
completely unrelated allocations.
> This series is counter-sensical.
You mean clarifying things for me makes less sense for you ?
> And there are no issues when using the helpers, so I do not see what
> this solves even.
I can already find a person who had issues with the helpers: me.
More information about the vlc-devel