[PATCH 3 of 6 STABLE V3] dirstate: use util.normcase() instead of str.lower() or os.path.normcase()

Matt Mackall mpm at selenic.com
Tue Dec 13 14:49:26 CST 2011


On Mon, 2011-12-12 at 22:25 +0900, FUJIWARA Katsunori wrote:
> # HG changeset patch
> # User FUJIWARA Katsunori <foozy at lares.dti.ne.jp>
> # Date 1323677419 -32400
> # Branch stable
> # Node ID d7229f60ad8aacd657d780935235a84048413dc6
> # Parent  7ef5cd9f4a5501388f73efb1f225c52e87acef76
> dirstate: use util.normcase() instead of str.lower() or os.path.normcase()

I really need you to take two concepts to heart:

- one change at a time
- obvious changes first

If you keep sending me complex patches with tricky and obvious changes
mixed together, we'll never get anywhere.

> 1. put '.' into foldmap of dirstate to prevent useless util.fspath()
>    invocation:

Here's a rule of thumb: "if you need a numbered list of changes, then
you probably have more than one change in your patch."

-- 
Mathematics is the supreme nostalgia of our time.




More information about the Mercurial-devel mailing list