Re: [Lldb-commits] [lldb] r341080 - Remove redundant initialization

2018-08-30 Thread Adrian Prantl via lldb-commits
> On Aug 30, 2018, at 9:43 AM, Leonard Mosescu wrote: > > Just curious, what prompted this change? (compiler diagnostic? forcing value > initialization in the member initializer list is harmless in this case) This particular change was (for whatever reason) in the swift.org

Re: [Lldb-commits] [lldb] r341080 - Remove redundant initialization

2018-08-30 Thread Leonard Mosescu via lldb-commits
Just curious, what prompted this change? (compiler diagnostic? forcing value initialization in the member initializer list is harmless in this case) Also, if we want to do this kind of cleanup, m_compiler initialization is also redundant. On Thu, Aug 30, 2018 at 8:39 AM, Adrian Prantl via lldb-co

[Lldb-commits] [lldb] r341080 - Remove redundant initialization

2018-08-30 Thread Adrian Prantl via lldb-commits
Author: adrian Date: Thu Aug 30 08:39:08 2018 New Revision: 341080 URL: http://llvm.org/viewvc/llvm-project?rev=341080&view=rev Log: Remove redundant initialization Modified: lldb/trunk/source/Plugins/ExpressionParser/Clang/ClangExpressionParser.cpp Modified: lldb/trunk/source/Plugins/Expre