<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Dec 12, 2013 at 1:52 PM, Hans Wennborg <span dir="ltr"><<a href="mailto:hans@chromium.org" target="_blank">hans@chromium.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Thu, Dec 12, 2013 at 11:38 AM, Alp Toker <<a href="mailto:alp@nuanti.com">alp@nuanti.com</a>> wrote:<br>

> Reading through SVN history shows that over years any syntax that isn't<br>
> shored up tends to break.<br>
<br>
</div>Yes, I agree we might have relaxed these tests a bit too much.<br>
<div class="im"><br>
>> Requiring -re on directives with regexes makes them easier to grep<br>
>> for, but I don't know how important that is.<br>
><br>
> Looking at the changes today, I'm starting to think anything to make regex<br>
> expressions stand out is a good idea.<br>
<br>
</div>I think they do stand out a bit already with the extra {{'s, but sure,<br>
explicitly putting -re on the directives makes them stand out even<br>
more.<br>
<br>
Anyway, I'm curious to hear what Richard thinks here.</blockquote><div><br></div><div>OK, easy things first: if this is the direction we want, the patch looks good. The [{]... stuff is ugly, but doesn't seem unacceptably so, especially when we have only two affected expect directives in the entire testsuite.</div>
<div><br></div><div>More difficult question: do we want this change? I agree with Alp that the proliferation of overly-broad -re checks is a bad thing, but that seems like a separate issue to me -- I don't think changing the syntax from -re to {{...}} really makes it any easier or harder to write these checks. For me the most interesting argument against this change is that right now it's easy to find the tests that use a regexp, and the most interesting arguments for the change are that they make -verify's interface simpler and more similar to FileCheck. I find the latter to be a stronger argument.</div>
</div></div></div>