<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Feb 13, 2013 at 12:05 AM, Simon Heimberg <span dir="ltr"><<a href="mailto:simohe@besonet.ch" target="_blank">simohe@besonet.ch</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">* Patch 4 checks for glob lines not containing any glob character (one of /*?).<br>


* Patch 3 fixes the found lines (before they pop up in test-check-code-hg.t<br>
  because of patch 4)<br>
* Patch 2 reports the remaining unnecessary glob lines when running the tests<br>
  on windows. (This is the system with "/" matching for "\". When it matches<br>
  here, the glob is unnecessary.)<br>
* Patch 1 is a step in the direction of the next patch.<br>
<br>
All modified and mentioned tests pass.<br></blockquote><div><br></div><div>Looks good (except for some typos) in some patch descriptions. Did you run the tests on windows?</div><div><br></div><div style>Cheers,</div><div style>

<br></div><div style>Benoit</div></div></div></div>