From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Eli Zaretskii Newsgroups: gmane.emacs.devel Subject: Re: PROPOSAL: Move to git, now that bzr is no longer a req. Date: Tue, 07 Jan 2014 21:20:59 +0200 Message-ID: <83siszr42s.fsf@gnu.org> References: <20140102095347.6834E381D0C@snark.thyrsus.com> <87fvp6bdd9.fsf_-_@ktab.red-bean.com> <8761q1ljny.fsf@gmail.com> <20140103175006.GE17261@thyrsus.com> <87ppo6u3mr.fsf@mid.deneb.enyo.de> <874n5i40th.fsf@mid.deneb.enyo.de> <87y52tupp3.fsf@fencepost.gnu.org> <8361pxt7lc.fsf@gnu.org> <87iotvppok.fsf@fencepost.gnu.org> Reply-To: Eli Zaretskii NNTP-Posting-Host: plane.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: QUOTED-PRINTABLE X-Trace: ger.gmane.org 1389122486 28131 80.91.229.3 (7 Jan 2014 19:21:26 GMT) X-Complaints-To: usenet@ger.gmane.org NNTP-Posting-Date: Tue, 7 Jan 2014 19:21:26 +0000 (UTC) Cc: emacs-devel@gnu.org To: David Kastrup Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Tue Jan 07 20:21:32 2014 Return-path: Envelope-to: ged-emacs-devel@m.gmane.org Original-Received: from lists.gnu.org ([208.118.235.17]) by plane.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1W0cDS-0002cH-Fj for ged-emacs-devel@m.gmane.org; Tue, 07 Jan 2014 20:21:30 +0100 Original-Received: from localhost ([::1]:42424 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1W0cDS-0002sJ-1l for ged-emacs-devel@m.gmane.org; Tue, 07 Jan 2014 14:21:30 -0500 Original-Received: from eggs.gnu.org ([2001:4830:134:3::10]:51397) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1W0cDJ-0002sC-ON for emacs-devel@gnu.org; Tue, 07 Jan 2014 14:21:27 -0500 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1W0cDD-00062g-CQ for emacs-devel@gnu.org; Tue, 07 Jan 2014 14:21:21 -0500 Original-Received: from mtaout21.012.net.il ([80.179.55.169]:33242) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1W0cD6-000613-Sz; Tue, 07 Jan 2014 14:21:09 -0500 Original-Received: from conversion-daemon.a-mtaout21.012.net.il by a-mtaout21.012.net.il (HyperSendmail v2007.08) id <0MZ100J00PDS2P00@a-mtaout21.012.net.il>; Tue, 07 Jan 2014 21:21:07 +0200 (IST) Original-Received: from HOME-C4E4A596F7 ([87.69.4.28]) by a-mtaout21.012.net.il (HyperSendmail v2007.08) with ESMTPA id <0MZ100J9APR61W30@a-mtaout21.012.net.il>; Tue, 07 Jan 2014 21:21:07 +0200 (IST) In-reply-to: <87iotvppok.fsf@fencepost.gnu.org> X-012-Sender: halo1@inter.net.il X-detected-operating-system: by eggs.gnu.org: Solaris 10 X-Received-From: 80.179.55.169 X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: "Emacs development discussions." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Original-Sender: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.devel:167664 Archived-At: > From: David Kastrup > Date: Tue, 07 Jan 2014 20:17:15 +0100 >=20 > At any rate, I'm currently just analyzing the code for git-blame, a= nd > for better or worse, it's appallingly bad (quadratic in file size, = times > number of commits). It should be fairly straightforward to bring d= own > the time rather dramatically. Several inner loops are run more tha= n > 2^32=A0times for "git blame src/xdisp.c". >=20 > It's not a design decision, it is just bad programming. Salvageabl= e. > Considering how much life time I spent waiting for some git-gui bla= me (I > conveniently forgot about that, I have to admit) it's sort of amusi= ng > that I never thought of looking at its code (several years ago, > I=A0improved some of the performance-critical but already efficient= parts > of git). This should be rather low-hanging fruit. It would be nice, as for me xdisp.c is a file into whose history I must dig very frequently. Thanks.