[vlc-devel] commit: Don't treat strict-aliasing as error, this?break Win32 compilation too much (Jean-Baptiste Kempf )

Rémi Denis-Courmont remi at remlab.net
Tue Aug 25 21:21:18 CEST 2009


Le mardi 25 août 2009 22:14:43 Jean-Baptiste Kempf, vous avez écrit :
> On Tue, Aug 25, 2009 at 10:13:11PM +0300, Rémi Denis-Courmont wrote :
> > Le mardi 25 août 2009 21:55:32 Jean-Baptiste Kempf, vous avez écrit :
> > > Breaking other platforms compilation without even trying to fix them
> > > before is not nice. VLC used to be cross-platform, but deliberating
> > > breaking various platform will not help...
> >
> > Nobody forced anybody to turn Werror on.
>
> It is by default on on release

AFAIK, it is enabled by default only in debug builds, and you can override it 
anyway.

> and on Win32 configure scripts.

I dislike errors on harmless warnings such as unused variable. But, I agree 
with Pierre that harmful-and-always-fixable warnings should be error. It's 
just that it was not selectively possible in earlier GCC versions to my 
knowledge.

Besides, I refuse to stop or slow down VLC development just because one damn 
platform has no developers. If you want a "portable" with more bugs and fewer 
features, you can revive the 0.8.6 branch or fork.

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



More information about the vlc-devel mailing list