From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Bob Rogers Newsgroups: gmane.emacs.devel Subject: Re: C-x v l does not move to current log entry Date: Fri, 3 Aug 2007 23:08:33 -0400 Message-ID: <18099.60849.120058.919034@rgrjr.dyndns.org> References: <46B03B75.4000008@swipnet.se> NNTP-Posting-Host: lo.gmane.org Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="aPqeXV8Jxn" Content-Transfer-Encoding: 7bit X-Trace: sea.gmane.org 1186196949 23055 80.91.229.12 (4 Aug 2007 03:09:09 GMT) X-Complaints-To: usenet@sea.gmane.org NNTP-Posting-Date: Sat, 4 Aug 2007 03:09:09 +0000 (UTC) Cc: esr@snark.thyrsus.com, emacs-devel@gnu.org To: rms@gnu.org Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Sat Aug 04 05:09:06 2007 Return-path: Envelope-to: ged-emacs-devel@m.gmane.org Original-Received: from lists.gnu.org ([199.232.76.165]) by lo.gmane.org with esmtp (Exim 4.50) id 1IHA0m-00019H-FN for ged-emacs-devel@m.gmane.org; Sat, 04 Aug 2007 05:09:05 +0200 Original-Received: from localhost ([127.0.0.1] helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1IHA0j-0005rp-Rv for ged-emacs-devel@m.gmane.org; Fri, 03 Aug 2007 23:09:01 -0400 Original-Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1IHA0g-0005ra-VS for emacs-devel@gnu.org; Fri, 03 Aug 2007 23:08:58 -0400 Original-Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1IHA0b-0005a2-5P for emacs-devel@gnu.org; Fri, 03 Aug 2007 23:08:58 -0400 Original-Received: from [199.232.76.173] (helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1IHA0b-0005Zk-0c for emacs-devel@gnu.org; Fri, 03 Aug 2007 23:08:53 -0400 Original-Received: from c-24-34-111-252.hsd1.ma.comcast.net ([24.34.111.252] helo=rgrjr.dyndns.org) by monty-python.gnu.org with smtp (Exim 4.60) (envelope-from ) id 1IHA0Y-000429-27 for emacs-devel@gnu.org; Fri, 03 Aug 2007 23:08:51 -0400 Original-Received: (qmail 16485 invoked by uid 500); 4 Aug 2007 03:08:33 -0000 In-Reply-To: X-Mailer: VM 7.19 under Emacs 22.1.50.1 X-Detected-Kernel: Linux 2.4-2.6 X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "Emacs development discussions." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Original-Sender: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.devel:76024 Archived-At: --aPqeXV8Jxn Content-Type: text/plain; charset=iso-8859-1 Content-Description: message body text Content-Transfer-Encoding: 8bit From: Richard Stallman Date: Thu, 02 Aug 2007 11:45:17 -0400 Would someone please DTRT and ack? Date: Wed, 01 Aug 2007 09:51:17 +0200 From: Jan Djärv % emacs -Q src/gmalloc.c C-x v l The cursor stays at the top of the log view buffer. Contrast this to emacs 22.1 where the cursor is positioned to the current log entry for the revision of src/gmalloc.c that you are editing. Hmm. vc-default-show-log-entry does appear to be going to the right place, but something is messing it up afterwards. I think I'll go to bed now and let somebody who knows what they're doing have a crack at it. ;-} This from *Messages* may be relevant: Running cvs log gmalloc.c in the background... done Loading log-view... Loading easy-mmode...done Loading log-view...done Error during redisplay: (void-function ,log-view-file-re) [2 times] This turns out to be coming from log-view-font-lock-keywords, which has a value that looks something like this: ((eval \` (\,log-view-file-re (1 ...) (0 log-view-file-face append))) (eval \` (\,log-view-message-re . log-view-message-face))) The source (log-view.el:170) specifies backquoted forms inside a quoted form; it appears that the reader is not handling this correctly. The attached patch replaces the backquotes with explicit "cons" calls; this is not a real fix, and I include it only to show that it does make the error messages go away. Fixing the reader is undoubtedly hairier. However, fixing the "Error during redisplay" problem does not affect the original symptom. Hope this is of some use, -- Bob Rogers http://rgrjr.dyndns.org/ --aPqeXV8Jxn Content-Type: text/plain Content-Description: Content-Disposition: inline; filename="no-bq-in-log-view-flk-1.patch" Content-Transfer-Encoding: 7bit Index: lisp/log-view.el =================================================================== RCS file: /sources/emacs/emacs/lisp/log-view.el,v retrieving revision 1.37 diff -c -r1.37 log-view.el *** lisp/log-view.el 23 Jun 2007 03:06:21 -0000 1.37 --- lisp/log-view.el 4 Aug 2007 02:41:52 -0000 *************** *** 170,179 **** (defvar log-view-font-lock-keywords ;; We use `eval' so as to use the buffer-local value of log-view-file-re ;; and log-view-message-re, if applicable. ! '((eval . `(,log-view-file-re ! (1 (if (boundp 'cvs-filename-face) cvs-filename-face)) ! (0 log-view-file-face append))) ! (eval . `(,log-view-message-re . log-view-message-face)))) (defconst log-view-font-lock-defaults '(log-view-font-lock-keywords t nil nil nil)) --- 170,180 ---- (defvar log-view-font-lock-keywords ;; We use `eval' so as to use the buffer-local value of log-view-file-re ;; and log-view-message-re, if applicable. ! '((eval . (cons log-view-file-re ! '((1 (if (boundp 'cvs-filename-face) cvs-filename-face)) ! (0 log-view-file-face append)))) ! (eval . (cons log-view-message-re ! 'log-view-message-face)))) (defconst log-view-font-lock-defaults '(log-view-font-lock-keywords t nil nil nil)) --aPqeXV8Jxn Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Emacs-devel mailing list Emacs-devel@gnu.org http://lists.gnu.org/mailman/listinfo/emacs-devel --aPqeXV8Jxn--