Thanks for vigilance. I added a couple more tests in r299729. On Thu, Apr 6, 2017 at 4:06 PM, Jim Ingham via Phabricator < revi...@reviews.llvm.org> wrote:
> jingham added a comment. > > I'm sorry, I don't have time actually review the code here for > correctness... But can you make sure that this also rejects a two or three > field selector, not just "selector:" but "selector:otherField:"? That > seems sufficiently different that you might get the one : but not the two : > form right. You could test 3 & more colons, but at that point it's > probably overkill. > > > Repository: > rL LLVM > > https://reviews.llvm.org/D31451 > > > > -- Thanks, Eugene Zemtsov.
_______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
