[Lldb-commits] [PATCH] D12025: Make UriParser to support [$HOSTNAME] notation.
Oleksiy Vyalov via lldb-commits
lldb-commits at lists.llvm.org
Thu Aug 20 07:30:48 PDT 2015
ovyalov added inline comments.
Comment at: source/Utility/UriParser.cpp:13
@@ -12,4 +12,3 @@
// C Includes
> I think you wanted to include <string>, not <string.h>
I need string.h for strlen - it's required even with <string> included .
Comment at: source/Utility/UriParser.cpp:52-61
@@ +51,12 @@
+ // Extract hostname
+ if (host_port == '[')
+ // hostname is enclosed with square brackets.
+ pos = host_port.find(']');
+ if (pos == std::string::npos)
+ return false;
+ tmp_hostname = host_port.substr(1, pos - 1);
+ host_port.erase(0, pos + 1);
> I would prefer no to remove the '[' and the ']' characters from the hostname. At the moment you don't use the hostname for the adb protocol so it isn't matter but if we consider IPv6 then having '[' and ']' around the address sounds like a good idea to me (later stages can use this information).
Let me leave  removal as-is since we treat hostname as device_id in case of adb and then use it for port forwarding setup - https://github.com/llvm-mirror/lldb/blob/master/source/Plugins/Platform/Android/PlatformAndroid.cpp#L199
More information about the lldb-commits