From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: David Kastrup Newsgroups: gmane.emacs.devel Subject: Re: Progress report on git-blame Date: Sat, 25 Jan 2014 19:59:50 +0100 Organization: Organization?!? Message-ID: <87r47vc2ft.fsf@fencepost.gnu.org> References: <20140109140226.57D6C38085A@snark.thyrsus.com> <20140110155121.GA8178@thyrsus.com> <20140111205925.GC17111@thyrsus.com> <87y52mdoha.fsf@fencepost.gnu.org> <87fvoceuos.fsf_-_@fencepost.gnu.org> <83d2jgcy5z.fsf@gnu.org> <87vbx8cu8a.fsf@fencepost.gnu.org> <87eh3v7wiq.fsf@building.gnus.org> <8738kb3ndx.fsf@wanadoo.es> NNTP-Posting-Host: plane.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: 8bit X-Trace: ger.gmane.org 1390676410 30466 80.91.229.3 (25 Jan 2014 19:00:10 GMT) X-Complaints-To: usenet@ger.gmane.org NNTP-Posting-Date: Sat, 25 Jan 2014 19:00:10 +0000 (UTC) To: emacs-devel@gnu.org Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Sat Jan 25 20:00:17 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 1W78Sm-0002MU-TQ for ged-emacs-devel@m.gmane.org; Sat, 25 Jan 2014 20:00:17 +0100 Original-Received: from localhost ([::1]:52194 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1W78Sm-00009q-L9 for ged-emacs-devel@m.gmane.org; Sat, 25 Jan 2014 14:00:16 -0500 Original-Received: from eggs.gnu.org ([2001:4830:134:3::10]:34916) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1W78Se-00008o-PN for emacs-devel@gnu.org; Sat, 25 Jan 2014 14:00:14 -0500 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1W78SY-0004o1-I0 for emacs-devel@gnu.org; Sat, 25 Jan 2014 14:00:08 -0500 Original-Received: from plane.gmane.org ([80.91.229.3]:52958) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1W78SY-0004mR-CS for emacs-devel@gnu.org; Sat, 25 Jan 2014 14:00:02 -0500 Original-Received: from list by plane.gmane.org with local (Exim 4.69) (envelope-from ) id 1W78SX-0002HA-IP for emacs-devel@gnu.org; Sat, 25 Jan 2014 20:00:01 +0100 Original-Received: from x2f416b0.dyn.telefonica.de ([2.244.22.176]) by main.gmane.org with esmtp (Gmexim 0.1 (Debian)) id 1AlnuQ-0007hv-00 for ; Sat, 25 Jan 2014 20:00:01 +0100 Original-Received: from dak by x2f416b0.dyn.telefonica.de with local (Gmexim 0.1 (Debian)) id 1AlnuQ-0007hv-00 for ; Sat, 25 Jan 2014 20:00:01 +0100 X-Injected-Via-Gmane: http://gmane.org/ Original-Lines: 41 Original-X-Complaints-To: usenet@ger.gmane.org X-Gmane-NNTP-Posting-Host: x2f416b0.dyn.telefonica.de X-Face: 2FEFf>]>q>2iw=B6, xrUubRI>pR&Ml9=ao@P@i)L:\urd*t9M~y1^:+Y]'C0~{mAl`oQuAl \!3KEIp?*w`|bL5qr,H)LFO6Q=qx~iH4DN; i"; /yuIsqbLLCh/!U#X[S~(5eZ41to5f%E@'ELIi$t^ Vc\LWP@J5p^rst0+('>Er0=^1{]M9!p?&:\z]|;&=NP3AhB!B_bi^]Pfkw User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.3.50 (gnu/linux) Cancel-Lock: sha1:Q0zWzphLY+X2eETy/5K1f+n41jY= X-detected-operating-system: by eggs.gnu.org: Genre and OS details not recognized. X-Received-From: 80.91.229.3 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:169074 Archived-At: Óscar Fuentes writes: > Some random data points: > > $ time git blame -L 1000,+100 -- src/xdisp.c > /dev/null > > real 0m11.034s > user 0m10.417s > sys 0m0.599s > > > $ time git blame -L 1000,+1000 -- src/xdisp.c > /dev/null > > real 0m15.089s > user 0m14.297s > sys 0m0.767s > > > $ time git blame -L 1000,+10000 -- src/xdisp.c > /dev/null > > real 1m5.144s > user 1m3.555s > sys 0m1.502s > > > It seems that there is not a lot of a difference among blaming 20 lines > and blaming 1000. It very much depends on _which_ lines. Basically, lines forming a block sticking together when doing a unified diff are not all that expensive since they travel in a single data structure. The worst are lots of small changes introduced in different branches. > AFAIK current `blame' functionality on VC (and on git-blame.el) is > based on working on the whole file, Sure? If so, it maybe should at least be made to support git blame --incremental usefully. -- David Kastrup