diff options
author | Zachary Turner <zturner@google.com> | 2015-04-02 20:57:38 +0000 |
---|---|---|
committer | Zachary Turner <zturner@google.com> | 2015-04-02 20:57:38 +0000 |
commit | 48b475cbaa098095b47d220aa3503a275ee70982 (patch) | |
tree | 549e17d17d08085eb196ffa9fcc2321f5d120a9f /lldb/source/Plugins/Process/Windows/ProcessWindows.cpp | |
parent | d4e1b8685e4c6f9514ffff5f7cd030d4aa4d58a1 (diff) | |
download | bcm5719-llvm-48b475cbaa098095b47d220aa3503a275ee70982.tar.gz bcm5719-llvm-48b475cbaa098095b47d220aa3503a275ee70982.zip |
Fix warnings generated by clang-cl.
There were a couple of real bugs here regarding error checking and
signed/unsigned comparisons, but mostly these were just noise.
There was one class of bugs fixed here which is particularly
annoying, dealing with MSVC's non-standard behavior regarding
the underlying type of enums. See the comment in
lldb-enumerations.h for details. In short, from now on please use
FLAGS_ENUM and FLAGS_ANONYMOUS_ENUM when defining enums which
contain values larger than can fit into a signed integer.
llvm-svn: 233943
Diffstat (limited to 'lldb/source/Plugins/Process/Windows/ProcessWindows.cpp')
-rw-r--r-- | lldb/source/Plugins/Process/Windows/ProcessWindows.cpp | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/lldb/source/Plugins/Process/Windows/ProcessWindows.cpp b/lldb/source/Plugins/Process/Windows/ProcessWindows.cpp index 7770b1470f3..f7cef048dca 100644 --- a/lldb/source/Plugins/Process/Windows/ProcessWindows.cpp +++ b/lldb/source/Plugins/Process/Windows/ProcessWindows.cpp @@ -56,8 +56,8 @@ class ProcessWindowsData { public: ProcessWindowsData(const ProcessLaunchInfo &launch_info) - : m_initial_stop_event(nullptr) - , m_launch_info(launch_info) + : m_launch_info(launch_info) + , m_initial_stop_event(nullptr) , m_initial_stop_received(false) { m_initial_stop_event = ::CreateEvent(nullptr, TRUE, FALSE, nullptr); @@ -335,7 +335,6 @@ ProcessWindows::RefreshStateAfterStop() BreakpointSiteSP site(GetBreakpointSiteList().FindByAddress(pc - 1)); if (site && site->ValidForThisThread(stop_thread.get())) { - lldb::break_id_t break_id = LLDB_INVALID_BREAK_ID; stop_info = StopInfo::CreateStopReasonWithBreakpointSiteID(*stop_thread, site->GetID()); register_context->SetPC(pc - 1); } @@ -394,7 +393,6 @@ void ProcessWindows::DidLaunch() { llvm::sys::ScopedLock lock(m_mutex); - StateType state = GetPrivateState(); // The initial stop won't broadcast the state change event, so account for that here. if (m_session_data && GetPrivateState() == eStateStopped && m_session_data->m_launch_info.GetFlags().Test(eLaunchFlagStopAtEntry)) @@ -558,7 +556,6 @@ ProcessWindows::OnDebugException(bool first_chance, const ExceptionRecord &recor } ExceptionResult result = ExceptionResult::SendToApplication; - lldb::StateType state = GetPrivateState(); switch (record.GetExceptionCode()) { case EXCEPTION_BREAKPOINT: |