[vlc-devel] [vlc-commits] demux: make p_sys void * (refs #18504, #17078)
robux4 at gmail.com
Mon Dec 4 12:43:20 CET 2017
On Mon, Dec 4, 2017 at 12:31 PM, Rémi Denis-Courmont <remi at remlab.net> wrote:
> So how does this solve the unification of stream_t and demux_t
> (p_sys is void in stream_t)?
Two wrongs don't make one right. IMO void* should never be used unless
all other options failed. But apparently it's already been decided
that void * is the way to go. So this discussion is moot.
> How does this permit storing anything but a structure as opaque data (incomplete struct pointers can only represent struct pointers)?
> And most ridiculously, how does this not just shift the ODR violation issue from demux_sys_t to demux_t?
The ODR issue can only be solved on a per module basis. At least my
patch would make it explicit. Maybe a similar system forcing a local
type for all XXX_sys_t would make it ever more explicit. Ensuring that
modules don't cause issues (at least that one) for static linking.
> Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté.
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
More information about the vlc-devel