[PATCH 0 of 4] detect unnecessary glob lines

Benoit Boissinot bboissin at gmail.com
Tue Feb 12 17:22:58 CST 2013


On Wed, Feb 13, 2013 at 12:05 AM, Simon Heimberg <simohe at besonet.ch> wrote:

> * Patch 4 checks for glob lines not containing any glob character (one of
> /*?).
> * Patch 3 fixes the found lines (before they pop up in test-check-code-hg.t
>   because of patch 4)
> * Patch 2 reports the remaining unnecessary glob lines when running the
> tests
>   on windows. (This is the system with "/" matching for "\". When it
> matches
>   here, the glob is unnecessary.)
> * Patch 1 is a step in the direction of the next patch.
>
> All modified and mentioned tests pass.
>

Looks good (except for some typos) in some patch descriptions. Did you run
the tests on windows?

Cheers,

Benoit
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://selenic.com/pipermail/mercurial-devel/attachments/20130213/d676f7c7/attachment.html>


More information about the Mercurial-devel mailing list