[Lldb-commits] [PATCH] D36977: Add 'break' into GDBRemoteClientBase::SendContinuePacketAndWaitForResponse to fix a warning

Kuba (Brecka) Mracek via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Mon Aug 21 12:55:40 PDT 2017


kubamracek created this revision.

Currently, clang complains here about an non-annotated fall-through.


Repository:
  rL LLVM

https://reviews.llvm.org/D36977

Files:
  source/Plugins/Process/gdb-remote/GDBRemoteClientBase.cpp


Index: source/Plugins/Process/gdb-remote/GDBRemoteClientBase.cpp
===================================================================
--- source/Plugins/Process/gdb-remote/GDBRemoteClientBase.cpp
+++ source/Plugins/Process/gdb-remote/GDBRemoteClientBase.cpp
@@ -60,6 +60,7 @@
         continue;
       if (steady_clock::now() >= m_interrupt_time + kInterruptTimeout)
         return eStateInvalid;
+      break;
     }
     case PacketResult::Success:
       break;


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D36977.112038.patch
Type: text/x-patch
Size: 465 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20170821/20a8db4a/attachment.bin>


More information about the lldb-commits mailing list