labath added a subscriber: labath.
labath added a comment.
I think this would be better if you could specify the connection direction in
the url. Also, a test for this functionality would be great.
================
Comment at: source/Interpreter/CommandInterpreter.cpp:638
@@ +637,3 @@
+ connect_gdb_remote_cmd_ap->AddRegexCommand("^([^:]+:[[:digit:]]+)
reverse$",
+ "process connect
--plugin gdb-remote connect://%1 reverse") &&
+ connect_gdb_remote_cmd_ap->AddRegexCommand("^([[:digit:]]+)
reverse$",
----------------
how about just using `listen://...` to mean reverse connection?
================
Comment at: source/Plugins/Platform/gdb-server/PlatformRemoteGDBServer.h:222
@@ -226,1 +221,3 @@
+ ConnectProcess(const char *connect_url, const char *plugin_name,
lldb_private::Debugger &debugger,
+ lldb_private::Target *target, lldb_private::Error &error,
bool reverse = false) override;
----------------
I don't think we need to pass this `reverse=false` around everywhere. We
already support `listen://host:port` url scheme (see e.g.
ConnectionFileDescriptor::Connect`). Could you use that to specify connection
direction?
http://reviews.llvm.org/D17099
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
