Optimize dirstate walking This generally cuts the time for hg status/diff in half, from 2s down to 1s. The main parts I'm trying to optimize are: 1) os.walk stats every file. dirstate.changes then stats every file again. 2) os.walk yields every file and subdir to dirstate.traverse who yields every file and everything in the dirstate map. dirstate.walk then filters this mass and yields every file to the caller. There should be fewer steps in here, and fewer duplicate strings yielded. 3) dirstate.walk runs util.unique on the results from dirstate.traverse, even though it is also passing things through dirstate.seen to look for duplicates. I've turned os.walk into something hg specific that takes all the dirstate ignore and matching rules into account. The new function also takes an function arg (statmatch()) the caller supplies to help filter out files it doesn't care about. dirstate.changes uses this to update state for each file, avoiding the second stat call. dirstate.walk is changed to turn the match function it is passed into a statmatch function. The only real difference is that a statmatch function takes the stat data as a second parameter. It now calls dirstate.walkhelper, who requires a statmatch function to be passed. This fails test-walk, but right now I think this is from a sorting error fixed by this patch. Index: crew/mercurial/dirstate.py ===================================================================

     1

Andrea Arcangeli <andrea at suse.de>

     2

Thomas Arendsen Hein <thomas at intevation.de>

     3

Goffredo Baroncelli <kreijack at libero.it>

     4

Muli Ben-Yehuda <mulix at mulix.org>

     5

Mikael Berthe <mikael at lilotux.net>

     6

Vincent Danjean <vdanjean.ml at free.fr>

     7

Jake Edge <jake at edge2.net>

     8

Michael Fetterman <michael.fetterman at intel.com>

     9

Edouard Gomez <ed.gomez at free.fr>

    10

Alecs King <alecsk at gmail.com>

    11

Vadim Lebedev <vadim at mbdsys.com>

    12

Christopher Li <hg at chrisli.org>

    13

Chris Mason <mason at suse.com>

    14

Wojciech Milkowski <wmilkowski at interia.pl>

    15

Chad Netzer <chad.netzer at gmail.com>

    16

Bryan O'Sullivan <bos at serpentine.com>

    17

Vicent Seguí Pascual <vseguip at gmail.com>

    18

Sean Perry <shaleh at speakeasy.net>

    19

Nguyen Anh Quynh <aquynh at gmail.com>

    20

Arun Sharma <arun at sharma-home.net>

    21

Josef "Jeff" Sipek <jeffpc@optonline.net>

    22

TK Soh <teekaysoh@yahoo.com>

    23

Kevin Smith <yarcs at qualitycode.com>

    24

Radoslaw Szkodzinski <astralstorm at gorzow.mm.pl>

    25

Samuel Tardieu <sam at rfc1149.net>

    26

K Thananchayan <thananck at yahoo.com>

    27

Andrew Thompson <andrewkt at aktzero.com>

    28

Michael S. Tsirkin <mst at mellanox.co.il>

    29

Rafael Villar Burke <pachi at mmn-arquitectos.com>

    30

Tristan Wibberley <tristan at wibberley.org>

    31

Mark Williamson <mark.williamson at cl.cam.ac.uk>

    32

    33

If you are a contributor and don't see your name here, please let me know.