<div dir="ltr"><div>Hi</div><div> </div><div>Attached is a patch to change the warnings for libcxx.</div><div> </div><div>For gcc/clang I've turned the warning about failing to return a value from a non void function into a hard error. I hadn't realised it didn't already do that for g++ when -Wall is on as the cmake file already does.</div>
<div>As a warning it's too easy to miss for returns.</div><div> </div><div>For MSVC I've disabled the warnings about using functions that are "deprecated" by ms for security reasons.</div><div>These are a distraction right now but these warnings could be re-enabled later once other more pressing problems are resolved.</div>
<div> </div><div>I've also revised the patch to revert the decision to disable implicit conversion of  int to bool as Reid pointed out that this wouldn't solve these warnings in system headers for MSVC as it as no system header pragma so the only real way to fix these "problems" is to actually go change the code.</div>
<div> </div><div>I've already submitted patches that remove some of these errors. I'll continue to submit more.</div><div> </div><div>This is changing code like bool b = v & m; to b = (v&m) != 0;</div><div>
 </div><div>Howard please advise if you object to this as I'd like to go ahead and make these changes. I want to get the MSVC error and warning count to near zero and it almost is for /W3 which is the default right now..</div>
<div> </div><div>Please ignore the prior patch I posted relating to cmakelists.txt where I added the /wd4800 option as it's not viable.</div><div> </div><div>BTW, Howard I think buildit might want the -Werror=return-type option too.</div>
</div>