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

Pavel Labath via lldb-commits lldb-commits at lists.llvm.org
Tue Nov 22 10:47:15 PST 2016


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

A new version, which uses a helper class which enables all the implicit
conversions that one would normally expect from the duration classes.

Things left TBD:

- name
- where to put it
- whether inheriting from Optional is fine (it is slightly dodgy, but it reduces boilerplate).
- using namespace std::chrono, where it makes sense.


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.78895.patch
Type: text/x-patch
Size: 15759 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20161122/d1b34674/attachment.bin>


More information about the lldb-commits mailing list