[Lldb-commits] [PATCH] D26971: Introduce chrono to more gdb-remote functions

Pavel Labath via lldb-commits lldb-commits at lists.llvm.org
Wed Nov 23 03:22:28 PST 2016


labath updated this revision to Diff 79050.
labath added a comment.

Changes since last version:

- Rename the class to Timeout<>, to reflect the fact that I'd like to use it as a general method for describing timeouts.
- add "using namespace std::chrono" to files with significat chrono usage.

I did not use chrono literals, as those are a c++14 feature. I don't think that
matters much as thanks to the previous import, something like seconds(X) is quite
short already.

Let me know what you think.


https://reviews.llvm.org/D26971

Files:
  source/Plugins/Process/gdb-remote/GDBRemoteClientBase.cpp
  source/Plugins/Process/gdb-remote/GDBRemoteCommunication.cpp
  source/Plugins/Process/gdb-remote/GDBRemoteCommunication.h
  source/Plugins/Process/gdb-remote/GDBRemoteCommunicationClient.cpp
  source/Plugins/Process/gdb-remote/GDBRemoteCommunicationServer.cpp
  source/Plugins/Process/gdb-remote/GDBRemoteCommunicationServer.h
  source/Plugins/Process/gdb-remote/GDBRemoteCommunicationServerLLGS.cpp
  tools/lldb-server/lldb-platform.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D26971.79050.patch
Type: text/x-patch
Size: 19717 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20161123/47511aea/attachment-0001.bin>


More information about the lldb-commits mailing list