[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Patch to remove minor modes in tutorial
From: |
Richard Stallman |
Subject: |
Re: Patch to remove minor modes in tutorial |
Date: |
Mon, 24 Jul 2006 10:42:57 -0400 |
(default-keys
'(
;; These are not mentioned but are basic:
(ESC-prefix [27])
(Control-X-prefix [?\C-x])
(mode-specific-command-prefix [?\C-c])
Please put that constant list into a variable
instead of embedding it inside the function.
(sort-keys
(lambda(left right)
(let ((x (append (cadr left) nil))
(y (append (cadr right) nil)))
(while (and x y
(equal (car x) (car y)))
(setq x (cdr x))
(setq y (cdr y)))
Please give this a defun and give it a doc string that explains
precisely what it tests.
(insert-button (car remark)
'action
(lambda(b) (interactive)
(let ((value (button-get b 'value)))
(help--describe-nonstandard-key value)))
'value (cdr remark)
'follow-link t
'face '(:inherit link
:background "yellow"))
What is the purpose of presenting that button?
What does it do when the user clicks on it?
Comments have to say such things.
In general, your code has hardly any comments. Please add comments enough
to make it _clear_.
- Re: Patch to remove minor modes in tutorial, Richard Stallman, 2006/07/01
- Re: Patch to remove minor modes in tutorial, Lennart Borgman, 2006/07/02
- Re: Patch to remove minor modes in tutorial, Richard Stallman, 2006/07/02
- Re: Patch to remove minor modes in tutorial, Lennart Borgman, 2006/07/06
- Re: Patch to remove minor modes in tutorial, Richard Stallman, 2006/07/07
- Re: Patch to remove minor modes in tutorial, Giorgos Keramidas, 2006/07/07
- Re: Patch to remove minor modes in tutorial, Richard Stallman, 2006/07/08
- Re: Patch to remove minor modes in tutorial, Lennart Borgman, 2006/07/09
- Re: Patch to remove minor modes in tutorial, Lennart Borgman, 2006/07/17
- Re: Patch to remove minor modes in tutorial, Lennart Borgman, 2006/07/17
- Re: Patch to remove minor modes in tutorial,
Richard Stallman <=
- Re: Patch to remove minor modes in tutorial, Lennart Borgman, 2006/07/30