git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: Big performance regression with --no-color option in git-log

From
Pierre Habouzit <madcoder@debian.org>
Date
Sep 29, 2007, 11:01 UTC
Message-ID
<20070929110130.GC4216@artemis.corp>
In-Reply-To
<e5bfff550709290302h6016c6e8mf9d5dfddce07d985@mail.gmail.com>
On Sat, Sep 29, 2007 at 10:02:23AM +0000, Marco Costalba wrote:
> On today git tree
hmmm interesting... Here is what happens with --no-color:
    38708    33.2920  diff_tree
    23908    20.5628  base_name_compare
    18478    15.8926  decode_tree_entry
    8310      7.1473  find_pack_entry_one
    5654      4.8629  update_tree_entry
    2748      2.3635  pretty_print_commit
    1838      1.5808  get_one_line
    1460      1.2557  .plt
and wihtout it:
    2594     18.0893  pretty_print_commit
    1887     13.1590  get_one_line
    1574     10.9763  find_pack_entry_one
    967       6.7434  get_short_sha1
    892       6.2204  parse_commit_buffer
    863       6.0181  lookup_object
    595       4.1492  strbuf_grow
    440       3.0683  insert_obj_hash
Why on earth does --no-color has such a different code path ...
-- 
·O·  Pierre Habouzit
··O                                                madcoder@debian.org
OOO                                                http://www.madism.org
Previous: Marco CostalbaNext: Pierre Habouzit
Message 2 of 12 in “Big performance regression with --no-color option in git-log”
  1. Marco CostalbaSep 29, 2007
  2. Pierre HabouzitSep 29, 2007
  3. Pierre HabouzitSep 29, 2007
  4. Marco CostalbaSep 29, 2007
  5. 1/1 --color and --no-color git-log options don't need diffs to be computed.Pierre Habouzit, Sep 29, 2007
  6. David KastrupSep 29, 2007
  7. Pierre HabouzitSep 29, 2007
  8. Marco CostalbaSep 29, 2007
  9. Marco CostalbaSep 29, 2007
  10. Fix revision log diff setup, avoid unnecessary diff generationLinus Torvalds, Sep 29, 2007
  11. Junio C HamanoSep 29, 2007
  12. Junio C HamanoSep 29, 2007

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.