unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#6224: 23.1; last-command docstring "multiple displays" info node link
@ 2010-05-19 23:39 Kevin Ryde
  2010-05-20  0:21 ` Juri Linkov
  0 siblings, 1 reply; 8+ messages in thread
From: Kevin Ryde @ 2010-05-19 23:39 UTC (permalink / raw)
  To: 6224

[-- Attachment #1: Type: text/plain, Size: 532 bytes --]

In the C-h v docstring of `last-command', pressing Ret on the link

    See Info node `(elisp)Multiple displays'.

gets an error

    No such node or anchor: Multiple displays

I suppose it's a typo for "Multiple Displays", in five spots altogether
in keyboard.c.  (Search and replace if the diff is out of date.)

2010-05-19  Kevin Ryde  <user42@zip.com.au>

	* src/keyboard.c (last-command, keyboard-translate-table,
	overriding-terminal-local-map, system-key-alist,
	local-function-key-map): Docstring typo "Multiple Displays".


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: keyboard.c.info-multiple-displays.diff --]
[-- Type: text/x-diff, Size: 2001 bytes --]

--- keyboard.c.~1.1026.~	2009-12-27 08:05:18.000000000 +1100
+++ keyboard.c	2010-05-20 09:33:41.000000000 +1000
@@ -12087,7 +12087,7 @@
 was a kill command.
 
 `last-command' has a separate binding for each terminal device.
-See Info node `(elisp)Multiple displays'.  */);
+See Info node `(elisp)Multiple Displays'.  */);
 
   DEFVAR_KBOARD ("real-last-command", Vreal_last_command,
 		 doc: /* Same as `last-command', but never altered by Lisp code.  */);
@@ -12226,7 +12226,7 @@
 output.  See also `translation-table-for-input'.
 
 This variable has a separate binding for each terminal.  See Info node
-`(elisp)Multiple displays'.  */);
+`(elisp)Multiple Displays'.  */);
 
   DEFVAR_BOOL ("cannot-suspend", &cannot_suspend,
 	       doc: /* Non-nil means to always spawn a subshell instead of suspending.
@@ -12317,7 +12317,7 @@
 
 `overriding-terminal-local-map' has a separate binding for each
 terminal device.
-See Info node `(elisp)Multiple displays'.  */);
+See Info node `(elisp)Multiple Displays'.  */);
 
   DEFVAR_LISP ("overriding-local-map", &Voverriding_local_map,
 	       doc: /* Keymap that overrides all other local keymaps.
@@ -12345,7 +12345,7 @@
 and SYMBOL is its name.
 
 `system-key-alist' has a separate binding for each terminal device.
-See Info node `(elisp)Multiple displays'.  */);
+See Info node `(elisp)Multiple Displays'.  */);
 
   DEFVAR_KBOARD ("local-function-key-map", Vlocal_function_key_map,
                  doc: /* Keymap that translates key sequences to key sequences during input.
@@ -12371,7 +12371,7 @@
 typing `ESC O P x' would return [f1 x].
 
 `local-function-key-map' has a separate binding for each terminal
-device.  See Info node `(elisp)Multiple displays'.  If you need to
+device.  See Info node `(elisp)Multiple Displays'.  If you need to
 define a binding on all terminals, change `function-key-map'
 instead.  Initially, `local-function-key-map' is an empty keymap that
 has `function-key-map' as its parent on all terminal devices.  */);

[-- Attachment #3: Type: text/plain, Size: 1076 bytes --]



In GNU Emacs 23.1.1 (i486-pc-linux-gnu, GTK+ Version 2.16.5)
 of 2009-09-14 on raven, modified by Debian
configured using `configure  '--build=i486-linux-gnu' '--host=i486-linux-gnu' '--prefix=/usr' '--sharedstatedir=/var/lib' '--libexecdir=/usr/lib' '--localstatedir=/var/lib' '--infodir=/usr/share/info' '--mandir=/usr/share/man' '--with-pop=yes' '--enable-locallisppath=/etc/emacs23:/etc/emacs:/usr/local/share/emacs/23.1/site-lisp:/usr/local/share/emacs/site-lisp:/usr/share/emacs/23.1/site-lisp:/usr/share/emacs/site-lisp:/usr/share/emacs/23.1/leim' '--with-x=yes' '--with-x-toolkit=gtk' '--with-toolkit-scroll-bars' 'build_alias=i486-linux-gnu' 'host_alias=i486-linux-gnu' 'CFLAGS=-DDEBIAN -g -O2' 'LDFLAGS=-g' 'CPPFLAGS=''

Important settings:
  value of $LC_ALL: nil
  value of $LC_COLLATE: nil
  value of $LC_CTYPE: nil
  value of $LC_MESSAGES: nil
  value of $LC_MONETARY: nil
  value of $LC_NUMERIC: nil
  value of $LC_TIME: nil
  value of $LANG: en_AU
  value of $XMODIFIERS: nil
  locale-coding-system: iso-latin-1-unix
  default-enable-multibyte-characters: t

^ permalink raw reply	[flat|nested] 8+ messages in thread

* bug#6224: 23.1; last-command docstring "multiple displays" info node link
  2010-05-19 23:39 bug#6224: 23.1; last-command docstring "multiple displays" info node link Kevin Ryde
@ 2010-05-20  0:21 ` Juri Linkov
  2010-05-20  0:32   ` Kevin Ryde
  0 siblings, 1 reply; 8+ messages in thread
From: Juri Linkov @ 2010-05-20  0:21 UTC (permalink / raw)
  To: Kevin Ryde; +Cc: 6224

> In the C-h v docstring of `last-command', pressing Ret on the link
>
>     See Info node `(elisp)Multiple displays'.
>
> gets an error
>
>     No such node or anchor: Multiple displays
>
> I suppose it's a typo for "Multiple Displays", in five spots altogether
> in keyboard.c.  (Search and replace if the diff is out of date.)

BTW, do you have a command that automatically checks for broken links
in docstrings?

-- 
Juri Linkov
http://www.jurta.org/emacs/





^ permalink raw reply	[flat|nested] 8+ messages in thread

* bug#6224: 23.1; last-command docstring "multiple displays" info node link
  2010-05-20  0:21 ` Juri Linkov
@ 2010-05-20  0:32   ` Kevin Ryde
  2010-05-20  0:35     ` Kevin Ryde
  2010-05-20  0:46     ` Juri Linkov
  0 siblings, 2 replies; 8+ messages in thread
From: Kevin Ryde @ 2010-05-20  0:32 UTC (permalink / raw)
  To: Juri Linkov; +Cc: 6224

Juri Linkov <juri@jurta.org> writes:
>
> BTW, do you have a command that automatically checks for broken links
> in docstrings?

:-)  I threw down a couple of lines the other day

(defun my-emacs-info-xref-docstrings (filename-list)
  (require 'info-xref)
  (require 'lisp-mnt)
  (require 'help-mode)
  (dolist (filename filename-list)
    (lm-with-file filename
      (goto-char (point-min))
      (while (re-search-forward help-xref-info-regexp nil t)
        (let ((match (match-string 0))
              (node  (match-string 2)))
          (save-match-data ;; for the while loop
            (unless (string-match "\\`(" node)
              (setq node (concat "(emacs)" node)))
            (setq match (replace-regexp-in-string "\n" " " match))
            (setq node (replace-regexp-in-string "\n" " " node))
            (unless (or (string-match "%" node)
                        (info-xref-goto-node-p node))
              (lwarn 'emacs-info-xref-docstrings 'warning
                     "\n  %s:\n  no such node: %S"
                     filename match))))))))


to be used as say

(my-emacs-info-xref-docstrings (file-expand-wildcards "/down/emacs/src/*.c"))

or on your whole load-path

(progn
  (kill-buffer (get-buffer-create "*Warnings*"))
  (require 'cl)
  (dolist (dir load-path)
    (let ((lst (file-expand-wildcards (concat dir "/*.elc"))))
      (dolist (el (file-expand-wildcards (concat dir "/*.el")))
        (setq lst (remove (concat el "c") lst))
        (push el lst))
      (my-emacs-info-xref-docstrings lst))))

It picked up a typo in one of my own files.  I'm thinking of working it
up into a presentable form and adding to my info-xref.el (which is in
emacs).

I started with mapatoms and getting each `documentation', but a grep
through the .el doesn't load up lots of strings and can be applied to
packages without loading them.





^ permalink raw reply	[flat|nested] 8+ messages in thread

* bug#6224: 23.1; last-command docstring "multiple displays" info node link
  2010-05-20  0:32   ` Kevin Ryde
@ 2010-05-20  0:35     ` Kevin Ryde
  2010-05-20  0:55       ` Kevin Ryde
  2010-05-20  0:46     ` Juri Linkov
  1 sibling, 1 reply; 8+ messages in thread
From: Kevin Ryde @ 2010-05-20  0:35 UTC (permalink / raw)
  To: 6224

[-- Attachment #1: Type: text/plain, Size: 65 bytes --]

Oops, it's the emacs manual instead of the elisp manual too ...


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: keyboard.c.info-multiple-displays-2.diff --]
[-- Type: text/x-diff, Size: 2364 bytes --]

--- keyboard.c.~1.1026.~	2009-12-27 08:05:18.000000000 +1100
+++ keyboard.c	2010-05-20 10:34:18.000000000 +1000
@@ -1,7 +1,7 @@
 /* Keyboard and mouse input; editor command loop.
    Copyright (C) 1985, 1986, 1987, 1988, 1989, 1993, 1994, 1995,
                  1996, 1997, 1999, 2000, 2001, 2002, 2003, 2004,
-                 2005, 2006, 2007, 2008, 2009 Free Software Foundation, Inc.
+                 2005, 2006, 2007, 2008, 2009, 2010 Free Software Foundation, Inc.
 
 This file is part of GNU Emacs.
 
@@ -12087,7 +12087,7 @@
 was a kill command.
 
 `last-command' has a separate binding for each terminal device.
-See Info node `(elisp)Multiple displays'.  */);
+See Info node `Multiple Displays'.  */);
 
   DEFVAR_KBOARD ("real-last-command", Vreal_last_command,
 		 doc: /* Same as `last-command', but never altered by Lisp code.  */);
@@ -12226,7 +12226,7 @@
 output.  See also `translation-table-for-input'.
 
 This variable has a separate binding for each terminal.  See Info node
-`(elisp)Multiple displays'.  */);
+`Multiple Displays'.  */);
 
   DEFVAR_BOOL ("cannot-suspend", &cannot_suspend,
 	       doc: /* Non-nil means to always spawn a subshell instead of suspending.
@@ -12317,7 +12317,7 @@
 
 `overriding-terminal-local-map' has a separate binding for each
 terminal device.
-See Info node `(elisp)Multiple displays'.  */);
+See Info node `Multiple Displays'.  */);
 
   DEFVAR_LISP ("overriding-local-map", &Voverriding_local_map,
 	       doc: /* Keymap that overrides all other local keymaps.
@@ -12345,7 +12345,7 @@
 and SYMBOL is its name.
 
 `system-key-alist' has a separate binding for each terminal device.
-See Info node `(elisp)Multiple displays'.  */);
+See Info node `Multiple Displays'.  */);
 
   DEFVAR_KBOARD ("local-function-key-map", Vlocal_function_key_map,
                  doc: /* Keymap that translates key sequences to key sequences during input.
@@ -12371,7 +12371,7 @@
 typing `ESC O P x' would return [f1 x].
 
 `local-function-key-map' has a separate binding for each terminal
-device.  See Info node `(elisp)Multiple displays'.  If you need to
+device.  See Info node `Multiple Displays'.  If you need to
 define a binding on all terminals, change `function-key-map'
 instead.  Initially, `local-function-key-map' is an empty keymap that
 has `function-key-map' as its parent on all terminal devices.  */);

^ permalink raw reply	[flat|nested] 8+ messages in thread

* bug#6224: 23.1; last-command docstring "multiple displays" info node link
  2010-05-20  0:32   ` Kevin Ryde
  2010-05-20  0:35     ` Kevin Ryde
@ 2010-05-20  0:46     ` Juri Linkov
  2010-05-20  1:06       ` Kevin Ryde
  1 sibling, 1 reply; 8+ messages in thread
From: Juri Linkov @ 2010-05-20  0:46 UTC (permalink / raw)
  To: Kevin Ryde; +Cc: 6224

> I'm thinking of working it up into a presentable form and adding to my
> info-xref.el (which is in emacs).

Thanks, it would be a good addition to info-xref.el.

> I started with mapatoms and getting each `documentation', but a grep
> through the .el doesn't load up lots of strings and can be applied to
> packages without loading them.

How would you check docstrings of C primitives?  Maybe checking the
etc/DOC file will help?

-- 
Juri Linkov
http://www.jurta.org/emacs/





^ permalink raw reply	[flat|nested] 8+ messages in thread

* bug#6224: 23.1; last-command docstring "multiple displays" info node link
  2010-05-20  0:35     ` Kevin Ryde
@ 2010-05-20  0:55       ` Kevin Ryde
  2010-05-20 14:49         ` Chong Yidong
  0 siblings, 1 reply; 8+ messages in thread
From: Kevin Ryde @ 2010-05-20  0:55 UTC (permalink / raw)
  To: 6224

Ah dear, third time lucky!, is it meant to be

    `(elisp)Multiple Terminals'

per the merger of "Multiple Displays" and "Multiple Terminals" in
frames.texi 2009-04-04 ...





^ permalink raw reply	[flat|nested] 8+ messages in thread

* bug#6224: 23.1; last-command docstring "multiple displays" info node link
  2010-05-20  0:46     ` Juri Linkov
@ 2010-05-20  1:06       ` Kevin Ryde
  0 siblings, 0 replies; 8+ messages in thread
From: Kevin Ryde @ 2010-05-20  1:06 UTC (permalink / raw)
  To: Juri Linkov; +Cc: 6224

Juri Linkov <juri@jurta.org> writes:
>
> Maybe checking the etc/DOC file will help?

Ah yeah, if you don't have the .c files handy.





^ permalink raw reply	[flat|nested] 8+ messages in thread

* bug#6224: 23.1; last-command docstring "multiple displays" info node link
  2010-05-20  0:55       ` Kevin Ryde
@ 2010-05-20 14:49         ` Chong Yidong
  0 siblings, 0 replies; 8+ messages in thread
From: Chong Yidong @ 2010-05-20 14:49 UTC (permalink / raw)
  To: Kevin Ryde; +Cc: 6224-done

Kevin Ryde <user42@zip.com.au> writes:

> Ah dear, third time lucky!, is it meant to be
>
>     `(elisp)Multiple Terminals'
>
> per the merger of "Multiple Displays" and "Multiple Terminals" in
> frames.texi 2009-04-04 ...

Fixed, thanks.





^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2010-05-20 14:49 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-05-19 23:39 bug#6224: 23.1; last-command docstring "multiple displays" info node link Kevin Ryde
2010-05-20  0:21 ` Juri Linkov
2010-05-20  0:32   ` Kevin Ryde
2010-05-20  0:35     ` Kevin Ryde
2010-05-20  0:55       ` Kevin Ryde
2010-05-20 14:49         ` Chong Yidong
2010-05-20  0:46     ` Juri Linkov
2010-05-20  1:06       ` Kevin Ryde

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).