[Lldb-commits] [lldb] r169633 - in /lldb/trunk: source/ source/API/ source/Core/ source/Host/common/ source/Interpreter/ source/Plugins/ABI/MacOSX-i386/ source/Plugins/DynamicLoader/Darwin-Kernel/ source/Plugins/DynamicLoader/MacOSX-DYLD/ source/Plugi

[Malea, Daniel]

Hi Andy,

On 2012-12-07, at 8:09 PM, Kaylor, Andrew wrote:

> Will the current build settings produce an error for an uncovered switch case with no default?  
> 

No, -Werror is not enabled on any LLDB builds for now. I think eventually enabling that flag might be an option, but seeing as there's a few warnings on the clang-mac and gcc-linux builds at the moment, I don't think it's possible to enable it right now.

> If not, this patch makes it pretty important for us to keep the build free of warnings so we don't miss new uncovered cases appearing.
> 

Agreed. Now that we have a clean (clang-linux) build, new warnings should stand out like a sore thumb. I'll be happy to review/commit any warnings fixes that come my way!

Cheers,
Dan