> We have a Clang warning for this - I would like to know why that didn't catch it? Are we not building this with -Wreturn-type?
Currently we don't compile our tests with any warning flags. I'm not sure if it was just too noisy but looking at other defects Coverity found its clear we need to enable at least some of them. There are a bunch of assignments that should be comparisons as well. /Eric On Sat, Aug 23, 2014 at 1:33 AM, Chandler Carruth <[email protected]> wrote: > > On Fri, Aug 22, 2014 at 11:36 PM, Eric Fiselier <[email protected]> wrote: > >> Add return statement to a test class's assignment operator. Defect found >> by Coverity Scan. >> > > We have a Clang warning for this - I would like to know why that didn't > catch it? Are we not building this with -Wreturn-type? >
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
