<div dir="ltr"><div class="gmail_extra">Oh, sorry, I wasn't aware that we were testing the generic infrastructure elsewhere (it seems like wildcards.s? Sorry, it is a bit hard to grep for).</div><div class="gmail_extra">I guess I'm confused, because we have version-script-complex-wildcards.s and version-script-extern-wildcards.s that seem to be trying to test different globbing cases too. Are those also covered by other tests? Neither of those seems to test that `*` matches zero or multiple characters though.</div><div class="gmail_extra"><br></div><div class="gmail_extra">It seems like an organization like the following might be useful:</div><div class="gmail_extra">- version-script-extern.s: checks that we match against the demangled name for extern "C++"</div><div class="gmail_extra">- version-script-glob.s: checks all supported matching constructs (or is just a basic sanity check with a comment pointing to a file with more thorough generic pattern matching tests)</div>







<div class="gmail_extra"><br></div><div class="gmail_extra">-- Sean Silva</div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Sep 8, 2016 at 12:28 PM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="gmail-">On 8 September 2016 at 15:04, Sean Silva <<a href="mailto:chisophugis@gmail.com">chisophugis@gmail.com</a>> wrote:<br>
> The glob `*` can match zero, one, or many characters. Can you make sure that<br>
> the test covers those cases?<br>
><br>
> Currently this test only verifies that it matches a single character, and<br>
> the test would pass if we treated `foo*` as `foo?`.<br>
<br>
</span>True, but the patch used the existing matching infrastructure that is<br>
tested elsewhere. Do you thing expanding the test would add much<br>
value?<br>
<br>
Cheers,<br>
Rafael<br>
</blockquote></div><br></div></div>