git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Ingo Molnar <mingo@elte.hu>
To: Marco Costalba <mcostalba@yahoo.it>
Cc: git@vger.kernel.org
Subject: Re: qgit-0.7
Date: Sun, 10 Jul 2005 12:01:06 +0200	[thread overview]
Message-ID: <20050710100106.GA18875@elte.hu> (raw)
In-Reply-To: <20050709081512.33503.qmail@web26306.mail.ukl.yahoo.com>


* Marco Costalba <mcostalba@yahoo.it> wrote:

> Here is qgit-0.7, a GUI git viewer.
> 
> you can download from:
> 
> http://prdownloads.sourceforge.net/qgit/qgit-0.7.tar.gz?download
> 
> 
> This time a small changelog, but a lot of work ;-)
> 
> - rewrite of graph drawing
> - start-up loading: switch to use git-rev-list --topo-order
> - final fixes to annotation
> - cache of file lists to speed-up loading of file names
> - added color background on heads

the good news: it's really fast now and very usable for browsing 
changes.  Kudos!

the bad news: except for annotations. I started qgit in the current 
kernel GIT repository, and clicked on the following commit:

  5bbcfd9000887c0da7d57cc7b3ac869fc0dd5aa9

then i clicked on sched.c to see the annotated file. Firstly, it took 
roughly 2 minutes (!) for the annotated sched.c to show up. All the qgit 
windows were fully frozen during that time, no refreshes or anything.  
My kernel tree was fully cached in RAM, so it was pure CPU overhead 
(qgit was taking 99% of CPU time). It is clearly not usable in this 
form.

then the annotations were plain wrong. Almost all lines are attributed 
to Tony Luck, while much of the file comes from the initial repository.  
So something's quite fishy here. Also, a number of lines were attributed 
to 'merge', which isnt very informative.

	Ingo

  reply	other threads:[~2005-07-10 10:01 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-07-09  8:15 qgit-0.7 Marco Costalba
2005-07-10 10:01 ` Ingo Molnar [this message]
  -- strict thread matches above, loose matches on Subject: below --
2005-07-09 15:06 qgit-0.7 Marco Costalba
2005-07-10 12:25 qgit-0.7 Marco Costalba
2005-07-10 13:43 ` qgit-0.7 Ingo Molnar

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20050710100106.GA18875@elte.hu \
    --to=mingo@elte.hu \
    --cc=git@vger.kernel.org \
    --cc=mcostalba@yahoo.it \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).