[PATCH 4 of 5] normcase: for darwin, use fast ASCII lower

Siddharth Agarwal sid0 at fb.com
Fri Oct 3 22:23:26 CDT 2014


# HG changeset patch
# User Siddharth Agarwal <sid0 at fb.com>
# Date 1412387248 25200
#      Fri Oct 03 18:47:28 2014 -0700
# Node ID ceee117160a138ef2896111ae699fc74d9048088
# Parent  5b9344d9025ff4325805b87ff53e3656778193f6
normcase: for darwin, use fast ASCII lower

Constructing the foldmap is much faster on OS X now. For a large real-world repo:

before: wall 0.805278 comb 0.800000 user 0.790000 sys 0.010000 (best of 13)
after:  wall 0.407404 comb 0.410000 user 0.390000 sys 0.020000 (best of 25)

This is a nice boost to 'hg status', especially with the third-party hgwatchman
extension enabled (which eliminates stat calls). For the above repo, 'hg
status' goes from 1.17 seconds to 0.75.

diff --git a/mercurial/posix.py b/mercurial/posix.py
--- a/mercurial/posix.py
+++ b/mercurial/posix.py
@@ -217,8 +217,7 @@
         '''
 
         try:
-            path.decode('ascii') # throw exception for non-ASCII character
-            return path.lower()
+            return encoding.asciilower(path)  # exception for non-ASCII
         except UnicodeDecodeError:
             pass
         try:


More information about the Mercurial-devel mailing list