<div dir="ltr">It's too bad llvm doesn't have an equivalent of boost::any, because that would be perfect here :-/</div><br><div class="gmail_quote"><div dir="ltr">On Tue, May 10, 2016 at 12:39 PM Greg Clayton <<a href="mailto:clayborg@gmail.com">clayborg@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">clayborg requested changes to this revision.<br>
clayborg added a comment.<br>
This revision now requires changes to proceed.<br>
<br>
Just get rid of the extra typedef as specified in inline comments and this is good to go.<br>
<br>
<br>
================<br>
Comment at: include/lldb/Host/HostProcess.h:41<br>
@@ -41,1 +40,3 @@<br>
+public:<br>
+    typedef Host::MonitorChildProcessCallback MonitorCallback;<br>
<br>
----------------<br>
We can probably get rid of this and just use Host::MonitorChildProcessCallback<br>
<br>
================<br>
Comment at: include/lldb/Host/HostProcess.h:54<br>
@@ -53,1 +53,3 @@<br>
+    HostThread<br>
+    StartMonitoring(const MonitorCallback &callback, bool monitor_signals);<br>
<br>
----------------<br>
Use Host::MonitorChildProcessCallback directly.<br>
<br>
================<br>
Comment at: include/lldb/Host/posix/HostProcessPosix.h:43<br>
@@ -43,1 +42,3 @@<br>
+    HostThread<br>
+    StartMonitoring(const HostProcess::MonitorCallback &callback, bool monitor_signals) override;<br>
 };<br>
----------------<br>
Use Host::MonitorChildProcessCallback directly<br>
<br>
<br>
<a href="http://reviews.llvm.org/D20106" rel="noreferrer" target="_blank">http://reviews.llvm.org/D20106</a><br>
<br>
<br>
<br>
</blockquote></div>