[vlc-devel] [PATCH] configure: fail on casting incompatible pointers

Sebastian Ramacher sebastian+lists at ramacher.at
Wed Oct 2 21:34:58 CEST 2019

On 2019-10-02 21:05:05, Rémi Denis-Courmont wrote:
> Le keskiviikkona 2. lokakuuta 2019, 17.12.26 EEST Denis Charmet a écrit :
> > On 2019-10-02 15:38, Steve Lhomme wrote:
> > > That could be nice. Also there's no build script for Linux, so most
> > > Linux devs won't have them by default. So I'd vote to have the flag
> > > enabled by default.
> > 
> > You mean disabled :)
> > IMNSHO,-Werror should never be enabled by default on any widely used
> > projects.
> Exactly. Exactly what was said by many people the previous times too.
> We only have one warning as error, for missing declarations, because that's 
> really almost always triggering a run-time bug: it really caused a major 
> issues on 64-bits platforms way back when devs used almost exclusively 32-bits 
> day to day.

Agreed. Please don't enable it per default. This just causes unnecessary
work for packagers when builds suddenly break for no good reason.

Sebastian Ramacher

More information about the vlc-devel mailing list