[Lldb-commits] [PATCH] D76736: [LLDB] Fix parsing of IPv6 host:port inside brackets

Pavel Labath via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Wed Mar 25 02:08:20 PDT 2020


labath accepted this revision.
labath added a comment.
This revision is now accepted and ready to land.

Looks good. You already mention that in the commit message, but it may be nice to also mention somewhere near the test case that these kinds of "URL"s can occur when and connecting to and android device over IP (in which case the port is part of the "hostname"). So one isn't left wondering why are we testing such "nonsensical" hostnames...

Also, do you have commit access?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D76736/new/

https://reviews.llvm.org/D76736





More information about the lldb-commits mailing list