[Lldb-commits] [PATCH] D103483: [lldb] Convert the default constructor’s member initializers into default member initializers

[Raphael Isemann via Phabricator via lldb-commits]

teemperor added a comment.

+1. I'm anyway doing the same whenever I have to touching constructors, so we 
might as well pull out the big hammer.

I'm also glad you volunteer to resolve all the downstream conflicts :)



================
Comment at: lldb/include/lldb/Breakpoint/BreakpointOptions.h:47
     CommandData()
-        : user_source(), script_source(),
-          interpreter(lldb::eScriptLanguageNone), stop_on_error(true) {}
+        : user_source(), script_source() {}
 
----------------
If we anyway touch this, could we also delete these default member initializers?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D103483/new/

https://reviews.llvm.org/D103483

_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits