emacs-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Last commit to mouse.el has a bug


From: CHENG Gao
Subject: Last commit to mouse.el has a bug
Date: Thu, 28 Oct 2004 21:50:36 +0800
User-agent: Gnus/5.110003 (No Gnus v0.3) Emacs/21.3.50 (windows-nt)

,----
| CVSROOT:      /cvsroot/emacs
| Module name:  emacs
| Branch:       
| Changes by:   Richard M. Stallman <address@hidden>    04/10/27 17:44:59
| 
| Modified files:
|       lisp           : mouse.el 
| 
| Log message:
|       (mouse-show-mark): Do most processing the same
|       regardless of transient-mark-mode.
| 
| CVSWeb URLs:
| 
http://savannah.gnu.org/cgi-bin/viewcvs/emacs/emacs/lisp/mouse.el.diff?tr1=1.251&tr2=1.252&r1=text&r2=text
`----
It's latest commit of mouse.el by RMS.

I found it has a bug.

,----
| (let ((inhibit-quit t)
|        (echo-keystrokes 0)
|         event events key ignore
|        (x-lost-selection-hooks (copy-sequence
|        x-lost-selection-hooks)))...
`----

(let (x-lost-selection-hooks (copy-sequence x-lost-selection-hooks)))
<-- Bug here

At this time, x-lost-selection-hooks is not set, so copy-sequence will
fail.

You can try to use mouse to select a region, and you'll get error
message as:

"mouse-drag-region-1: Symbol's value as variable is void:
x-lost-selection-hooks"

Just revert it to

 (let ((inhibit-quit t)
        (echo-keystrokes 0)
        event events key ignore
        x-lost-selection-hooks )

fixes this bug.

-- 
"When the government fears the people, you have liberty.
 When the people fear the government, you have tyranny."
                --Thomas Jefferson






reply via email to

[Prev in Thread] Current Thread [Next in Thread]