Mercurial > hg > xemacs-beta
annotate lisp/config.el @ 5753:dbd8305e13cb
Warn about non-string non-integer ARG to #'gensym, bytecomp.el.
lisp/ChangeLog addition:
2013-08-21 Aidan Kehoe <kehoea@parhasard.net>
* bytecomp.el:
* bytecomp.el (gensym):
* bytecomp.el (byte-compile-gensym): New.
Warn that gensym called in a for-effect context is unlikely to be
useful.
Warn about non-string non-integer ARGs, this is incorrect.
Am not changing the function to error with same, most code that
makes the mistake is has no problems, which is why it has survived
so long.
* window-xemacs.el (save-window-excursion/mapping):
* window.el (save-window-excursion):
Call #'gensym with a string, not a symbol.
| author | Aidan Kehoe <kehoea@parhasard.net> |
|---|---|
| date | Wed, 21 Aug 2013 19:02:59 +0100 |
| parents | 308d34e9f07d |
| children |
| rev | line source |
|---|---|
| 428 | 1 ;;; config.el --- access configuration parameters |
| 2 | |
| 3 ;; Copyright (C) 1997 Sun Microsystems, Inc. | |
| 4 | |
| 5 ;; Author: Martin Buchholz | |
| 6 ;; Keywords: configure | |
| 7 | |
| 8 ;; This file is part of XEmacs. | |
| 9 | |
|
5402
308d34e9f07d
Changed bulk of GPLv2 or later files identified by script
Mats Lidell <matsl@xemacs.org>
parents:
428
diff
changeset
|
10 ;; XEmacs is free software: you can redistribute it and/or modify it |
|
308d34e9f07d
Changed bulk of GPLv2 or later files identified by script
Mats Lidell <matsl@xemacs.org>
parents:
428
diff
changeset
|
11 ;; under the terms of the GNU General Public License as published by the |
|
308d34e9f07d
Changed bulk of GPLv2 or later files identified by script
Mats Lidell <matsl@xemacs.org>
parents:
428
diff
changeset
|
12 ;; Free Software Foundation, either version 3 of the License, or (at your |
|
308d34e9f07d
Changed bulk of GPLv2 or later files identified by script
Mats Lidell <matsl@xemacs.org>
parents:
428
diff
changeset
|
13 ;; option) any later version. |
| 428 | 14 |
|
5402
308d34e9f07d
Changed bulk of GPLv2 or later files identified by script
Mats Lidell <matsl@xemacs.org>
parents:
428
diff
changeset
|
15 ;; XEmacs is distributed in the hope that it will be useful, but WITHOUT |
|
308d34e9f07d
Changed bulk of GPLv2 or later files identified by script
Mats Lidell <matsl@xemacs.org>
parents:
428
diff
changeset
|
16 ;; ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or |
|
308d34e9f07d
Changed bulk of GPLv2 or later files identified by script
Mats Lidell <matsl@xemacs.org>
parents:
428
diff
changeset
|
17 ;; FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License |
|
308d34e9f07d
Changed bulk of GPLv2 or later files identified by script
Mats Lidell <matsl@xemacs.org>
parents:
428
diff
changeset
|
18 ;; for more details. |
| 428 | 19 |
| 20 ;; You should have received a copy of the GNU General Public License | |
|
5402
308d34e9f07d
Changed bulk of GPLv2 or later files identified by script
Mats Lidell <matsl@xemacs.org>
parents:
428
diff
changeset
|
21 ;; along with XEmacs. If not, see <http://www.gnu.org/licenses/>. |
| 428 | 22 |
| 23 ;;; Synched up with: not in FSF. | |
| 24 | |
| 25 ;;; Commentary: | |
| 26 | |
| 27 ;;; Code: | |
| 28 | |
| 29 | |
| 30 (defvar config-value-file (expand-file-name "config.values" doc-directory) | |
| 31 "File containing configuration parameters and their values.") | |
| 32 | |
| 33 (defvar config-value-hash-table nil | |
| 34 "Hash table to store configuration parameters and their values.") | |
| 35 | |
| 36 ;;;###autoload | |
| 37 (defun config-value-hash-table () | |
| 38 "Return hash table of configuration parameters and their values." | |
| 39 (when (null config-value-hash-table) | |
| 40 (setq config-value-hash-table (make-hash-table :size 300)) | |
| 41 (save-excursion | |
| 42 (let ((buf (get-buffer-create " *Config*"))) | |
| 43 (set-buffer buf) | |
| 44 (erase-buffer) | |
| 45 (insert-file-contents config-value-file) | |
| 46 (goto-char (point-min)) | |
| 47 (condition-case nil | |
| 48 (while t | |
| 49 (let* ((key (read buf)) | |
| 50 (value (read buf)) | |
| 51 (prev (gethash key config-value-hash-table))) | |
| 52 (cond ((null prev) | |
| 53 (puthash key value config-value-hash-table)) | |
| 54 ((atom prev) | |
| 55 (puthash key (list prev value) config-value-hash-table)) | |
| 56 (t | |
| 57 (nconc prev (list value)))))) | |
| 58 (end-of-file nil))) | |
| 59 (kill-buffer " *Config*"))) | |
| 60 config-value-hash-table) | |
| 61 | |
| 62 ;;;###autoload | |
| 63 (defun config-value (config-symbol) | |
| 64 "Return the value of the configuration parameter CONFIG_SYMBOL." | |
| 65 (gethash config-symbol (config-value-hash-table))) | |
| 66 | |
| 67 (provide 'config) | |
| 68 ;;; config.el ends here |