tberghammer added a comment.

The implementation looks good, but I don't like the approach you try to handle 
the problem.

I think you do far too much work to use UriParser in a case where it isn't 
necessary and isn't make things easier. I think a better (and definitely 
simpler) solution would be to change only ConnectionFileDescriptorPosix.cpp to 
parse the adb addresses with a locally specified regexp or with looking for the 
last colon in the address and split the address to host and port based on that.


http://reviews.llvm.org/D12025



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to