<div dir="ltr"><div class="gmail_default" style="font-family:verdana,sans-serif">Thanks for vigilance. I added a couple more tests in r299729.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Apr 6, 2017 at 4:06 PM, Jim Ingham via Phabricator <span dir="ltr"><<a href="mailto:reviews@reviews.llvm.org" target="_blank">reviews@reviews.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">jingham added a comment.<br>
<br>
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.<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D31451" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D31451</a><br>
<br>
<br>
<br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><font face="verdana, sans-serif">Thanks,</font><div><font face="verdana, sans-serif">Eugene Zemtsov.</font></div></div></div>
</div>