[vlc-devel] [PATCH v2] package/win32: build.sh: trigger build errors on dubious pointer use
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.
More information about the vlc-devel