search

lldb-tools / lldb-mi

LLDB's machine interface driver
https://lldb.llvm.org
Other
165 stars 56 forks source link

Avoid warnings by initializing fields in the right order #72

Closed mstorsjo closed 3 years ago

mstorsjo commented 3 years ago

This fixes this warning:

../src/MICmnLLDBDebugSessionInfo.cpp:55:7: warning: field 'm_bCreateTty' will be initialized after field 'm_nNextMiStoppointId' [-Wreorder-ctor] m_bCreateTty(false), m_nNextMiStoppointId(1UL) {} ^

tkrasnukha commented 3 years ago

Would be great if you replace the constructor with in-class initialization.

mstorsjo commented 3 years ago

Would be great if you replace the constructor with in-class initialization.

Ok, done

tkrasnukha commented 3 years ago

Please, take a look at the build failure https://travis-ci.org/github/lldb-tools/lldb-mi/jobs/769623045 Clang-format suggests another formatting.