]> git.ipfire.org Git - thirdparty/git.git/commit - dir.c
dir.c: git-status: avoid is_excluded checks for tracked files
authorKarsten Blees <karsten.blees@gmail.com>
Mon, 15 Apr 2013 19:13:35 +0000 (21:13 +0200)
committerJunio C Hamano <gitster@pobox.com>
Mon, 15 Apr 2013 19:34:01 +0000 (12:34 -0700)
commit8aaf8d7728e8ac50cbf6bcad05b6e896d4e69e0b
tree2d5e859c61225f786a0b49432ac97127dde5dd2b
parentb07bc8c8c3c492d657a8bedf04ff939763ea8222
dir.c: git-status: avoid is_excluded checks for tracked files

Checking if a file is in the index is much faster (hashtable lookup) than
checking if the file is excluded (linear search over exclude patterns).

Skip is_excluded checks for files: move the cache_name_exists check from
treat_file to treat_one_path and return early if the file is tracked.

This can safely be done as all other code paths also return path_ignored
for tracked files, and dir_add_ignored skips tracked files as well.

There's just one line left in treat_file, so move this to treat_one_path
as well.

Here's some performance data for git-status from the linux and WebKit
repos (best of 10 runs on a Debian Linux on SSD, core.preloadIndex=true):

       |    status      | status --ignored
       | linux | WebKit | linux | WebKit
-------+-------+--------+-------+---------
before | 0.218 |  1.583 | 0.321 |  2.579
after  | 0.156 |  0.988 | 0.202 |  1.279
gain   | 1.397 |  1.602 | 1.589 |  2.016

Signed-off-by: Karsten Blees <blees@dcon.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
dir.c