[vlc-devel] [PATCH v2] package/win32: build.sh: trigger build errors on dubious pointer use

Rémi Denis-Courmont remi at remlab.net
Tue Mar 17 17:24:07 CET 2020


Le tiistaina 17. maaliskuuta 2020, 16.32.55 EET Steve Lhomme a écrit :
> > Again *almost* every COM instantiation in the VLC code base (and in many
> > other code bases) is an invalid pointer conversion. And it's helplessly
> > muted by an explicit but invalid cast to void **.
> 
> I am not advocating to remove all casts from the code. Sometimes it's
> useful. I'm talking about pointers that are simply from incompatible
> types (incompatible-pointer-types) that cannot be casted away without
> breaking things.

And I'm saying that those warnings are better be left as warnings, because 
otherwise they will cause build failures for harmless or mostly harmless 
matters, *and* they will create an extra incentive to make wrong casts, 
exactly like COM already shows how NOT to do.

And that's what was decided already.

-- 
Rémi Denis-Courmont
http://www.remlab.net/





More information about the vlc-devel mailing list