* bug#1319: Change in where-is-internal causing describe-key problems @ 2008-11-09 3:19 ` Geoff Gole 2008-11-11 9:50 ` martin rudalics 2008-11-15 6:30 ` bug#1319: marked as done (Change in where-is-internal causing describe-key problems) Emacs bug Tracking System 0 siblings, 2 replies; 8+ messages in thread From: Geoff Gole @ 2008-11-09 3:19 UTC (permalink / raw To: emacs-pretest-bug The behaviour of where-is-internal has changed in CVS emacs. Enter view mode, M-: (where-is-internal 'self-insert-command nil nil nil) : 22.2 => ([491520] [458752] [442368] [425984] [409600]...) CVS => ([(128 . 4194303)]) This change breaks describe-function-1, used internally in describe-key, causing a bound key to be printed as a non-ascii character in some conditions. To reproduce the problem: emacs -Q C-h k k Switch to the *Help* buffer C-h k k where-is-internal seems to be buggy, but perhaps describe-function-1 should also be doing more checking. I'm not sure if it should handle input of the form ([(128 . 4194303)]) or not. ^ permalink raw reply [flat|nested] 8+ messages in thread
* bug#1319: Change in where-is-internal causing describe-key problems 2008-11-09 3:19 ` bug#1319: Change in where-is-internal causing describe-key problems Geoff Gole @ 2008-11-11 9:50 ` martin rudalics [not found] ` <f5bc73230811110314i86b57f3la079f8c1d17f8513@mail.gmail.com> 2008-11-15 6:30 ` bug#1319: marked as done (Change in where-is-internal causing describe-key problems) Emacs bug Tracking System 1 sibling, 1 reply; 8+ messages in thread From: martin rudalics @ 2008-11-11 9:50 UTC (permalink / raw To: 1319; +Cc: Geoff Gole > but perhaps describe-function-1 > should also be doing more checking. It chokes at the very last conjunct of (if (and (eq function 'self-insert-command) (eq (key-binding "a") 'self-insert-command) (eq (key-binding "b") 'self-insert-command) (eq (key-binding "c") 'self-insert-command)) because in `help-mode' "c" is bound to `View-leave'. IIUC, the heuristic above should make "many" in the text (princ "It is bound to many ordinary text characters.\n") sound reasonable but apparently fails miserably when one of a, b, or c is bound to something else. martin ^ permalink raw reply [flat|nested] 8+ messages in thread
[parent not found: <f5bc73230811110314i86b57f3la079f8c1d17f8513@mail.gmail.com>]
* bug#1319: Change in where-is-internal causing describe-key problems [not found] ` <f5bc73230811110314i86b57f3la079f8c1d17f8513@mail.gmail.com> @ 2008-11-11 12:40 ` martin rudalics 2008-11-11 14:12 ` Geoff Gole 0 siblings, 1 reply; 8+ messages in thread From: martin rudalics @ 2008-11-11 12:40 UTC (permalink / raw To: Geoff Gole; +Cc: 1319 > I saw that, but I thought it was more likely to be a cheap early > out than a test that was ever meant to be bulletproof. Maybe you were right in thinking so, I don't know. > This code later in the describe-function-1 should handle the case > where that test fails but there are many keys bound to > self-insert-command: > > (when keys > (princ (if remapped ", which is bound to " "It is bound to ")) > ;; If lots of ordinary text characters run this command, > ;; don't mention them one by one. > (if (< (length non-modified-keys) 10) > (princ (mapconcat 'key-description keys ", ")) > (dolist (key non-modified-keys) > (setq keys (delq key keys))) > (if keys > (progn > (princ (mapconcat 'key-description keys ", ")) > (princ ", and many ordinary text characters")) > (princ "many ordinary text characters")))) > > And indeed in 22.2, it does. But the aforementioned change in > where-is-internal breaks it. What precisely was the change in `where-is-internal' that broke it? martin ^ permalink raw reply [flat|nested] 8+ messages in thread
* bug#1319: Change in where-is-internal causing describe-key problems 2008-11-11 12:40 ` martin rudalics @ 2008-11-11 14:12 ` Geoff Gole 2008-11-11 19:05 ` martin rudalics 2008-11-12 3:01 ` Stefan Monnier 0 siblings, 2 replies; 8+ messages in thread From: Geoff Gole @ 2008-11-11 14:12 UTC (permalink / raw To: martin rudalics, 1319 The change in return value described in the original report: > The behaviour of where-is-internal has changed in CVS emacs. Enter > view mode, M-: (where-is-internal 'self-insert-command nil nil nil) : > > 22.2 => ([491520] [458752] [442368] [425984] [409600]...) > CVS => ([(128 . 4194303)]) Unfortunately I don't have the 22.2 source at the moment to compare what differences in the code might cause that. If it would be helpful, I'll try to figure out how to pull them from cvs. ^ permalink raw reply [flat|nested] 8+ messages in thread
* bug#1319: Change in where-is-internal causing describe-key problems 2008-11-11 14:12 ` Geoff Gole @ 2008-11-11 19:05 ` martin rudalics 2008-11-12 3:01 ` Stefan Monnier 1 sibling, 0 replies; 8+ messages in thread From: martin rudalics @ 2008-11-11 19:05 UTC (permalink / raw To: Geoff Gole; +Cc: 1319 >> The behaviour of where-is-internal has changed in CVS emacs. Enter >> view mode, M-: (where-is-internal 'self-insert-command nil nil nil) : >> >> 22.2 => ([491520] [458752] [442368] [425984] [409600]...) >> CVS => ([(128 . 4194303)]) > > Unfortunately I don't have the 22.2 source at the moment to > compare what differences in the code might cause that. If it > would be helpful, I'll try to figure out how to pull them from > cvs. From your previous mail I concluded that you knew the change that caused this. I have no idea what the cons above stands for (some optimization, I presume) so I'm totally lost here. martin ^ permalink raw reply [flat|nested] 8+ messages in thread
* bug#1319: Change in where-is-internal causing describe-key problems 2008-11-11 14:12 ` Geoff Gole 2008-11-11 19:05 ` martin rudalics @ 2008-11-12 3:01 ` Stefan Monnier 2008-11-12 18:32 ` martin rudalics 1 sibling, 1 reply; 8+ messages in thread From: Stefan Monnier @ 2008-11-12 3:01 UTC (permalink / raw To: Geoff Gole; +Cc: 1319 >> 22.2 => ([491520] [458752] [442368] [425984] [409600]...) >> CVS => ([(128 . 4194303)]) The 22.2 answer lists generic chars (these are chars that stand for a whole charset). In Emacs-23, generic chars have been dropped in favor of using char-ranges, represented a a cons cell (START . END). Stefan ^ permalink raw reply [flat|nested] 8+ messages in thread
* bug#1319: Change in where-is-internal causing describe-key problems 2008-11-12 3:01 ` Stefan Monnier @ 2008-11-12 18:32 ` martin rudalics 0 siblings, 0 replies; 8+ messages in thread From: martin rudalics @ 2008-11-12 18:32 UTC (permalink / raw To: Stefan Monnier; +Cc: 1319, Geoff Gole >>> 22.2 => ([491520] [458752] [442368] [425984] [409600]...) >>> CVS => ([(128 . 4194303)]) > > The 22.2 answer lists generic chars (these are chars that stand for > a whole charset). In Emacs-23, generic chars have been dropped in favor > of using char-ranges, represented a a cons cell (START . END). This would deserve a NEWS entry. Anyway, we can either replace (if (and (eq function 'self-insert-command) (eq (key-binding "a") 'self-insert-command) (eq (key-binding "b") 'self-insert-command) (eq (key-binding "c") 'self-insert-command)) (princ "It is bound to many ordinary text characters.\n") ... by something like (if (and (eq function 'self-insert-command) (catch 'plenty (let ((count 0)) (dolist (item (where-is-internal 'self-insert-command)) (when (vectorp item) (let ((range (elt item 0))) (when (consp range) (setq count (+ count (- (cdr range) (car range)))) (when (> count 100) (throw 'plenty count))))))))) (princ "It is bound to many ordinary text characters.\n") ... or drop that "It is bound to many ordinary text characters" stuff. martin, who'd drop it unless someone finds a better solution. ^ permalink raw reply [flat|nested] 8+ messages in thread
* bug#1319: marked as done (Change in where-is-internal causing describe-key problems) 2008-11-09 3:19 ` bug#1319: Change in where-is-internal causing describe-key problems Geoff Gole 2008-11-11 9:50 ` martin rudalics @ 2008-11-15 6:30 ` Emacs bug Tracking System 1 sibling, 0 replies; 8+ messages in thread From: Emacs bug Tracking System @ 2008-11-15 6:30 UTC (permalink / raw To: Chong Yidong [-- Attachment #1: Type: text/plain, Size: 886 bytes --] Your message dated Sat, 15 Nov 2008 01:22:27 -0500 with message-id <87y6zljyos.fsf@cyd.mit.edu> and subject line Re: bug#1319: Change in where-is-internal causing describe-key problems has caused the Emacs bug report #1319, regarding Change in where-is-internal causing describe-key problems to be marked as done. This means that you claim that the problem has been dealt with. If this is not the case it is now your responsibility to reopen the bug report if necessary, and/or fix the problem forthwith. (NB: If you are a system administrator and have no idea what this message is talking about, this may indicate a serious mail system misconfiguration somewhere. Please contact don@donarmstrong.com immediately.) -- 1319: http://emacsbugs.donarmstrong.com/cgi-bin/bugreport.cgi?bug=1319 Emacs Bug Tracking System Contact don@donarmstrong.com with problems [-- Attachment #2: Type: message/rfc822, Size: 3575 bytes --] From: "Geoff Gole" <geoffgole@gmail.com> To: emacs-pretest-bug@gnu.org Subject: Change in where-is-internal causing describe-key problems Date: Sat, 8 Nov 2008 19:19:57 -0800 Message-ID: <f5bc73230811081919k1bed5c01q1ff37c159a7d1a69@mail.gmail.com> The behaviour of where-is-internal has changed in CVS emacs. Enter view mode, M-: (where-is-internal 'self-insert-command nil nil nil) : 22.2 => ([491520] [458752] [442368] [425984] [409600]...) CVS => ([(128 . 4194303)]) This change breaks describe-function-1, used internally in describe-key, causing a bound key to be printed as a non-ascii character in some conditions. To reproduce the problem: emacs -Q C-h k k Switch to the *Help* buffer C-h k k where-is-internal seems to be buggy, but perhaps describe-function-1 should also be doing more checking. I'm not sure if it should handle input of the form ([(128 . 4194303)]) or not. [-- Attachment #3: Type: message/rfc822, Size: 1128 bytes --] From: Chong Yidong <cyd@stupidchicken.com> To: martin rudalics <rudalics@gmx.at> Cc: 1319-done@emacsbugs.donarmstrong.com Subject: Re: bug#1319: Change in where-is-internal causing describe-key problems Date: Sat, 15 Nov 2008 01:22:27 -0500 Message-ID: <87y6zljyos.fsf@cyd.mit.edu> I checked in a slightly simpler fix to describe-function-1. It's probably not necessary to lose too much hair over this part of the code. ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2008-11-15 6:30 UTC | newest] Thread overview: 8+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- [not found] <87y6zljyos.fsf@cyd.mit.edu> 2008-11-09 3:19 ` bug#1319: Change in where-is-internal causing describe-key problems Geoff Gole 2008-11-11 9:50 ` martin rudalics [not found] ` <f5bc73230811110314i86b57f3la079f8c1d17f8513@mail.gmail.com> 2008-11-11 12:40 ` martin rudalics 2008-11-11 14:12 ` Geoff Gole 2008-11-11 19:05 ` martin rudalics 2008-11-12 3:01 ` Stefan Monnier 2008-11-12 18:32 ` martin rudalics 2008-11-15 6:30 ` bug#1319: marked as done (Change in where-is-internal causing describe-key problems) Emacs bug Tracking System
Code repositories for project(s) associated with this external index https://git.savannah.gnu.org/cgit/emacs.git https://git.savannah.gnu.org/cgit/emacs/org-mode.git This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.