Small change, prevention is a better mechanism than admonishment.
# HG changeset patch
# User Aidan Kehoe <kehoea(a)parhasard.net>
# Date 1229880186 0
# Branch bytecomp-coding-system-2008-10-29
# Node ID 16906fefc8df0d642aec6fd43d78637af0054965
# Parent b58af9a9765dec7ac331ed18350012172389688d
Return a list copy in #'built-in-face-specifiers, pre-empting modification.
2008-12-21 Aidan Kehoe <kehoea(a)parhasard.net>
* faces.c (Fbuilt_in_face_specifiers):
Don't simply warn the user not to modify
Vbuilt_in_face_specifiers, enforce this by returning a copy of the
list.
diff -r b58af9a9765d -r 16906fefc8df src/ChangeLog
--- a/src/ChangeLog Sun Nov 02 03:59:54 2008 +0900
+++ b/src/ChangeLog Sun Dec 21 17:23:06 2008 +0000
@@ -1,3 +1,10 @@
+2008-12-21 Aidan Kehoe <kehoea(a)parhasard.net>
+
+ * faces.c (Fbuilt_in_face_specifiers):
+ Don't simply warn the user not to modify
+ Vbuilt_in_face_specifiers, enforce this by returning a copy of the
+ list.
+
2008-11-02 Stephen J. Turnbull <stephen(a)xemacs.org>
G++ 4.3 complains a lot about implicit conversions of string
diff -r b58af9a9765d -r 16906fefc8df src/faces.c
--- a/src/faces.c Sun Nov 02 03:59:54 2008 +0900
+++ b/src/faces.c Sun Dec 21 17:23:06 2008 +0000
@@ -703,11 +703,10 @@
DEFUN ("built-in-face-specifiers", Fbuilt_in_face_specifiers, 0, 0, 0, /*
Return a list of all built-in face specifier properties.
-Don't modify this list!
*/
())
{
- return Vbuilt_in_face_specifiers;
+ return Fcopy_list(Vbuilt_in_face_specifiers);
}
/* These values are retrieved so often that we make a special
--
¿Dónde estará ahora mi sobrino Yoghurtu Nghé, que tuvo que huir
precipitadamente de la aldea por culpa de la escasez de rinocerontes?
_______________________________________________
XEmacs-Patches mailing list
XEmacs-Patches(a)xemacs.org
http://calypso.tux.org/cgi-bin/mailman/listinfo/xemacs-patches