If we want to enable warnings and use static analysis tools these sort of errors simply need to be fixed to prevent noise.
/Eric On Aug 23, 2014 7:06 PM, "Chandler Carruth" <[email protected]> wrote: > > On Sat, Aug 23, 2014 at 6:01 PM, Jonathan Roelofs < > [email protected]> wrote: > >> Wait a second, this test is *supposed* to be a compile failure, but not >> for the reason I originally thought. It's testing that unique_ptr requires >> the deleter have a public default constructor. Because it fails to >> compile, '-Wall -Wextra -pedantic -Werror' won't help us here :( >> > > Hah. > > I don't think we should spend our time fixing warnings (coverity or > otherwise) is code that is supposed to produce compile errors... seems a > waste. > > _______________________________________________ > cfe-commits mailing list > [email protected] > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits > >
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
