changeset 4532:16906fefc8df bytecomp-coding-system-2008-10-29

Return a list copy in #'built-in-face-specifiers, pre-empting modification. 2008-12-21 Aidan Kehoe <kehoea@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.
author Aidan Kehoe <kehoea@parhasard.net>
date Sun, 21 Dec 2008 17:23:06 +0000
parents b58af9a9765d
children 4a7c4ccac2fe 004ce6211106
files src/ChangeLog src/faces.c
diffstat 2 files changed, 8 insertions(+), 2 deletions(-) [+]
line wrap: on
line diff
--- 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@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@xemacs.org>
 
 	G++ 4.3 complains a lot about implicit conversions of string
--- 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