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



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

Reply via email to