Mercurial > emacs
annotate lispref/tips.texi @ 59061:a7985894de81
Comment change.
| author | Richard M. Stallman <rms@gnu.org> |
|---|---|
| date | Tue, 21 Dec 2004 11:50:52 +0000 |
| parents | 7c9759696709 |
| children | a0cca815ceaa f2ebccfa87d4 |
| rev | line source |
|---|---|
| 6552 | 1 @c -*-texinfo-*- |
| 2 @c This is part of the GNU Emacs Lisp Reference Manual. | |
| 27189 | 3 @c Copyright (C) 1990, 1991, 1992, 1993, 1995, 1998, 1999 |
|
49600
23a1cea22d13
Trailing whitespace deleted.
Juanma Barranquero <lekktu@gmail.com>
parents:
44733
diff
changeset
|
4 @c Free Software Foundation, Inc. |
| 6552 | 5 @c See the file elisp.texi for copying conditions. |
| 6 @setfilename ../info/tips | |
| 30904 | 7 @node Tips, GNU Emacs Internals, GPL, Top |
|
17280
96762d1abb7c
(Coding Conventions): Node renamed from Style Tips.
Richard M. Stallman <rms@gnu.org>
parents:
16736
diff
changeset
|
8 @appendix Tips and Conventions |
| 6552 | 9 @cindex tips |
| 10 @cindex standards of coding style | |
| 11 @cindex coding standards | |
| 12 | |
|
17280
96762d1abb7c
(Coding Conventions): Node renamed from Style Tips.
Richard M. Stallman <rms@gnu.org>
parents:
16736
diff
changeset
|
13 This chapter describes no additional features of Emacs Lisp. Instead |
|
96762d1abb7c
(Coding Conventions): Node renamed from Style Tips.
Richard M. Stallman <rms@gnu.org>
parents:
16736
diff
changeset
|
14 it gives advice on making effective use of the features described in the |
|
96762d1abb7c
(Coding Conventions): Node renamed from Style Tips.
Richard M. Stallman <rms@gnu.org>
parents:
16736
diff
changeset
|
15 previous chapters, and describes conventions Emacs Lisp programmers |
|
96762d1abb7c
(Coding Conventions): Node renamed from Style Tips.
Richard M. Stallman <rms@gnu.org>
parents:
16736
diff
changeset
|
16 should follow. |
| 6552 | 17 |
|
25751
467b88fab665
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
24862
diff
changeset
|
18 You can automatically check some of the conventions described below by |
|
467b88fab665
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
24862
diff
changeset
|
19 running the command @kbd{M-x checkdoc RET} when visiting a Lisp file. |
|
467b88fab665
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
24862
diff
changeset
|
20 It cannot check all of the conventions, and not all the warnings it |
|
467b88fab665
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
24862
diff
changeset
|
21 gives necessarily correspond to problems, but it is worth examining them |
|
467b88fab665
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
24862
diff
changeset
|
22 all. |
|
467b88fab665
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
24862
diff
changeset
|
23 |
| 6552 | 24 @menu |
|
17280
96762d1abb7c
(Coding Conventions): Node renamed from Style Tips.
Richard M. Stallman <rms@gnu.org>
parents:
16736
diff
changeset
|
25 * Coding Conventions:: Conventions for clean and robust programs. |
| 6552 | 26 * Compilation Tips:: Making compiled code run fast. |
| 27 * Documentation Tips:: Writing readable documentation strings. | |
| 28 * Comment Tips:: Conventions for writing comments. | |
| 29 * Library Headers:: Standard headers for library packages. | |
| 30 @end menu | |
| 31 | |
|
17280
96762d1abb7c
(Coding Conventions): Node renamed from Style Tips.
Richard M. Stallman <rms@gnu.org>
parents:
16736
diff
changeset
|
32 @node Coding Conventions |
|
96762d1abb7c
(Coding Conventions): Node renamed from Style Tips.
Richard M. Stallman <rms@gnu.org>
parents:
16736
diff
changeset
|
33 @section Emacs Lisp Coding Conventions |
| 6552 | 34 |
|
17280
96762d1abb7c
(Coding Conventions): Node renamed from Style Tips.
Richard M. Stallman <rms@gnu.org>
parents:
16736
diff
changeset
|
35 Here are conventions that you should follow when writing Emacs Lisp |
|
96762d1abb7c
(Coding Conventions): Node renamed from Style Tips.
Richard M. Stallman <rms@gnu.org>
parents:
16736
diff
changeset
|
36 code intended for widespread use: |
| 6552 | 37 |
| 38 @itemize @bullet | |
| 39 @item | |
| 38162 | 40 Since all global variables share the same name space, and all |
| 41 functions share another name space, you should choose a short word to | |
| 42 distinguish your program from other Lisp programs.@footnote{The | |
| 43 benefits of a Common Lisp-style package system are considered not to | |
| 44 outweigh the costs.} Then take care to begin the names of all global | |
| 45 variables, constants, and functions in your program with the chosen | |
| 46 prefix. This helps avoid name conflicts. | |
| 6552 | 47 |
| 48 This recommendation applies even to names for traditional Lisp | |
|
21682
90da2489c498
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21007
diff
changeset
|
49 primitives that are not primitives in Emacs Lisp---even to |
|
90da2489c498
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21007
diff
changeset
|
50 @code{copy-list}. Believe it or not, there is more than one plausible |
|
90da2489c498
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21007
diff
changeset
|
51 way to define @code{copy-list}. Play it safe; append your name prefix |
|
90da2489c498
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21007
diff
changeset
|
52 to produce a name like @code{foo-copy-list} or @code{mylib-copy-list} |
|
90da2489c498
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21007
diff
changeset
|
53 instead. |
| 6552 | 54 |
| 55 If you write a function that you think ought to be added to Emacs under | |
| 56 a certain name, such as @code{twiddle-files}, don't call it by that name | |
| 57 in your program. Call it @code{mylib-twiddle-files} in your program, | |
|
22138
d4ac295a98b3
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21682
diff
changeset
|
58 and send mail to @samp{bug-gnu-emacs@@gnu.org} suggesting we add |
| 6552 | 59 it to Emacs. If and when we do, we can change the name easily enough. |
| 60 | |
| 61 If one prefix is insufficient, your package may use two or three | |
| 62 alternative common prefixes, so long as they make sense. | |
| 63 | |
| 64 Separate the prefix from the rest of the symbol name with a hyphen, | |
| 65 @samp{-}. This will be consistent with Emacs itself and with most Emacs | |
| 66 Lisp programs. | |
| 67 | |
| 68 @item | |
| 69 It is often useful to put a call to @code{provide} in each separate | |
| 70 library program, at least if there is more than one entry point to the | |
| 71 program. | |
| 72 | |
| 73 @item | |
| 12098 | 74 If a file requires certain other library programs to be loaded |
| 75 beforehand, then the comments at the beginning of the file should say | |
| 76 so. Also, use @code{require} to make sure they are loaded. | |
| 77 | |
| 78 @item | |
| 6552 | 79 If one file @var{foo} uses a macro defined in another file @var{bar}, |
| 12098 | 80 @var{foo} should contain this expression before the first use of the |
| 81 macro: | |
| 82 | |
| 83 @example | |
| 84 (eval-when-compile (require '@var{bar})) | |
| 85 @end example | |
| 86 | |
| 87 @noindent | |
|
21682
90da2489c498
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21007
diff
changeset
|
88 (And the library @var{bar} should contain @code{(provide '@var{bar})}, |
|
90da2489c498
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21007
diff
changeset
|
89 to make the @code{require} work.) This will cause @var{bar} to be |
|
90da2489c498
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21007
diff
changeset
|
90 loaded when you byte-compile @var{foo}. Otherwise, you risk compiling |
|
90da2489c498
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21007
diff
changeset
|
91 @var{foo} without the necessary macro loaded, and that would produce |
|
90da2489c498
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21007
diff
changeset
|
92 compiled code that won't work right. @xref{Compiling Macros}. |
| 12098 | 93 |
| 94 Using @code{eval-when-compile} avoids loading @var{bar} when | |
| 95 the compiled version of @var{foo} is @emph{used}. | |
| 6552 | 96 |
| 97 @item | |
| 24799 | 98 Please don't require the @code{cl} package of Common Lisp extensions at |
| 99 run time. Use of this package is optional, and it is not part of the | |
| 100 standard Emacs namespace. If your package loads @code{cl} at run time, | |
| 101 that could cause name clashes for users who don't use that package. | |
| 102 | |
| 103 However, there is no problem with using the @code{cl} package at compile | |
| 104 time, for the sake of macros. You do that like this: | |
| 105 | |
| 106 @example | |
| 107 (eval-when-compile (require 'cl)) | |
| 108 @end example | |
| 109 | |
| 110 @item | |
|
17280
96762d1abb7c
(Coding Conventions): Node renamed from Style Tips.
Richard M. Stallman <rms@gnu.org>
parents:
16736
diff
changeset
|
111 When defining a major mode, please follow the major mode |
|
96762d1abb7c
(Coding Conventions): Node renamed from Style Tips.
Richard M. Stallman <rms@gnu.org>
parents:
16736
diff
changeset
|
112 conventions. @xref{Major Mode Conventions}. |
|
96762d1abb7c
(Coding Conventions): Node renamed from Style Tips.
Richard M. Stallman <rms@gnu.org>
parents:
16736
diff
changeset
|
113 |
|
96762d1abb7c
(Coding Conventions): Node renamed from Style Tips.
Richard M. Stallman <rms@gnu.org>
parents:
16736
diff
changeset
|
114 @item |
|
96762d1abb7c
(Coding Conventions): Node renamed from Style Tips.
Richard M. Stallman <rms@gnu.org>
parents:
16736
diff
changeset
|
115 When defining a minor mode, please follow the minor mode |
|
96762d1abb7c
(Coding Conventions): Node renamed from Style Tips.
Richard M. Stallman <rms@gnu.org>
parents:
16736
diff
changeset
|
116 conventions. @xref{Minor Mode Conventions}. |
| 6552 | 117 |
| 118 @item | |
|
9807
d5fa87d62d62
Add function and variable name conventions.
Richard M. Stallman <rms@gnu.org>
parents:
9393
diff
changeset
|
119 If the purpose of a function is to tell you whether a certain condition |
|
d5fa87d62d62
Add function and variable name conventions.
Richard M. Stallman <rms@gnu.org>
parents:
9393
diff
changeset
|
120 is true or false, give the function a name that ends in @samp{p}. If |
|
d5fa87d62d62
Add function and variable name conventions.
Richard M. Stallman <rms@gnu.org>
parents:
9393
diff
changeset
|
121 the name is one word, add just @samp{p}; if the name is multiple words, |
|
d5fa87d62d62
Add function and variable name conventions.
Richard M. Stallman <rms@gnu.org>
parents:
9393
diff
changeset
|
122 add @samp{-p}. Examples are @code{framep} and @code{frame-live-p}. |
|
d5fa87d62d62
Add function and variable name conventions.
Richard M. Stallman <rms@gnu.org>
parents:
9393
diff
changeset
|
123 |
|
d5fa87d62d62
Add function and variable name conventions.
Richard M. Stallman <rms@gnu.org>
parents:
9393
diff
changeset
|
124 @item |
|
d5fa87d62d62
Add function and variable name conventions.
Richard M. Stallman <rms@gnu.org>
parents:
9393
diff
changeset
|
125 If a user option variable records a true-or-false condition, give it a |
|
d5fa87d62d62
Add function and variable name conventions.
Richard M. Stallman <rms@gnu.org>
parents:
9393
diff
changeset
|
126 name that ends in @samp{-flag}. |
|
d5fa87d62d62
Add function and variable name conventions.
Richard M. Stallman <rms@gnu.org>
parents:
9393
diff
changeset
|
127 |
|
d5fa87d62d62
Add function and variable name conventions.
Richard M. Stallman <rms@gnu.org>
parents:
9393
diff
changeset
|
128 @item |
|
52769
00608f913b6f
(Coding Conventions): Mention naming conventions for hooks.
Lute Kamstra <lute@gnu.org>
parents:
52401
diff
changeset
|
129 If the purpose of a variable is to store a single function, give it a |
|
00608f913b6f
(Coding Conventions): Mention naming conventions for hooks.
Lute Kamstra <lute@gnu.org>
parents:
52401
diff
changeset
|
130 name that ends in @samp{-function}. If the purpose of a variable is |
|
00608f913b6f
(Coding Conventions): Mention naming conventions for hooks.
Lute Kamstra <lute@gnu.org>
parents:
52401
diff
changeset
|
131 to store a list of functions (i.e., the variable is a hook), please |
|
00608f913b6f
(Coding Conventions): Mention naming conventions for hooks.
Lute Kamstra <lute@gnu.org>
parents:
52401
diff
changeset
|
132 follow the naming conventions for hooks. @xref{Hooks}. |
|
00608f913b6f
(Coding Conventions): Mention naming conventions for hooks.
Lute Kamstra <lute@gnu.org>
parents:
52401
diff
changeset
|
133 |
|
00608f913b6f
(Coding Conventions): Mention naming conventions for hooks.
Lute Kamstra <lute@gnu.org>
parents:
52401
diff
changeset
|
134 @item |
|
22138
d4ac295a98b3
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21682
diff
changeset
|
135 @cindex reserved keys |
|
d4ac295a98b3
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21682
diff
changeset
|
136 @cindex keys, reserved |
| 6552 | 137 Please do not define @kbd{C-c @var{letter}} as a key in your major |
|
44733
d49f5ed1a977
Clarify which letters in C-c LETTER are reserved for users.
Richard M. Stallman <rms@gnu.org>
parents:
44654
diff
changeset
|
138 modes. Sequences consisting of @kbd{C-c} and a letter (either upper |
|
d49f5ed1a977
Clarify which letters in C-c LETTER are reserved for users.
Richard M. Stallman <rms@gnu.org>
parents:
44654
diff
changeset
|
139 or lower case) are reserved for users; they are the @strong{only} |
|
d49f5ed1a977
Clarify which letters in C-c LETTER are reserved for users.
Richard M. Stallman <rms@gnu.org>
parents:
44654
diff
changeset
|
140 sequences reserved for users, so do not block them. |
| 6552 | 141 |
|
44733
d49f5ed1a977
Clarify which letters in C-c LETTER are reserved for users.
Richard M. Stallman <rms@gnu.org>
parents:
44654
diff
changeset
|
142 Changing all the Emacs major modes to respect this convention was a |
|
d49f5ed1a977
Clarify which letters in C-c LETTER are reserved for users.
Richard M. Stallman <rms@gnu.org>
parents:
44654
diff
changeset
|
143 lot of work; abandoning this convention would make that work go to |
|
d49f5ed1a977
Clarify which letters in C-c LETTER are reserved for users.
Richard M. Stallman <rms@gnu.org>
parents:
44654
diff
changeset
|
144 waste, and inconvenience users. Please comply with it. |
| 6552 | 145 |
|
44733
d49f5ed1a977
Clarify which letters in C-c LETTER are reserved for users.
Richard M. Stallman <rms@gnu.org>
parents:
44654
diff
changeset
|
146 @item |
|
d49f5ed1a977
Clarify which letters in C-c LETTER are reserved for users.
Richard M. Stallman <rms@gnu.org>
parents:
44654
diff
changeset
|
147 Sequences consisting of @kbd{C-c} followed by a control character or a |
|
d49f5ed1a977
Clarify which letters in C-c LETTER are reserved for users.
Richard M. Stallman <rms@gnu.org>
parents:
44654
diff
changeset
|
148 digit are reserved for major modes. |
|
11648
e09e51d7c35a
Describe uses of C-c followed by punctuation chars.
Richard M. Stallman <rms@gnu.org>
parents:
10229
diff
changeset
|
149 |
|
e09e51d7c35a
Describe uses of C-c followed by punctuation chars.
Richard M. Stallman <rms@gnu.org>
parents:
10229
diff
changeset
|
150 @item |
|
e09e51d7c35a
Describe uses of C-c followed by punctuation chars.
Richard M. Stallman <rms@gnu.org>
parents:
10229
diff
changeset
|
151 Sequences consisting of @kbd{C-c} followed by @kbd{@{}, @kbd{@}}, |
|
e09e51d7c35a
Describe uses of C-c followed by punctuation chars.
Richard M. Stallman <rms@gnu.org>
parents:
10229
diff
changeset
|
152 @kbd{<}, @kbd{>}, @kbd{:} or @kbd{;} are also reserved for major modes. |
|
e09e51d7c35a
Describe uses of C-c followed by punctuation chars.
Richard M. Stallman <rms@gnu.org>
parents:
10229
diff
changeset
|
153 |
|
e09e51d7c35a
Describe uses of C-c followed by punctuation chars.
Richard M. Stallman <rms@gnu.org>
parents:
10229
diff
changeset
|
154 @item |
|
e09e51d7c35a
Describe uses of C-c followed by punctuation chars.
Richard M. Stallman <rms@gnu.org>
parents:
10229
diff
changeset
|
155 Sequences consisting of @kbd{C-c} followed by any other punctuation |
|
e09e51d7c35a
Describe uses of C-c followed by punctuation chars.
Richard M. Stallman <rms@gnu.org>
parents:
10229
diff
changeset
|
156 character are allocated for minor modes. Using them in a major mode is |
|
e09e51d7c35a
Describe uses of C-c followed by punctuation chars.
Richard M. Stallman <rms@gnu.org>
parents:
10229
diff
changeset
|
157 not absolutely prohibited, but if you do that, the major mode binding |
|
e09e51d7c35a
Describe uses of C-c followed by punctuation chars.
Richard M. Stallman <rms@gnu.org>
parents:
10229
diff
changeset
|
158 may be shadowed from time to time by minor modes. |
| 6552 | 159 |
| 160 @item | |
|
19361
8546df4cb304
Reserve some function keys for users.
Richard M. Stallman <rms@gnu.org>
parents:
17280
diff
changeset
|
161 Function keys @key{F5} through @key{F9} without modifier keys are |
|
8546df4cb304
Reserve some function keys for users.
Richard M. Stallman <rms@gnu.org>
parents:
17280
diff
changeset
|
162 reserved for users to define. |
|
8546df4cb304
Reserve some function keys for users.
Richard M. Stallman <rms@gnu.org>
parents:
17280
diff
changeset
|
163 |
|
8546df4cb304
Reserve some function keys for users.
Richard M. Stallman <rms@gnu.org>
parents:
17280
diff
changeset
|
164 @item |
|
21007
66d807bdc5b4
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
19361
diff
changeset
|
165 Do not bind @kbd{C-h} following any prefix character (including |
| 6552 | 166 @kbd{C-c}). If you don't bind @kbd{C-h}, it is automatically available |
| 167 as a help character for listing the subcommands of the prefix character. | |
| 168 | |
| 169 @item | |
|
21007
66d807bdc5b4
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
19361
diff
changeset
|
170 Do not bind a key sequence ending in @key{ESC} except following |
|
21682
90da2489c498
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21007
diff
changeset
|
171 another @key{ESC}. (That is, it is OK to bind a sequence ending in |
| 6552 | 172 @kbd{@key{ESC} @key{ESC}}.) |
| 173 | |
| 174 The reason for this rule is that a non-prefix binding for @key{ESC} in | |
| 175 any context prevents recognition of escape sequences as function keys in | |
| 176 that context. | |
| 177 | |
| 178 @item | |
|
22635
1e9786a5c63d
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
22252
diff
changeset
|
179 Anything which acts like a temporary mode or state which the user can |
| 24862 | 180 enter and leave should define @kbd{@key{ESC} @key{ESC}} or |
|
22635
1e9786a5c63d
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
22252
diff
changeset
|
181 @kbd{@key{ESC} @key{ESC} @key{ESC}} as a way to escape. |
|
1e9786a5c63d
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
22252
diff
changeset
|
182 |
|
1e9786a5c63d
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
22252
diff
changeset
|
183 For a state which accepts ordinary Emacs commands, or more generally any |
|
1e9786a5c63d
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
22252
diff
changeset
|
184 kind of state in which @key{ESC} followed by a function key or arrow key |
|
1e9786a5c63d
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
22252
diff
changeset
|
185 is potentially meaningful, then you must not define @kbd{@key{ESC} |
|
1e9786a5c63d
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
22252
diff
changeset
|
186 @key{ESC}}, since that would preclude recognizing an escape sequence |
|
1e9786a5c63d
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
22252
diff
changeset
|
187 after @key{ESC}. In these states, you should define @kbd{@key{ESC} |
|
1e9786a5c63d
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
22252
diff
changeset
|
188 @key{ESC} @key{ESC}} as the way to escape. Otherwise, define |
|
1e9786a5c63d
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
22252
diff
changeset
|
189 @kbd{@key{ESC} @key{ESC}} instead. |
|
1e9786a5c63d
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
22252
diff
changeset
|
190 |
|
1e9786a5c63d
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
22252
diff
changeset
|
191 @item |
|
6959
3b19456b877a
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6552
diff
changeset
|
192 Applications should not bind mouse events based on button 1 with the |
|
3b19456b877a
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6552
diff
changeset
|
193 shift key held down. These events include @kbd{S-mouse-1}, |
|
3b19456b877a
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6552
diff
changeset
|
194 @kbd{M-S-mouse-1}, @kbd{C-S-mouse-1}, and so on. They are reserved for |
|
3b19456b877a
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6552
diff
changeset
|
195 users. |
|
3b19456b877a
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6552
diff
changeset
|
196 |
|
3b19456b877a
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6552
diff
changeset
|
197 @item |
| 34305 | 198 @cindex mouse-2 |
| 199 @cindex references, following | |
|
21007
66d807bdc5b4
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
19361
diff
changeset
|
200 Special major modes used for read-only text should usually redefine |
|
66d807bdc5b4
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
19361
diff
changeset
|
201 @kbd{mouse-2} and @key{RET} to trace some sort of reference in the text. |
|
66d807bdc5b4
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
19361
diff
changeset
|
202 Modes such as Dired, Info, Compilation, and Occur redefine it in this |
|
66d807bdc5b4
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
19361
diff
changeset
|
203 way. |
|
6959
3b19456b877a
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6552
diff
changeset
|
204 |
|
3b19456b877a
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6552
diff
changeset
|
205 @item |
|
9393
0bec3b6bac2f
Add a tip about enabling/disabling features.
Richard M. Stallman <rms@gnu.org>
parents:
8669
diff
changeset
|
206 When a package provides a modification of ordinary Emacs behavior, it is |
|
40561
ec17075b77aa
(Coding Conventions): Fix typo.
Pavel Jan?k <Pavel@Janik.cz>
parents:
40517
diff
changeset
|
207 good to include a command to enable and disable the feature, provide a |
|
9393
0bec3b6bac2f
Add a tip about enabling/disabling features.
Richard M. Stallman <rms@gnu.org>
parents:
8669
diff
changeset
|
208 command named @code{@var{whatever}-mode} which turns the feature on or |
|
0bec3b6bac2f
Add a tip about enabling/disabling features.
Richard M. Stallman <rms@gnu.org>
parents:
8669
diff
changeset
|
209 off, and make it autoload (@pxref{Autoload}). Design the package so |
|
0bec3b6bac2f
Add a tip about enabling/disabling features.
Richard M. Stallman <rms@gnu.org>
parents:
8669
diff
changeset
|
210 that simply loading it has no visible effect---that should not enable |
| 34305 | 211 the feature.@footnote{Consider that the package may be loaded |
| 212 arbitrarily by Custom for instance.} Users will request the feature by | |
|
52845
1826a112381b
(Library Headers): Clean up Documentation.
Richard M. Stallman <rms@gnu.org>
parents:
52814
diff
changeset
|
213 invoking the command. It is a good idea to define this command |
|
1826a112381b
(Library Headers): Clean up Documentation.
Richard M. Stallman <rms@gnu.org>
parents:
52814
diff
changeset
|
214 as a minor mode. |
|
52785
2c00436e93f7
Package unloading. Index next-line, previous-line, forward-line.
Dave Love <fx@gnu.org>
parents:
52769
diff
changeset
|
215 |
|
2c00436e93f7
Package unloading. Index next-line, previous-line, forward-line.
Dave Love <fx@gnu.org>
parents:
52769
diff
changeset
|
216 @cindex unloading packages |
|
52845
1826a112381b
(Library Headers): Clean up Documentation.
Richard M. Stallman <rms@gnu.org>
parents:
52814
diff
changeset
|
217 If loading the file adds functions to hooks, define a function |
|
1826a112381b
(Library Headers): Clean up Documentation.
Richard M. Stallman <rms@gnu.org>
parents:
52814
diff
changeset
|
218 @code{@var{feature}-unload-hook}, where @var{feature} is the name of |
|
1826a112381b
(Library Headers): Clean up Documentation.
Richard M. Stallman <rms@gnu.org>
parents:
52814
diff
changeset
|
219 the feature the package provides, and make it undo any such changes. |
|
1826a112381b
(Library Headers): Clean up Documentation.
Richard M. Stallman <rms@gnu.org>
parents:
52814
diff
changeset
|
220 Using @code{unload-feature} to unload the file will run this function. |
|
1826a112381b
(Library Headers): Clean up Documentation.
Richard M. Stallman <rms@gnu.org>
parents:
52814
diff
changeset
|
221 @xref{Unloading}. |
|
9393
0bec3b6bac2f
Add a tip about enabling/disabling features.
Richard M. Stallman <rms@gnu.org>
parents:
8669
diff
changeset
|
222 |
|
0bec3b6bac2f
Add a tip about enabling/disabling features.
Richard M. Stallman <rms@gnu.org>
parents:
8669
diff
changeset
|
223 @item |
|
0bec3b6bac2f
Add a tip about enabling/disabling features.
Richard M. Stallman <rms@gnu.org>
parents:
8669
diff
changeset
|
224 It is a bad idea to define aliases for the Emacs primitives. Use the |
|
0bec3b6bac2f
Add a tip about enabling/disabling features.
Richard M. Stallman <rms@gnu.org>
parents:
8669
diff
changeset
|
225 standard names instead. |
| 6552 | 226 |
| 227 @item | |
| 34305 | 228 If a package needs to define an alias or a new function for |
| 38344 | 229 compatibility with some other version of Emacs, name it with the package |
| 34305 | 230 prefix, not with the raw name with which it occurs in the other version. |
| 231 Here is an example from Gnus, which provides many examples of such | |
| 232 compatibility issues. | |
| 233 | |
| 234 @example | |
| 235 (defalias 'gnus-point-at-bol | |
| 236 (if (fboundp 'point-at-bol) | |
| 237 'point-at-bol | |
| 238 'line-beginning-position)) | |
| 239 @end example | |
| 240 | |
| 241 @item | |
|
17280
96762d1abb7c
(Coding Conventions): Node renamed from Style Tips.
Richard M. Stallman <rms@gnu.org>
parents:
16736
diff
changeset
|
242 Redefining (or advising) an Emacs primitive is discouraged. It may do |
|
96762d1abb7c
(Coding Conventions): Node renamed from Style Tips.
Richard M. Stallman <rms@gnu.org>
parents:
16736
diff
changeset
|
243 the right thing for a particular program, but there is no telling what |
|
96762d1abb7c
(Coding Conventions): Node renamed from Style Tips.
Richard M. Stallman <rms@gnu.org>
parents:
16736
diff
changeset
|
244 other programs might break as a result. |
| 6552 | 245 |
| 246 @item | |
| 247 If a file does replace any of the functions or library programs of | |
| 248 standard Emacs, prominent comments at the beginning of the file should | |
| 249 say which functions are replaced, and how the behavior of the | |
| 250 replacements differs from that of the originals. | |
| 251 | |
| 252 @item | |
|
52845
1826a112381b
(Library Headers): Clean up Documentation.
Richard M. Stallman <rms@gnu.org>
parents:
52814
diff
changeset
|
253 Avoid using macros that define functions and variables with names that |
|
54042
ff694e194b01
(Coding Conventions): Minor fix.
Richard M. Stallman <rms@gnu.org>
parents:
53924
diff
changeset
|
254 are constructed. It is best for maintenance when the name of the |
|
52845
1826a112381b
(Library Headers): Clean up Documentation.
Richard M. Stallman <rms@gnu.org>
parents:
52814
diff
changeset
|
255 function or variable being defined is given explicitly in the source |
|
1826a112381b
(Library Headers): Clean up Documentation.
Richard M. Stallman <rms@gnu.org>
parents:
52814
diff
changeset
|
256 code, as the second element of the list---as it is when you use |
|
54964
4732f3096a88
(Coding Conventions): defopt -> defcustom.
Jesper Harder <harder@ifa.au.dk>
parents:
54124
diff
changeset
|
257 @code{defun}, @code{defalias}, @code{defvar} and @code{defcustom}. |
|
52845
1826a112381b
(Library Headers): Clean up Documentation.
Richard M. Stallman <rms@gnu.org>
parents:
52814
diff
changeset
|
258 |
|
1826a112381b
(Library Headers): Clean up Documentation.
Richard M. Stallman <rms@gnu.org>
parents:
52814
diff
changeset
|
259 @item |
| 6552 | 260 Please keep the names of your Emacs Lisp source files to 13 characters |
| 261 or less. This way, if the files are compiled, the compiled files' names | |
| 262 will be 14 characters or less, which is short enough to fit on all kinds | |
| 263 of Unix systems. | |
| 264 | |
| 265 @item | |
| 266 Don't use @code{next-line} or @code{previous-line} in programs; nearly | |
| 267 always, @code{forward-line} is more convenient as well as more | |
| 268 predictable and robust. @xref{Text Lines}. | |
| 269 | |
| 270 @item | |
|
7601
c5927c75b2b5
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6959
diff
changeset
|
271 Don't call functions that set the mark, unless setting the mark is one |
|
c5927c75b2b5
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6959
diff
changeset
|
272 of the intended features of your program. The mark is a user-level |
|
c5927c75b2b5
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6959
diff
changeset
|
273 feature, so it is incorrect to change the mark except to supply a value |
|
c5927c75b2b5
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6959
diff
changeset
|
274 for the user's benefit. @xref{The Mark}. |
| 6552 | 275 |
|
21007
66d807bdc5b4
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
19361
diff
changeset
|
276 In particular, don't use any of these functions: |
| 6552 | 277 |
| 278 @itemize @bullet | |
| 279 @item | |
| 280 @code{beginning-of-buffer}, @code{end-of-buffer} | |
| 281 @item | |
| 282 @code{replace-string}, @code{replace-regexp} | |
| 283 @end itemize | |
| 284 | |
| 285 If you just want to move point, or replace a certain string, without any | |
| 286 of the other features intended for interactive users, you can replace | |
| 287 these functions with one or two lines of simple Lisp code. | |
| 288 | |
| 289 @item | |
| 8669 | 290 Use lists rather than vectors, except when there is a particular reason |
| 291 to use a vector. Lisp has more facilities for manipulating lists than | |
| 292 for vectors, and working with lists is usually more convenient. | |
| 293 | |
| 294 Vectors are advantageous for tables that are substantial in size and are | |
| 295 accessed in random order (not searched front to back), provided there is | |
| 296 no need to insert or delete elements (only lists allow that). | |
| 297 | |
| 298 @item | |
| 6552 | 299 The recommended way to print a message in the echo area is with |
| 300 the @code{message} function, not @code{princ}. @xref{The Echo Area}. | |
| 301 | |
| 302 @item | |
| 303 When you encounter an error condition, call the function @code{error} | |
| 304 (or @code{signal}). The function @code{error} does not return. | |
| 305 @xref{Signaling Errors}. | |
| 306 | |
| 307 Do not use @code{message}, @code{throw}, @code{sleep-for}, | |
| 308 or @code{beep} to report errors. | |
| 309 | |
| 310 @item | |
| 12098 | 311 An error message should start with a capital letter but should not end |
| 312 with a period. | |
| 313 | |
| 314 @item | |
|
38043
3577e0c41a1e
Give advice about how to default position args and region args.
Richard M. Stallman <rms@gnu.org>
parents:
34305
diff
changeset
|
315 In @code{interactive}, if you use a Lisp expression to produce a list |
|
3577e0c41a1e
Give advice about how to default position args and region args.
Richard M. Stallman <rms@gnu.org>
parents:
34305
diff
changeset
|
316 of arguments, don't try to provide the ``correct'' default values for |
|
3577e0c41a1e
Give advice about how to default position args and region args.
Richard M. Stallman <rms@gnu.org>
parents:
34305
diff
changeset
|
317 region or position arguments. Instead, provide @code{nil} for those |
|
3577e0c41a1e
Give advice about how to default position args and region args.
Richard M. Stallman <rms@gnu.org>
parents:
34305
diff
changeset
|
318 arguments if they were not specified, and have the function body |
|
3577e0c41a1e
Give advice about how to default position args and region args.
Richard M. Stallman <rms@gnu.org>
parents:
34305
diff
changeset
|
319 compute the default value when the argument is @code{nil}. For |
|
3577e0c41a1e
Give advice about how to default position args and region args.
Richard M. Stallman <rms@gnu.org>
parents:
34305
diff
changeset
|
320 instance, write this: |
|
3577e0c41a1e
Give advice about how to default position args and region args.
Richard M. Stallman <rms@gnu.org>
parents:
34305
diff
changeset
|
321 |
|
3577e0c41a1e
Give advice about how to default position args and region args.
Richard M. Stallman <rms@gnu.org>
parents:
34305
diff
changeset
|
322 @example |
|
3577e0c41a1e
Give advice about how to default position args and region args.
Richard M. Stallman <rms@gnu.org>
parents:
34305
diff
changeset
|
323 (defun foo (pos) |
|
3577e0c41a1e
Give advice about how to default position args and region args.
Richard M. Stallman <rms@gnu.org>
parents:
34305
diff
changeset
|
324 (interactive |
|
3577e0c41a1e
Give advice about how to default position args and region args.
Richard M. Stallman <rms@gnu.org>
parents:
34305
diff
changeset
|
325 (list (if @var{specified} @var{specified-pos}))) |
|
3577e0c41a1e
Give advice about how to default position args and region args.
Richard M. Stallman <rms@gnu.org>
parents:
34305
diff
changeset
|
326 (unless pos (setq pos @var{default-pos})) |
|
3577e0c41a1e
Give advice about how to default position args and region args.
Richard M. Stallman <rms@gnu.org>
parents:
34305
diff
changeset
|
327 ...) |
|
3577e0c41a1e
Give advice about how to default position args and region args.
Richard M. Stallman <rms@gnu.org>
parents:
34305
diff
changeset
|
328 @end example |
|
3577e0c41a1e
Give advice about how to default position args and region args.
Richard M. Stallman <rms@gnu.org>
parents:
34305
diff
changeset
|
329 |
|
3577e0c41a1e
Give advice about how to default position args and region args.
Richard M. Stallman <rms@gnu.org>
parents:
34305
diff
changeset
|
330 @noindent |
|
3577e0c41a1e
Give advice about how to default position args and region args.
Richard M. Stallman <rms@gnu.org>
parents:
34305
diff
changeset
|
331 rather than this: |
|
3577e0c41a1e
Give advice about how to default position args and region args.
Richard M. Stallman <rms@gnu.org>
parents:
34305
diff
changeset
|
332 |
|
3577e0c41a1e
Give advice about how to default position args and region args.
Richard M. Stallman <rms@gnu.org>
parents:
34305
diff
changeset
|
333 @example |
|
3577e0c41a1e
Give advice about how to default position args and region args.
Richard M. Stallman <rms@gnu.org>
parents:
34305
diff
changeset
|
334 (defun foo (pos) |
|
3577e0c41a1e
Give advice about how to default position args and region args.
Richard M. Stallman <rms@gnu.org>
parents:
34305
diff
changeset
|
335 (interactive |
|
3577e0c41a1e
Give advice about how to default position args and region args.
Richard M. Stallman <rms@gnu.org>
parents:
34305
diff
changeset
|
336 (list (if @var{specified} @var{specified-pos} |
|
3577e0c41a1e
Give advice about how to default position args and region args.
Richard M. Stallman <rms@gnu.org>
parents:
34305
diff
changeset
|
337 @var{default-pos}))) |
|
3577e0c41a1e
Give advice about how to default position args and region args.
Richard M. Stallman <rms@gnu.org>
parents:
34305
diff
changeset
|
338 ...) |
|
3577e0c41a1e
Give advice about how to default position args and region args.
Richard M. Stallman <rms@gnu.org>
parents:
34305
diff
changeset
|
339 @end example |
|
3577e0c41a1e
Give advice about how to default position args and region args.
Richard M. Stallman <rms@gnu.org>
parents:
34305
diff
changeset
|
340 |
|
3577e0c41a1e
Give advice about how to default position args and region args.
Richard M. Stallman <rms@gnu.org>
parents:
34305
diff
changeset
|
341 @noindent |
|
3577e0c41a1e
Give advice about how to default position args and region args.
Richard M. Stallman <rms@gnu.org>
parents:
34305
diff
changeset
|
342 This is so that repetition of the command will recompute |
|
3577e0c41a1e
Give advice about how to default position args and region args.
Richard M. Stallman <rms@gnu.org>
parents:
34305
diff
changeset
|
343 these defaults based on the current circumstances. |
|
3577e0c41a1e
Give advice about how to default position args and region args.
Richard M. Stallman <rms@gnu.org>
parents:
34305
diff
changeset
|
344 |
|
3577e0c41a1e
Give advice about how to default position args and region args.
Richard M. Stallman <rms@gnu.org>
parents:
34305
diff
changeset
|
345 You do not need to take such precautions when you use interactive |
|
3577e0c41a1e
Give advice about how to default position args and region args.
Richard M. Stallman <rms@gnu.org>
parents:
34305
diff
changeset
|
346 specs @samp{d}, @samp{m} and @samp{r}, because they make special |
|
3577e0c41a1e
Give advice about how to default position args and region args.
Richard M. Stallman <rms@gnu.org>
parents:
34305
diff
changeset
|
347 arrangements to recompute the argument values on repetition of the |
|
3577e0c41a1e
Give advice about how to default position args and region args.
Richard M. Stallman <rms@gnu.org>
parents:
34305
diff
changeset
|
348 command. |
|
3577e0c41a1e
Give advice about how to default position args and region args.
Richard M. Stallman <rms@gnu.org>
parents:
34305
diff
changeset
|
349 |
|
3577e0c41a1e
Give advice about how to default position args and region args.
Richard M. Stallman <rms@gnu.org>
parents:
34305
diff
changeset
|
350 @item |
|
13893
2576d1142ed3
Explain style of "done" messages.
Richard M. Stallman <rms@gnu.org>
parents:
12098
diff
changeset
|
351 Many commands that take a long time to execute display a message that |
|
38043
3577e0c41a1e
Give advice about how to default position args and region args.
Richard M. Stallman <rms@gnu.org>
parents:
34305
diff
changeset
|
352 says something like @samp{Operating...} when they start, and change it to |
|
13893
2576d1142ed3
Explain style of "done" messages.
Richard M. Stallman <rms@gnu.org>
parents:
12098
diff
changeset
|
353 @samp{Operating...done} when they finish. Please keep the style of |
|
2576d1142ed3
Explain style of "done" messages.
Richard M. Stallman <rms@gnu.org>
parents:
12098
diff
changeset
|
354 these messages uniform: @emph{no} space around the ellipsis, and |
|
38043
3577e0c41a1e
Give advice about how to default position args and region args.
Richard M. Stallman <rms@gnu.org>
parents:
34305
diff
changeset
|
355 @emph{no} period after @samp{done}. |
|
13893
2576d1142ed3
Explain style of "done" messages.
Richard M. Stallman <rms@gnu.org>
parents:
12098
diff
changeset
|
356 |
|
2576d1142ed3
Explain style of "done" messages.
Richard M. Stallman <rms@gnu.org>
parents:
12098
diff
changeset
|
357 @item |
|
6959
3b19456b877a
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6552
diff
changeset
|
358 Try to avoid using recursive edits. Instead, do what the Rmail @kbd{e} |
|
3b19456b877a
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6552
diff
changeset
|
359 command does: use a new local keymap that contains one command defined |
|
3b19456b877a
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6552
diff
changeset
|
360 to switch back to the old local keymap. Or do what the |
|
3b19456b877a
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6552
diff
changeset
|
361 @code{edit-options} command does: switch to another buffer and let the |
|
3b19456b877a
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6552
diff
changeset
|
362 user switch back at will. @xref{Recursive Editing}. |
| 6552 | 363 |
| 364 @item | |
| 365 In some other systems there is a convention of choosing variable names | |
| 366 that begin and end with @samp{*}. We don't use that convention in Emacs | |
|
6959
3b19456b877a
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6552
diff
changeset
|
367 Lisp, so please don't use it in your programs. (Emacs uses such names |
|
21682
90da2489c498
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21007
diff
changeset
|
368 only for special-purpose buffers.) The users will find Emacs more |
|
6959
3b19456b877a
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6552
diff
changeset
|
369 coherent if all libraries use the same conventions. |
| 6552 | 370 |
| 371 @item | |
|
14028
e8d6c760f796
Explain eliminating compiler warnings about undefined variables.
Richard M. Stallman <rms@gnu.org>
parents:
13893
diff
changeset
|
372 Try to avoid compiler warnings about undefined free variables, by adding |
|
58281
b6f9481dcccd
(Coding Conventions): Separate defvar and require
Richard M. Stallman <rms@gnu.org>
parents:
54964
diff
changeset
|
373 dummy @code{defvar} definitions for these variables, like this: |
|
b6f9481dcccd
(Coding Conventions): Separate defvar and require
Richard M. Stallman <rms@gnu.org>
parents:
54964
diff
changeset
|
374 |
|
b6f9481dcccd
(Coding Conventions): Separate defvar and require
Richard M. Stallman <rms@gnu.org>
parents:
54964
diff
changeset
|
375 @example |
|
b6f9481dcccd
(Coding Conventions): Separate defvar and require
Richard M. Stallman <rms@gnu.org>
parents:
54964
diff
changeset
|
376 (defvar foo) |
|
b6f9481dcccd
(Coding Conventions): Separate defvar and require
Richard M. Stallman <rms@gnu.org>
parents:
54964
diff
changeset
|
377 @end example |
|
14028
e8d6c760f796
Explain eliminating compiler warnings about undefined variables.
Richard M. Stallman <rms@gnu.org>
parents:
13893
diff
changeset
|
378 |
|
58281
b6f9481dcccd
(Coding Conventions): Separate defvar and require
Richard M. Stallman <rms@gnu.org>
parents:
54964
diff
changeset
|
379 Such a definition has no effect except to tell the compiler |
|
b6f9481dcccd
(Coding Conventions): Separate defvar and require
Richard M. Stallman <rms@gnu.org>
parents:
54964
diff
changeset
|
380 not to warn about uses of the variable @code{foo} in this file. |
|
b6f9481dcccd
(Coding Conventions): Separate defvar and require
Richard M. Stallman <rms@gnu.org>
parents:
54964
diff
changeset
|
381 |
|
b6f9481dcccd
(Coding Conventions): Separate defvar and require
Richard M. Stallman <rms@gnu.org>
parents:
54964
diff
changeset
|
382 @item |
|
b6f9481dcccd
(Coding Conventions): Separate defvar and require
Richard M. Stallman <rms@gnu.org>
parents:
54964
diff
changeset
|
383 If you use many functions and variables from a certain file, you can |
|
b6f9481dcccd
(Coding Conventions): Separate defvar and require
Richard M. Stallman <rms@gnu.org>
parents:
54964
diff
changeset
|
384 add a @code{require} for that package to avoid compilation warnings |
|
b6f9481dcccd
(Coding Conventions): Separate defvar and require
Richard M. Stallman <rms@gnu.org>
parents:
54964
diff
changeset
|
385 for them. It is better if the @code{require} acts only at compile |
|
b6f9481dcccd
(Coding Conventions): Separate defvar and require
Richard M. Stallman <rms@gnu.org>
parents:
54964
diff
changeset
|
386 time. Here's how to do this: |
|
25751
467b88fab665
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
24862
diff
changeset
|
387 |
|
467b88fab665
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
24862
diff
changeset
|
388 @example |
|
467b88fab665
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
24862
diff
changeset
|
389 (eval-when-compile |
|
58281
b6f9481dcccd
(Coding Conventions): Separate defvar and require
Richard M. Stallman <rms@gnu.org>
parents:
54964
diff
changeset
|
390 (require 'foo)) |
|
25751
467b88fab665
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
24862
diff
changeset
|
391 @end example |
|
467b88fab665
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
24862
diff
changeset
|
392 |
|
58281
b6f9481dcccd
(Coding Conventions): Separate defvar and require
Richard M. Stallman <rms@gnu.org>
parents:
54964
diff
changeset
|
393 @item |
|
b6f9481dcccd
(Coding Conventions): Separate defvar and require
Richard M. Stallman <rms@gnu.org>
parents:
54964
diff
changeset
|
394 If you bind a variable in one function, and use it or set it in |
|
b6f9481dcccd
(Coding Conventions): Separate defvar and require
Richard M. Stallman <rms@gnu.org>
parents:
54964
diff
changeset
|
395 another function, the compiler warns about the latter function unless |
|
b6f9481dcccd
(Coding Conventions): Separate defvar and require
Richard M. Stallman <rms@gnu.org>
parents:
54964
diff
changeset
|
396 the variable has a definition. But adding a definition would be |
|
58323
7c9759696709
(Coding Conventions): Fix typo.
Thien-Thi Nguyen <ttn@gnuvola.org>
parents:
58281
diff
changeset
|
397 unclean if the variable has a short name, since Lisp packages should |
|
58281
b6f9481dcccd
(Coding Conventions): Separate defvar and require
Richard M. Stallman <rms@gnu.org>
parents:
54964
diff
changeset
|
398 not define short variable names. The right thing to do is to rename |
|
b6f9481dcccd
(Coding Conventions): Separate defvar and require
Richard M. Stallman <rms@gnu.org>
parents:
54964
diff
changeset
|
399 this variable to start with the name prefix used for the other |
|
b6f9481dcccd
(Coding Conventions): Separate defvar and require
Richard M. Stallman <rms@gnu.org>
parents:
54964
diff
changeset
|
400 functions and variables in your package. |
|
14028
e8d6c760f796
Explain eliminating compiler warnings about undefined variables.
Richard M. Stallman <rms@gnu.org>
parents:
13893
diff
changeset
|
401 |
|
e8d6c760f796
Explain eliminating compiler warnings about undefined variables.
Richard M. Stallman <rms@gnu.org>
parents:
13893
diff
changeset
|
402 @item |
| 6552 | 403 Indent each function with @kbd{C-M-q} (@code{indent-sexp}) using the |
| 404 default indentation parameters. | |
| 405 | |
| 406 @item | |
| 407 Don't make a habit of putting close-parentheses on lines by themselves; | |
| 408 Lisp programmers find this disconcerting. Once in a while, when there | |
| 409 is a sequence of many consecutive close-parentheses, it may make sense | |
|
21682
90da2489c498
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21007
diff
changeset
|
410 to split the sequence in one or two significant places. |
| 6552 | 411 |
| 412 @item | |
| 413 Please put a copyright notice on the file if you give copies to anyone. | |
|
21007
66d807bdc5b4
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
19361
diff
changeset
|
414 Use a message like this one: |
|
66d807bdc5b4
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
19361
diff
changeset
|
415 |
|
66d807bdc5b4
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
19361
diff
changeset
|
416 @smallexample |
|
66d807bdc5b4
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
19361
diff
changeset
|
417 ;; Copyright (C) @var{year} @var{name} |
|
66d807bdc5b4
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
19361
diff
changeset
|
418 |
|
66d807bdc5b4
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
19361
diff
changeset
|
419 ;; This program is free software; you can redistribute it and/or |
|
66d807bdc5b4
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
19361
diff
changeset
|
420 ;; modify it under the terms of the GNU General Public License as |
|
66d807bdc5b4
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
19361
diff
changeset
|
421 ;; published by the Free Software Foundation; either version 2 of |
|
66d807bdc5b4
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
19361
diff
changeset
|
422 ;; the License, or (at your option) any later version. |
|
66d807bdc5b4
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
19361
diff
changeset
|
423 |
|
66d807bdc5b4
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
19361
diff
changeset
|
424 ;; This program is distributed in the hope that it will be |
|
66d807bdc5b4
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
19361
diff
changeset
|
425 ;; useful, but WITHOUT ANY WARRANTY; without even the implied |
|
66d807bdc5b4
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
19361
diff
changeset
|
426 ;; warranty of MERCHANTABILITY or FITNESS FOR A PARTICULAR |
|
66d807bdc5b4
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
19361
diff
changeset
|
427 ;; PURPOSE. See the GNU General Public License for more details. |
|
66d807bdc5b4
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
19361
diff
changeset
|
428 |
|
66d807bdc5b4
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
19361
diff
changeset
|
429 ;; You should have received a copy of the GNU General Public |
|
66d807bdc5b4
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
19361
diff
changeset
|
430 ;; License along with this program; if not, write to the Free |
|
66d807bdc5b4
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
19361
diff
changeset
|
431 ;; Software Foundation, Inc., 59 Temple Place, Suite 330, Boston, |
|
66d807bdc5b4
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
19361
diff
changeset
|
432 ;; MA 02111-1307 USA |
|
66d807bdc5b4
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
19361
diff
changeset
|
433 @end smallexample |
|
66d807bdc5b4
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
19361
diff
changeset
|
434 |
|
66d807bdc5b4
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
19361
diff
changeset
|
435 If you have signed papers to assign the copyright to the Foundation, |
|
66d807bdc5b4
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
19361
diff
changeset
|
436 then use @samp{Free Software Foundation, Inc.} as @var{name}. |
|
66d807bdc5b4
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
19361
diff
changeset
|
437 Otherwise, use your name. |
| 6552 | 438 @end itemize |
| 439 | |
| 440 @node Compilation Tips | |
| 441 @section Tips for Making Compiled Code Fast | |
| 442 @cindex execution speed | |
| 443 @cindex speedups | |
| 444 | |
| 445 Here are ways of improving the execution speed of byte-compiled | |
|
6959
3b19456b877a
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6552
diff
changeset
|
446 Lisp programs. |
| 6552 | 447 |
| 448 @itemize @bullet | |
| 449 @item | |
| 450 @cindex profiling | |
| 451 @cindex timing programs | |
|
22138
d4ac295a98b3
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21682
diff
changeset
|
452 @cindex @file{elp.el} |
|
44654
774d49397ece
(Tips for Making Compiled Code Fast): Don't advertise profile.el.
Eli Zaretskii <eliz@gnu.org>
parents:
42979
diff
changeset
|
453 Profile your program with the @file{elp} library. See the file |
|
774d49397ece
(Tips for Making Compiled Code Fast): Don't advertise profile.el.
Eli Zaretskii <eliz@gnu.org>
parents:
42979
diff
changeset
|
454 @file{elp.el} for instructions. |
| 6552 | 455 |
| 456 @item | |
| 457 Use iteration rather than recursion whenever possible. | |
| 458 Function calls are slow in Emacs Lisp even when a compiled function | |
| 459 is calling another compiled function. | |
| 460 | |
| 461 @item | |
| 12098 | 462 Using the primitive list-searching functions @code{memq}, @code{member}, |
| 463 @code{assq}, or @code{assoc} is even faster than explicit iteration. It | |
|
21007
66d807bdc5b4
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
19361
diff
changeset
|
464 can be worth rearranging a data structure so that one of these primitive |
| 12098 | 465 search functions can be used. |
| 6552 | 466 |
| 467 @item | |
|
49600
23a1cea22d13
Trailing whitespace deleted.
Juanma Barranquero <lekktu@gmail.com>
parents:
44733
diff
changeset
|
468 Certain built-in functions are handled specially in byte-compiled code, |
| 6552 | 469 avoiding the need for an ordinary function call. It is a good idea to |
| 470 use these functions rather than alternatives. To see whether a function | |
| 471 is handled specially by the compiler, examine its @code{byte-compile} | |
| 472 property. If the property is non-@code{nil}, then the function is | |
| 473 handled specially. | |
| 474 | |
| 475 For example, the following input will show you that @code{aref} is | |
|
22138
d4ac295a98b3
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21682
diff
changeset
|
476 compiled specially (@pxref{Array Functions}): |
| 6552 | 477 |
|
6959
3b19456b877a
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6552
diff
changeset
|
478 @example |
| 6552 | 479 @group |
| 480 (get 'aref 'byte-compile) | |
| 481 @result{} byte-compile-two-args | |
| 482 @end group | |
|
6959
3b19456b877a
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6552
diff
changeset
|
483 @end example |
| 6552 | 484 |
| 485 @item | |
|
22252
40089afa2b1d
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
22138
diff
changeset
|
486 If calling a small function accounts for a substantial part of your |
| 6552 | 487 program's running time, make the function inline. This eliminates |
| 488 the function call overhead. Since making a function inline reduces | |
| 489 the flexibility of changing the program, don't do it unless it gives | |
|
6959
3b19456b877a
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6552
diff
changeset
|
490 a noticeable speedup in something slow enough that users care about |
| 6552 | 491 the speed. @xref{Inline Functions}. |
| 492 @end itemize | |
| 493 | |
| 494 @node Documentation Tips | |
| 495 @section Tips for Documentation Strings | |
| 496 | |
|
21682
90da2489c498
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21007
diff
changeset
|
497 @findex checkdoc-minor-mode |
|
90da2489c498
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21007
diff
changeset
|
498 Here are some tips and conventions for the writing of documentation |
|
90da2489c498
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21007
diff
changeset
|
499 strings. You can check many of these conventions by running the command |
|
90da2489c498
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21007
diff
changeset
|
500 @kbd{M-x checkdoc-minor-mode}. |
| 6552 | 501 |
| 502 @itemize @bullet | |
| 503 @item | |
|
7601
c5927c75b2b5
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6959
diff
changeset
|
504 Every command, function, or variable intended for users to know about |
| 6552 | 505 should have a documentation string. |
| 506 | |
| 507 @item | |
|
10225
2e5dcd5f3090
Recommend doc strings for all functions and variables.
Richard M. Stallman <rms@gnu.org>
parents:
9807
diff
changeset
|
508 An internal variable or subroutine of a Lisp program might as well have |
|
2e5dcd5f3090
Recommend doc strings for all functions and variables.
Richard M. Stallman <rms@gnu.org>
parents:
9807
diff
changeset
|
509 a documentation string. In earlier Emacs versions, you could save space |
|
2e5dcd5f3090
Recommend doc strings for all functions and variables.
Richard M. Stallman <rms@gnu.org>
parents:
9807
diff
changeset
|
510 by using a comment instead of a documentation string, but that is no |
|
26696
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
511 longer the case---documentation strings now take up very little space in |
|
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
512 a running Emacs. |
| 6552 | 513 |
| 514 @item | |
| 42979 | 515 Format the documentation string so that it fits in an Emacs window on an |
| 516 80-column screen. It is a good idea for most lines to be no wider than | |
| 517 60 characters. The first line should not be wider than 67 characters | |
| 518 or it will look bad in the output of @code{apropos}. | |
| 519 | |
| 520 You can fill the text if that looks good. However, rather than blindly | |
| 521 filling the entire documentation string, you can often make it much more | |
| 522 readable by choosing certain line breaks with care. Use blank lines | |
| 523 between topics if the documentation string is long. | |
| 524 | |
| 525 @item | |
| 6552 | 526 The first line of the documentation string should consist of one or two |
|
7601
c5927c75b2b5
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6959
diff
changeset
|
527 complete sentences that stand on their own as a summary. @kbd{M-x |
|
26696
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
528 apropos} displays just the first line, and if that line's contents don't |
|
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
529 stand on their own, the result looks bad. In particular, start the |
|
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
530 first line with a capital letter and end with a period. |
| 6552 | 531 |
|
40970
fb31125d0d7b
Say what questions first line of doc string should answer.
Richard M. Stallman <rms@gnu.org>
parents:
40561
diff
changeset
|
532 For a function, the first line should briefly answer the question, |
|
fb31125d0d7b
Say what questions first line of doc string should answer.
Richard M. Stallman <rms@gnu.org>
parents:
40561
diff
changeset
|
533 ``What does this function do?'' For a variable, the first line should |
|
fb31125d0d7b
Say what questions first line of doc string should answer.
Richard M. Stallman <rms@gnu.org>
parents:
40561
diff
changeset
|
534 briefly answer the question, ``What does this value mean?'' |
|
fb31125d0d7b
Say what questions first line of doc string should answer.
Richard M. Stallman <rms@gnu.org>
parents:
40561
diff
changeset
|
535 |
|
fb31125d0d7b
Say what questions first line of doc string should answer.
Richard M. Stallman <rms@gnu.org>
parents:
40561
diff
changeset
|
536 Don't limit the documentation string to one line; use as many lines as |
|
fb31125d0d7b
Say what questions first line of doc string should answer.
Richard M. Stallman <rms@gnu.org>
parents:
40561
diff
changeset
|
537 you need to explain the details of how to use the function or |
|
fb31125d0d7b
Say what questions first line of doc string should answer.
Richard M. Stallman <rms@gnu.org>
parents:
40561
diff
changeset
|
538 variable. Please use complete sentences for the rest of the text too. |
| 6552 | 539 |
| 540 @item | |
|
49960
10a8aba3dfc7
First line of function doc string should mention the most important
Richard M. Stallman <rms@gnu.org>
parents:
49600
diff
changeset
|
541 The first line should mention all the important arguments of the |
|
10a8aba3dfc7
First line of function doc string should mention the most important
Richard M. Stallman <rms@gnu.org>
parents:
49600
diff
changeset
|
542 function, and should mention them in the order that they are written |
|
10a8aba3dfc7
First line of function doc string should mention the most important
Richard M. Stallman <rms@gnu.org>
parents:
49600
diff
changeset
|
543 in a function call. If the function has many arguments, then it is |
|
10a8aba3dfc7
First line of function doc string should mention the most important
Richard M. Stallman <rms@gnu.org>
parents:
49600
diff
changeset
|
544 not feasible to mention them all in the first line; in that case, the |
|
10a8aba3dfc7
First line of function doc string should mention the most important
Richard M. Stallman <rms@gnu.org>
parents:
49600
diff
changeset
|
545 first line should mention the first few arguments, including the most |
|
10a8aba3dfc7
First line of function doc string should mention the most important
Richard M. Stallman <rms@gnu.org>
parents:
49600
diff
changeset
|
546 important arguments. |
|
10a8aba3dfc7
First line of function doc string should mention the most important
Richard M. Stallman <rms@gnu.org>
parents:
49600
diff
changeset
|
547 |
|
10a8aba3dfc7
First line of function doc string should mention the most important
Richard M. Stallman <rms@gnu.org>
parents:
49600
diff
changeset
|
548 @item |
|
25751
467b88fab665
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
24862
diff
changeset
|
549 For consistency, phrase the verb in the first sentence of a function's |
|
54124
17dfaa4af669
(Documentation Tips): Fix typo.
Luc Teirlinck <teirllm@auburn.edu>
parents:
54042
diff
changeset
|
550 documentation string as an imperative---for instance, use ``Return the |
|
25751
467b88fab665
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
24862
diff
changeset
|
551 cons of A and B.'' in preference to ``Returns the cons of A and B@.'' |
|
467b88fab665
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
24862
diff
changeset
|
552 Usually it looks good to do likewise for the rest of the first |
|
467b88fab665
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
24862
diff
changeset
|
553 paragraph. Subsequent paragraphs usually look better if each sentence |
| 42979 | 554 is indicative and has a proper subject. |
|
6959
3b19456b877a
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6552
diff
changeset
|
555 |
|
3b19456b877a
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6552
diff
changeset
|
556 @item |
| 6552 | 557 Write documentation strings in the active voice, not the passive, and in |
| 558 the present tense, not the future. For instance, use ``Return a list | |
| 559 containing A and B.'' instead of ``A list containing A and B will be | |
| 560 returned.'' | |
| 561 | |
| 562 @item | |
| 563 Avoid using the word ``cause'' (or its equivalents) unnecessarily. | |
| 564 Instead of, ``Cause Emacs to display text in boldface,'' write just | |
| 565 ``Display text in boldface.'' | |
| 566 | |
| 567 @item | |
|
26696
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
568 When a command is meaningful only in a certain mode or situation, |
|
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
569 do mention that in the documentation string. For example, |
|
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
570 the documentation of @code{dired-find-file} is: |
|
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
571 |
|
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
572 @example |
|
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
573 In Dired, visit the file or directory named on this line. |
|
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
574 @end example |
|
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
575 |
|
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
576 @item |
| 6552 | 577 Do not start or end a documentation string with whitespace. |
|
49600
23a1cea22d13
Trailing whitespace deleted.
Juanma Barranquero <lekktu@gmail.com>
parents:
44733
diff
changeset
|
578 |
| 6552 | 579 @item |
| 580 @strong{Do not} indent subsequent lines of a documentation string so | |
| 581 that the text is lined up in the source code with the text of the first | |
| 582 line. This looks nice in the source code, but looks bizarre when users | |
| 583 view the documentation. Remember that the indentation before the | |
| 584 starting double-quote is not part of the string! | |
| 585 | |
| 586 @item | |
|
16671
9fa09185bca0
Explain how disabled commands' doc strings are displayed.
Richard M. Stallman <rms@gnu.org>
parents:
15198
diff
changeset
|
587 When the user tries to use a disabled command, Emacs displays just the |
|
9fa09185bca0
Explain how disabled commands' doc strings are displayed.
Richard M. Stallman <rms@gnu.org>
parents:
15198
diff
changeset
|
588 first paragraph of its documentation string---everything through the |
|
9fa09185bca0
Explain how disabled commands' doc strings are displayed.
Richard M. Stallman <rms@gnu.org>
parents:
15198
diff
changeset
|
589 first blank line. If you wish, you can choose which information to |
|
9fa09185bca0
Explain how disabled commands' doc strings are displayed.
Richard M. Stallman <rms@gnu.org>
parents:
15198
diff
changeset
|
590 include before the first blank line so as to make this display useful. |
|
9fa09185bca0
Explain how disabled commands' doc strings are displayed.
Richard M. Stallman <rms@gnu.org>
parents:
15198
diff
changeset
|
591 |
|
9fa09185bca0
Explain how disabled commands' doc strings are displayed.
Richard M. Stallman <rms@gnu.org>
parents:
15198
diff
changeset
|
592 @item |
| 6552 | 593 A variable's documentation string should start with @samp{*} if the |
|
6959
3b19456b877a
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6552
diff
changeset
|
594 variable is one that users would often want to set interactively. If |
|
7601
c5927c75b2b5
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6959
diff
changeset
|
595 the value is a long list, or a function, or if the variable would be set |
|
c5927c75b2b5
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6959
diff
changeset
|
596 only in init files, then don't start the documentation string with |
| 6552 | 597 @samp{*}. @xref{Defining Variables}. |
| 598 | |
| 599 @item | |
| 600 The documentation string for a variable that is a yes-or-no flag should | |
|
6959
3b19456b877a
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6552
diff
changeset
|
601 start with words such as ``Non-nil means@dots{}'', to make it clear that |
|
3b19456b877a
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6552
diff
changeset
|
602 all non-@code{nil} values are equivalent and indicate explicitly what |
|
3b19456b877a
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6552
diff
changeset
|
603 @code{nil} and non-@code{nil} mean. |
| 6552 | 604 |
| 605 @item | |
|
40517
37a08b5afded
Document textual convention for doc strings of predicates.
Richard M. Stallman <rms@gnu.org>
parents:
38344
diff
changeset
|
606 The documentation string for a function that is a yes-or-no predicate |
|
37a08b5afded
Document textual convention for doc strings of predicates.
Richard M. Stallman <rms@gnu.org>
parents:
38344
diff
changeset
|
607 should start with words such as ``Return t if @dots{}'', to indicate |
|
37a08b5afded
Document textual convention for doc strings of predicates.
Richard M. Stallman <rms@gnu.org>
parents:
38344
diff
changeset
|
608 explicitly what constitutes ``truth''. The word ``return'' avoids |
|
37a08b5afded
Document textual convention for doc strings of predicates.
Richard M. Stallman <rms@gnu.org>
parents:
38344
diff
changeset
|
609 starting the sentence with lower-case ``t'', which is somewhat |
|
37a08b5afded
Document textual convention for doc strings of predicates.
Richard M. Stallman <rms@gnu.org>
parents:
38344
diff
changeset
|
610 distracting. |
|
37a08b5afded
Document textual convention for doc strings of predicates.
Richard M. Stallman <rms@gnu.org>
parents:
38344
diff
changeset
|
611 |
|
37a08b5afded
Document textual convention for doc strings of predicates.
Richard M. Stallman <rms@gnu.org>
parents:
38344
diff
changeset
|
612 @item |
| 6552 | 613 When a function's documentation string mentions the value of an argument |
| 614 of the function, use the argument name in capital letters as if it were | |
| 615 a name for that value. Thus, the documentation string of the function | |
|
26696
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
616 @code{eval} refers to its second argument as @samp{FORM}, because the |
|
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
617 actual argument name is @code{form}: |
| 6552 | 618 |
|
26696
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
619 @example |
|
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
620 Evaluate FORM and return its value. |
|
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
621 @end example |
|
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
622 |
|
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
623 Also write metasyntactic variables in capital letters, such as when you |
|
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
624 show the decomposition of a list or vector into subunits, some of which |
|
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
625 may vary. @samp{KEY} and @samp{VALUE} in the following example |
|
25751
467b88fab665
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
24862
diff
changeset
|
626 illustrate this practice: |
|
467b88fab665
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
24862
diff
changeset
|
627 |
|
467b88fab665
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
24862
diff
changeset
|
628 @example |
|
467b88fab665
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
24862
diff
changeset
|
629 The argument TABLE should be an alist whose elements |
|
467b88fab665
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
24862
diff
changeset
|
630 have the form (KEY . VALUE). Here, KEY is ... |
|
467b88fab665
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
24862
diff
changeset
|
631 @end example |
| 6552 | 632 |
| 633 @item | |
|
40517
37a08b5afded
Document textual convention for doc strings of predicates.
Richard M. Stallman <rms@gnu.org>
parents:
38344
diff
changeset
|
634 Never change the case of a Lisp symbol when you mention it in a doc |
|
37a08b5afded
Document textual convention for doc strings of predicates.
Richard M. Stallman <rms@gnu.org>
parents:
38344
diff
changeset
|
635 string. If the symbol's name is @code{foo}, write ``foo'', not |
|
37a08b5afded
Document textual convention for doc strings of predicates.
Richard M. Stallman <rms@gnu.org>
parents:
38344
diff
changeset
|
636 ``Foo'' (which is a different symbol). |
|
37a08b5afded
Document textual convention for doc strings of predicates.
Richard M. Stallman <rms@gnu.org>
parents:
38344
diff
changeset
|
637 |
|
37a08b5afded
Document textual convention for doc strings of predicates.
Richard M. Stallman <rms@gnu.org>
parents:
38344
diff
changeset
|
638 This might appear to contradict the policy of writing function |
|
37a08b5afded
Document textual convention for doc strings of predicates.
Richard M. Stallman <rms@gnu.org>
parents:
38344
diff
changeset
|
639 argument values, but there is no real contradiction; the argument |
|
37a08b5afded
Document textual convention for doc strings of predicates.
Richard M. Stallman <rms@gnu.org>
parents:
38344
diff
changeset
|
640 @emph{value} is not the same thing as the @emph{symbol} which the |
|
37a08b5afded
Document textual convention for doc strings of predicates.
Richard M. Stallman <rms@gnu.org>
parents:
38344
diff
changeset
|
641 function uses to hold the value. |
|
37a08b5afded
Document textual convention for doc strings of predicates.
Richard M. Stallman <rms@gnu.org>
parents:
38344
diff
changeset
|
642 |
|
37a08b5afded
Document textual convention for doc strings of predicates.
Richard M. Stallman <rms@gnu.org>
parents:
38344
diff
changeset
|
643 If this puts a lower-case letter at the beginning of a sentence |
|
37a08b5afded
Document textual convention for doc strings of predicates.
Richard M. Stallman <rms@gnu.org>
parents:
38344
diff
changeset
|
644 and that annoys you, rewrite the sentence so that the symbol |
|
37a08b5afded
Document textual convention for doc strings of predicates.
Richard M. Stallman <rms@gnu.org>
parents:
38344
diff
changeset
|
645 is not at the start of it. |
|
37a08b5afded
Document textual convention for doc strings of predicates.
Richard M. Stallman <rms@gnu.org>
parents:
38344
diff
changeset
|
646 |
|
37a08b5afded
Document textual convention for doc strings of predicates.
Richard M. Stallman <rms@gnu.org>
parents:
38344
diff
changeset
|
647 @item |
|
26696
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
648 If a line in a documentation string begins with an open-parenthesis, |
|
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
649 write a backslash before the open-parenthesis, like this: |
|
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
650 |
|
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
651 @example |
|
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
652 The argument FOO can be either a number |
|
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
653 \(a buffer position) or a string (a file name). |
|
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
654 @end example |
|
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
655 |
|
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
656 This prevents the open-parenthesis from being treated as the start of a |
|
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
657 defun (@pxref{Defuns,, Defuns, emacs, The GNU Emacs Manual}). |
|
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
658 |
|
53300
2cabf526af0e
(Documentation Tips): Add anchor.
Luc Teirlinck <teirllm@auburn.edu>
parents:
52912
diff
changeset
|
659 @anchor{Docstring hyperlinks} |
|
26696
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
660 @item |
| 6552 | 661 @iftex |
| 662 When a documentation string refers to a Lisp symbol, write it as it | |
| 663 would be printed (which usually means in lower case), with single-quotes | |
| 664 around it. For example: @samp{`lambda'}. There are two exceptions: | |
| 665 write @code{t} and @code{nil} without single-quotes. | |
| 666 @end iftex | |
| 27193 | 667 @ifnottex |
| 6552 | 668 When a documentation string refers to a Lisp symbol, write it as it |
| 669 would be printed (which usually means in lower case), with single-quotes | |
| 670 around it. For example: @samp{lambda}. There are two exceptions: write | |
|
21682
90da2489c498
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21007
diff
changeset
|
671 t and nil without single-quotes. (In this manual, we use a different |
|
90da2489c498
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21007
diff
changeset
|
672 convention, with single-quotes for all symbols.) |
| 27193 | 673 @end ifnottex |
| 6552 | 674 |
|
22252
40089afa2b1d
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
22138
diff
changeset
|
675 Help mode automatically creates a hyperlink when a documentation string |
|
40089afa2b1d
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
22138
diff
changeset
|
676 uses a symbol name inside single quotes, if the symbol has either a |
|
22138
d4ac295a98b3
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21682
diff
changeset
|
677 function or a variable definition. You do not need to do anything |
|
d4ac295a98b3
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21682
diff
changeset
|
678 special to make use of this feature. However, when a symbol has both a |
|
d4ac295a98b3
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21682
diff
changeset
|
679 function definition and a variable definition, and you want to refer to |
|
d4ac295a98b3
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21682
diff
changeset
|
680 just one of them, you can specify which one by writing one of the words |
|
d4ac295a98b3
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21682
diff
changeset
|
681 @samp{variable}, @samp{option}, @samp{function}, or @samp{command}, |
|
d4ac295a98b3
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21682
diff
changeset
|
682 immediately before the symbol name. (Case makes no difference in |
|
d4ac295a98b3
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21682
diff
changeset
|
683 recognizing these indicator words.) For example, if you write |
|
21682
90da2489c498
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21007
diff
changeset
|
684 |
|
90da2489c498
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21007
diff
changeset
|
685 @example |
|
22138
d4ac295a98b3
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21682
diff
changeset
|
686 This function sets the variable `buffer-file-name'. |
|
21682
90da2489c498
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21007
diff
changeset
|
687 @end example |
|
90da2489c498
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21007
diff
changeset
|
688 |
|
22138
d4ac295a98b3
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21682
diff
changeset
|
689 @noindent |
|
d4ac295a98b3
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21682
diff
changeset
|
690 then the hyperlink will refer only to the variable documentation of |
|
d4ac295a98b3
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21682
diff
changeset
|
691 @code{buffer-file-name}, and not to its function documentation. |
|
d4ac295a98b3
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21682
diff
changeset
|
692 |
|
d4ac295a98b3
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21682
diff
changeset
|
693 If a symbol has a function definition and/or a variable definition, but |
|
d4ac295a98b3
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21682
diff
changeset
|
694 those are irrelevant to the use of the symbol that you are documenting, |
|
d4ac295a98b3
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21682
diff
changeset
|
695 you can write the word @samp{symbol} before the symbol name to prevent |
|
d4ac295a98b3
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21682
diff
changeset
|
696 making any hyperlink. For example, |
|
d4ac295a98b3
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21682
diff
changeset
|
697 |
|
d4ac295a98b3
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21682
diff
changeset
|
698 @example |
|
d4ac295a98b3
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21682
diff
changeset
|
699 If the argument KIND-OF-RESULT is the symbol `list', |
|
d4ac295a98b3
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21682
diff
changeset
|
700 this function returns a list of all the objects |
|
d4ac295a98b3
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21682
diff
changeset
|
701 that satisfy the criterion. |
|
d4ac295a98b3
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21682
diff
changeset
|
702 @end example |
|
d4ac295a98b3
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21682
diff
changeset
|
703 |
|
d4ac295a98b3
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21682
diff
changeset
|
704 @noindent |
|
d4ac295a98b3
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21682
diff
changeset
|
705 does not make a hyperlink to the documentation, irrelevant here, of the |
|
d4ac295a98b3
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21682
diff
changeset
|
706 function @code{list}. |
|
d4ac295a98b3
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21682
diff
changeset
|
707 |
|
52912
2fdb8b16bceb
(Documentation Tips): Document new behavior for face and variable
Luc Teirlinck <teirllm@auburn.edu>
parents:
52845
diff
changeset
|
708 Normally, no hyperlink is made for a variable without variable |
|
2fdb8b16bceb
(Documentation Tips): Document new behavior for face and variable
Luc Teirlinck <teirllm@auburn.edu>
parents:
52845
diff
changeset
|
709 documentation. You can force a hyperlink for such variables by |
|
2fdb8b16bceb
(Documentation Tips): Document new behavior for face and variable
Luc Teirlinck <teirllm@auburn.edu>
parents:
52845
diff
changeset
|
710 preceding them with one of the words @samp{variable} or |
|
2fdb8b16bceb
(Documentation Tips): Document new behavior for face and variable
Luc Teirlinck <teirllm@auburn.edu>
parents:
52845
diff
changeset
|
711 @samp{option}. |
|
2fdb8b16bceb
(Documentation Tips): Document new behavior for face and variable
Luc Teirlinck <teirllm@auburn.edu>
parents:
52845
diff
changeset
|
712 |
|
2fdb8b16bceb
(Documentation Tips): Document new behavior for face and variable
Luc Teirlinck <teirllm@auburn.edu>
parents:
52845
diff
changeset
|
713 Hyperlinks for faces are only made if the face name is preceded or |
|
2fdb8b16bceb
(Documentation Tips): Document new behavior for face and variable
Luc Teirlinck <teirllm@auburn.edu>
parents:
52845
diff
changeset
|
714 followed by the word @samp{face}. In that case, only the face |
|
2fdb8b16bceb
(Documentation Tips): Document new behavior for face and variable
Luc Teirlinck <teirllm@auburn.edu>
parents:
52845
diff
changeset
|
715 documentation will be shown, even if the symbol is also defined as a |
|
2fdb8b16bceb
(Documentation Tips): Document new behavior for face and variable
Luc Teirlinck <teirllm@auburn.edu>
parents:
52845
diff
changeset
|
716 variable or as a function. |
|
2fdb8b16bceb
(Documentation Tips): Document new behavior for face and variable
Luc Teirlinck <teirllm@auburn.edu>
parents:
52845
diff
changeset
|
717 |
|
25751
467b88fab665
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
24862
diff
changeset
|
718 To make a hyperlink to Info documentation, write the name of the Info |
|
53457
3bd5e3d6a230
(Documentation Tips): Update item on hyperlinks in documentation strings.
Luc Teirlinck <teirllm@auburn.edu>
parents:
53300
diff
changeset
|
719 node (or anchor) in single quotes, preceded by @samp{info node}, |
|
3bd5e3d6a230
(Documentation Tips): Update item on hyperlinks in documentation strings.
Luc Teirlinck <teirllm@auburn.edu>
parents:
53300
diff
changeset
|
720 @samp{Info node}, @samp{info anchor} or @samp{Info anchor}. The Info |
|
3bd5e3d6a230
(Documentation Tips): Update item on hyperlinks in documentation strings.
Luc Teirlinck <teirllm@auburn.edu>
parents:
53300
diff
changeset
|
721 file name defaults to @samp{emacs}. For example, |
|
25751
467b88fab665
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
24862
diff
changeset
|
722 |
|
467b88fab665
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
24862
diff
changeset
|
723 @smallexample |
|
467b88fab665
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
24862
diff
changeset
|
724 See Info node `Font Lock' and Info node `(elisp)Font Lock Basics'. |
|
467b88fab665
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
24862
diff
changeset
|
725 @end smallexample |
|
467b88fab665
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
24862
diff
changeset
|
726 |
| 6552 | 727 @item |
| 728 Don't write key sequences directly in documentation strings. Instead, | |
| 729 use the @samp{\\[@dots{}]} construct to stand for them. For example, | |
|
16736
981e116b4ac6
Minor cleanups for overfull hboxes.
Richard M. Stallman <rms@gnu.org>
parents:
16671
diff
changeset
|
730 instead of writing @samp{C-f}, write the construct |
|
981e116b4ac6
Minor cleanups for overfull hboxes.
Richard M. Stallman <rms@gnu.org>
parents:
16671
diff
changeset
|
731 @samp{\\[forward-char]}. When Emacs displays the documentation string, |
|
981e116b4ac6
Minor cleanups for overfull hboxes.
Richard M. Stallman <rms@gnu.org>
parents:
16671
diff
changeset
|
732 it substitutes whatever key is currently bound to @code{forward-char}. |
|
981e116b4ac6
Minor cleanups for overfull hboxes.
Richard M. Stallman <rms@gnu.org>
parents:
16671
diff
changeset
|
733 (This is normally @samp{C-f}, but it may be some other character if the |
|
981e116b4ac6
Minor cleanups for overfull hboxes.
Richard M. Stallman <rms@gnu.org>
parents:
16671
diff
changeset
|
734 user has moved key bindings.) @xref{Keys in Documentation}. |
| 6552 | 735 |
| 736 @item | |
| 737 In documentation strings for a major mode, you will want to refer to the | |
| 738 key bindings of that mode's local map, rather than global ones. | |
| 739 Therefore, use the construct @samp{\\<@dots{}>} once in the | |
| 740 documentation string to specify which key map to use. Do this before | |
| 741 the first use of @samp{\\[@dots{}]}. The text inside the | |
| 742 @samp{\\<@dots{}>} should be the name of the variable containing the | |
| 743 local keymap for the major mode. | |
| 744 | |
| 745 It is not practical to use @samp{\\[@dots{}]} very many times, because | |
| 746 display of the documentation string will become slow. So use this to | |
| 747 describe the most important commands in your major mode, and then use | |
| 748 @samp{\\@{@dots{}@}} to display the rest of the mode's keymap. | |
| 749 @end itemize | |
| 750 | |
| 751 @node Comment Tips | |
| 752 @section Tips on Writing Comments | |
| 753 | |
| 754 We recommend these conventions for where to put comments and how to | |
| 755 indent them: | |
| 756 | |
| 757 @table @samp | |
| 758 @item ; | |
| 759 Comments that start with a single semicolon, @samp{;}, should all be | |
| 760 aligned to the same column on the right of the source code. Such | |
| 761 comments usually explain how the code on the same line does its job. In | |
| 762 Lisp mode and related modes, the @kbd{M-;} (@code{indent-for-comment}) | |
| 763 command automatically inserts such a @samp{;} in the right place, or | |
|
6959
3b19456b877a
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6552
diff
changeset
|
764 aligns such a comment if it is already present. |
| 6552 | 765 |
|
7601
c5927c75b2b5
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6959
diff
changeset
|
766 This and following examples are taken from the Emacs sources. |
| 6552 | 767 |
| 768 @smallexample | |
| 769 @group | |
| 770 (setq base-version-list ; there was a base | |
| 771 (assoc (substring fn 0 start-vn) ; version to which | |
| 772 file-version-assoc-list)) ; this looks like | |
| 773 ; a subversion | |
| 774 @end group | |
| 775 @end smallexample | |
| 776 | |
| 777 @item ;; | |
| 778 Comments that start with two semicolons, @samp{;;}, should be aligned to | |
|
6959
3b19456b877a
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6552
diff
changeset
|
779 the same level of indentation as the code. Such comments usually |
| 6552 | 780 describe the purpose of the following lines or the state of the program |
| 781 at that point. For example: | |
| 782 | |
| 783 @smallexample | |
| 784 @group | |
| 785 (prog1 (setq auto-fill-function | |
| 786 @dots{} | |
| 787 @dots{} | |
|
6959
3b19456b877a
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6552
diff
changeset
|
788 ;; update mode line |
| 6552 | 789 (force-mode-line-update))) |
| 790 @end group | |
| 791 @end smallexample | |
| 792 | |
|
26696
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
793 We also normally use two semicolons for comments outside functions. |
|
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
794 |
|
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
795 @smallexample |
|
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
796 @group |
|
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
797 ;; This Lisp code is run in Emacs |
|
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
798 ;; when it is to operate as a server |
|
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
799 ;; for other processes. |
|
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
800 @end group |
|
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
801 @end smallexample |
|
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
802 |
|
21682
90da2489c498
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21007
diff
changeset
|
803 Every function that has no documentation string (presumably one that is |
|
26696
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
804 used only internally within the package it belongs to), should instead |
|
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
805 have a two-semicolon comment right before the function, explaining what |
|
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
806 the function does and how to call it properly. Explain precisely what |
|
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
807 each argument means and how the function interprets its possible values. |
| 6552 | 808 |
| 809 @item ;;; | |
| 810 Comments that start with three semicolons, @samp{;;;}, should start at | |
|
26696
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
811 the left margin. These are used, occasionally, for comments within |
|
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
812 functions that should start at the margin. We also use them sometimes |
|
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
813 for comments that are between functions---whether to use two or three |
|
53924
ed40e77a5176
(Comment Tips): Document the new conventions for commenting out code.
Luc Teirlinck <teirllm@auburn.edu>
parents:
53457
diff
changeset
|
814 semicolons depends on whether the comment should be considered a |
|
ed40e77a5176
(Comment Tips): Document the new conventions for commenting out code.
Luc Teirlinck <teirllm@auburn.edu>
parents:
53457
diff
changeset
|
815 ``heading'' by Outline minor mode. By default, comments starting with |
|
ed40e77a5176
(Comment Tips): Document the new conventions for commenting out code.
Luc Teirlinck <teirllm@auburn.edu>
parents:
53457
diff
changeset
|
816 at least three semicolons (followed by a single space and a |
|
ed40e77a5176
(Comment Tips): Document the new conventions for commenting out code.
Luc Teirlinck <teirllm@auburn.edu>
parents:
53457
diff
changeset
|
817 non-whitespace character) are considered headings, comments starting |
|
ed40e77a5176
(Comment Tips): Document the new conventions for commenting out code.
Luc Teirlinck <teirllm@auburn.edu>
parents:
53457
diff
changeset
|
818 with two or less are not. |
| 6552 | 819 |
|
7601
c5927c75b2b5
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6959
diff
changeset
|
820 Another use for triple-semicolon comments is for commenting out lines |
|
26696
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
821 within a function. We use three semicolons for this precisely so that |
|
53924
ed40e77a5176
(Comment Tips): Document the new conventions for commenting out code.
Luc Teirlinck <teirllm@auburn.edu>
parents:
53457
diff
changeset
|
822 they remain at the left margin. By default, Outline minor mode does |
|
ed40e77a5176
(Comment Tips): Document the new conventions for commenting out code.
Luc Teirlinck <teirllm@auburn.edu>
parents:
53457
diff
changeset
|
823 not consider a comment to be a heading (even if it starts with at |
|
ed40e77a5176
(Comment Tips): Document the new conventions for commenting out code.
Luc Teirlinck <teirllm@auburn.edu>
parents:
53457
diff
changeset
|
824 least three semicolons) if the semicolons are followed by at least two |
|
ed40e77a5176
(Comment Tips): Document the new conventions for commenting out code.
Luc Teirlinck <teirllm@auburn.edu>
parents:
53457
diff
changeset
|
825 spaces. Thus, if you add an introductory comment to the commented out |
|
ed40e77a5176
(Comment Tips): Document the new conventions for commenting out code.
Luc Teirlinck <teirllm@auburn.edu>
parents:
53457
diff
changeset
|
826 code, make sure to indent it by at least two spaces after the three |
|
ed40e77a5176
(Comment Tips): Document the new conventions for commenting out code.
Luc Teirlinck <teirllm@auburn.edu>
parents:
53457
diff
changeset
|
827 semicolons. |
|
6959
3b19456b877a
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6552
diff
changeset
|
828 |
|
3b19456b877a
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6552
diff
changeset
|
829 @smallexample |
|
3b19456b877a
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6552
diff
changeset
|
830 (defun foo (a) |
|
53924
ed40e77a5176
(Comment Tips): Document the new conventions for commenting out code.
Luc Teirlinck <teirllm@auburn.edu>
parents:
53457
diff
changeset
|
831 ;;; This is no longer necessary. |
|
6959
3b19456b877a
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6552
diff
changeset
|
832 ;;; (force-mode-line-update) |
|
3b19456b877a
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6552
diff
changeset
|
833 (message "Finished with %s" a)) |
|
3b19456b877a
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6552
diff
changeset
|
834 @end smallexample |
|
3b19456b877a
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6552
diff
changeset
|
835 |
|
53924
ed40e77a5176
(Comment Tips): Document the new conventions for commenting out code.
Luc Teirlinck <teirllm@auburn.edu>
parents:
53457
diff
changeset
|
836 When commenting out entire functions, use two semicolons. |
|
ed40e77a5176
(Comment Tips): Document the new conventions for commenting out code.
Luc Teirlinck <teirllm@auburn.edu>
parents:
53457
diff
changeset
|
837 |
| 6552 | 838 @item ;;;; |
| 839 Comments that start with four semicolons, @samp{;;;;}, should be aligned | |
| 840 to the left margin and are used for headings of major sections of a | |
| 841 program. For example: | |
| 842 | |
| 843 @smallexample | |
| 844 ;;;; The kill ring | |
| 845 @end smallexample | |
| 846 @end table | |
| 847 | |
| 848 @noindent | |
| 849 The indentation commands of the Lisp modes in Emacs, such as @kbd{M-;} | |
|
21682
90da2489c498
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21007
diff
changeset
|
850 (@code{indent-for-comment}) and @key{TAB} (@code{lisp-indent-line}), |
| 6552 | 851 automatically indent comments according to these conventions, |
|
7601
c5927c75b2b5
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6959
diff
changeset
|
852 depending on the number of semicolons. @xref{Comments,, |
| 6552 | 853 Manipulating Comments, emacs, The GNU Emacs Manual}. |
| 854 | |
| 855 @node Library Headers | |
| 856 @section Conventional Headers for Emacs Libraries | |
| 857 @cindex header comments | |
| 858 @cindex library header comments | |
| 859 | |
|
21007
66d807bdc5b4
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
19361
diff
changeset
|
860 Emacs has conventions for using special comments in Lisp libraries |
| 6552 | 861 to divide them into sections and give information such as who wrote |
|
25751
467b88fab665
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
24862
diff
changeset
|
862 them. This section explains these conventions. |
|
467b88fab665
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
24862
diff
changeset
|
863 |
|
467b88fab665
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
24862
diff
changeset
|
864 We'll start with an example, a package that is included in the Emacs |
|
467b88fab665
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
24862
diff
changeset
|
865 distribution. |
|
467b88fab665
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
24862
diff
changeset
|
866 |
|
467b88fab665
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
24862
diff
changeset
|
867 Parts of this example reflect its status as part of Emacs; for |
|
467b88fab665
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
24862
diff
changeset
|
868 example, the copyright notice lists the Free Software Foundation as the |
|
467b88fab665
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
24862
diff
changeset
|
869 copyright holder, and the copying permission says the file is part of |
|
467b88fab665
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
24862
diff
changeset
|
870 Emacs. When you write a package and post it, the copyright holder would |
|
467b88fab665
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
24862
diff
changeset
|
871 be you (unless your employer claims to own it instead), and you should |
|
467b88fab665
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
24862
diff
changeset
|
872 get the suggested copying permission from the end of the GNU General |
|
467b88fab665
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
24862
diff
changeset
|
873 Public License itself. Don't say your file is part of Emacs |
|
467b88fab665
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
24862
diff
changeset
|
874 if we haven't installed it in Emacs yet! |
|
467b88fab665
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
24862
diff
changeset
|
875 |
|
467b88fab665
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
24862
diff
changeset
|
876 With that warning out of the way, on to the example: |
| 6552 | 877 |
| 878 @smallexample | |
| 879 @group | |
| 880 ;;; lisp-mnt.el --- minor mode for Emacs Lisp maintainers | |
| 881 | |
| 882 ;; Copyright (C) 1992 Free Software Foundation, Inc. | |
| 883 @end group | |
| 884 | |
| 885 ;; Author: Eric S. Raymond <esr@@snark.thyrsus.com> | |
| 886 ;; Maintainer: Eric S. Raymond <esr@@snark.thyrsus.com> | |
| 887 ;; Created: 14 Jul 1992 | |
| 888 ;; Version: 1.2 | |
| 889 @group | |
| 890 ;; Keywords: docs | |
| 891 | |
| 892 ;; This file is part of GNU Emacs. | |
|
21682
90da2489c498
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21007
diff
changeset
|
893 @dots{} |
|
90da2489c498
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21007
diff
changeset
|
894 ;; Free Software Foundation, Inc., 59 Temple Place - Suite 330, |
|
90da2489c498
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21007
diff
changeset
|
895 ;; Boston, MA 02111-1307, USA. |
| 6552 | 896 @end group |
| 897 @end smallexample | |
| 898 | |
| 899 The very first line should have this format: | |
| 900 | |
| 901 @example | |
| 902 ;;; @var{filename} --- @var{description} | |
| 903 @end example | |
| 904 | |
| 905 @noindent | |
|
52149
9928a29a3e80
(Library Headers): Explain where to put -*-.
Richard M. Stallman <rms@gnu.org>
parents:
49960
diff
changeset
|
906 The description should be complete in one line. If the file |
|
9928a29a3e80
(Library Headers): Explain where to put -*-.
Richard M. Stallman <rms@gnu.org>
parents:
49960
diff
changeset
|
907 needs a @samp{-*-} specification, put it after @var{description}. |
| 6552 | 908 |
| 909 After the copyright notice come several @dfn{header comment} lines, | |
|
6959
3b19456b877a
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6552
diff
changeset
|
910 each beginning with @samp{;; @var{header-name}:}. Here is a table of |
| 6552 | 911 the conventional possibilities for @var{header-name}: |
| 912 | |
| 913 @table @samp | |
| 914 @item Author | |
| 915 This line states the name and net address of at least the principal | |
| 916 author of the library. | |
| 917 | |
| 918 If there are multiple authors, you can list them on continuation lines | |
|
6959
3b19456b877a
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6552
diff
changeset
|
919 led by @code{;;} and a tab character, like this: |
| 6552 | 920 |
| 921 @smallexample | |
| 922 @group | |
| 923 ;; Author: Ashwin Ram <Ram-Ashwin@@cs.yale.edu> | |
|
6959
3b19456b877a
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6552
diff
changeset
|
924 ;; Dave Sill <de5@@ornl.gov> |
|
3b19456b877a
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6552
diff
changeset
|
925 ;; Dave Brennan <brennan@@hal.com> |
|
3b19456b877a
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6552
diff
changeset
|
926 ;; Eric Raymond <esr@@snark.thyrsus.com> |
| 6552 | 927 @end group |
| 928 @end smallexample | |
| 929 | |
| 930 @item Maintainer | |
| 931 This line should contain a single name/address as in the Author line, or | |
|
6959
3b19456b877a
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6552
diff
changeset
|
932 an address only, or the string @samp{FSF}. If there is no maintainer |
|
3b19456b877a
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6552
diff
changeset
|
933 line, the person(s) in the Author field are presumed to be the |
|
3b19456b877a
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6552
diff
changeset
|
934 maintainers. The example above is mildly bogus because the maintainer |
|
3b19456b877a
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
6552
diff
changeset
|
935 line is redundant. |
| 6552 | 936 |
| 937 The idea behind the @samp{Author} and @samp{Maintainer} lines is to make | |
| 938 possible a Lisp function to ``send mail to the maintainer'' without | |
| 939 having to mine the name out by hand. | |
| 940 | |
| 941 Be sure to surround the network address with @samp{<@dots{}>} if | |
| 942 you include the person's full name as well as the network address. | |
| 943 | |
| 944 @item Created | |
| 945 This optional line gives the original creation date of the | |
| 946 file. For historical interest only. | |
| 947 | |
| 948 @item Version | |
| 949 If you wish to record version numbers for the individual Lisp program, put | |
| 950 them in this line. | |
| 951 | |
| 952 @item Adapted-By | |
| 953 In this header line, place the name of the person who adapted the | |
| 954 library for installation (to make it fit the style conventions, for | |
| 955 example). | |
| 956 | |
| 957 @item Keywords | |
| 958 This line lists keywords for the @code{finder-by-keyword} help command. | |
|
22138
d4ac295a98b3
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21682
diff
changeset
|
959 Please use that command to see a list of the meaningful keywords. |
|
d4ac295a98b3
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21682
diff
changeset
|
960 |
| 6552 | 961 This field is important; it's how people will find your package when |
|
10229
634f36d4b2ae
Give syntax of keywords.
Richard M. Stallman <rms@gnu.org>
parents:
10225
diff
changeset
|
962 they're looking for things by topic area. To separate the keywords, you |
|
634f36d4b2ae
Give syntax of keywords.
Richard M. Stallman <rms@gnu.org>
parents:
10225
diff
changeset
|
963 can use spaces, commas, or both. |
| 6552 | 964 @end table |
| 965 | |
| 966 Just about every Lisp library ought to have the @samp{Author} and | |
| 967 @samp{Keywords} header comment lines. Use the others if they are | |
| 968 appropriate. You can also put in header lines with other header | |
| 969 names---they have no standard meanings, so they can't do any harm. | |
| 970 | |
| 971 We use additional stylized comments to subdivide the contents of the | |
|
26696
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
972 library file. These should be separated by blank lines from anything |
|
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
973 else. Here is a table of them: |
| 6552 | 974 |
| 975 @table @samp | |
| 976 @item ;;; Commentary: | |
| 977 This begins introductory comments that explain how the library works. | |
|
22138
d4ac295a98b3
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21682
diff
changeset
|
978 It should come right after the copying permissions, terminated by a |
|
d4ac295a98b3
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21682
diff
changeset
|
979 @samp{Change Log}, @samp{History} or @samp{Code} comment line. This |
|
d4ac295a98b3
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21682
diff
changeset
|
980 text is used by the Finder package, so it should make sense in that |
|
d4ac295a98b3
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21682
diff
changeset
|
981 context. |
| 6552 | 982 |
|
52814
df5527dde8fb
(Library Headers): Fix typo.
Luc Teirlinck <teirllm@auburn.edu>
parents:
52808
diff
changeset
|
983 @item ;;; Documentation: |
|
52845
1826a112381b
(Library Headers): Clean up Documentation.
Richard M. Stallman <rms@gnu.org>
parents:
52814
diff
changeset
|
984 This was used in some files in place of @samp{;;; Commentary:}, |
|
1826a112381b
(Library Headers): Clean up Documentation.
Richard M. Stallman <rms@gnu.org>
parents:
52814
diff
changeset
|
985 but it is deprecated. |
|
22138
d4ac295a98b3
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21682
diff
changeset
|
986 |
|
d4ac295a98b3
*** empty log message ***
Richard M. Stallman <rms@gnu.org>
parents:
21682
diff
changeset
|
987 @item ;;; Change Log: |
| 6552 | 988 This begins change log information stored in the library file (if you |
|
26696
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
989 store the change history there). For Lisp files distributed with Emacs, |
|
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
990 the change history is kept in the file @file{ChangeLog} and not in the |
|
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
991 source file at all; these files generally do not have a @samp{;;; Change |
|
ef5e7bbe6f19
Current version from /gd/gnu/elisp.
Dave Love <fx@gnu.org>
parents:
26211
diff
changeset
|
992 Log:} line. @samp{History} is an alternative to @samp{Change Log}. |
| 6552 | 993 |
| 994 @item ;;; Code: | |
| 995 This begins the actual code of the program. | |
| 996 | |
| 997 @item ;;; @var{filename} ends here | |
| 998 This is the @dfn{footer line}; it appears at the very end of the file. | |
| 999 Its purpose is to enable people to detect truncated versions of the file | |
| 1000 from the lack of a footer line. | |
| 1001 @end table | |
| 52401 | 1002 |
| 1003 @ignore | |
| 1004 arch-tag: 9ea911c2-6b1d-47dd-88b7-0a94e8b27c2e | |
| 1005 @end ignore |
