summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorStefan Monnier <monnier@iro.umontreal.ca>2008-02-10 03:20:40 +0000
committerStefan Monnier <monnier@iro.umontreal.ca>2008-02-10 03:20:40 +0000
commit4b75ffab30b7764136732398319d33d7104cae68 (patch)
tree5610831cc57e65ae1f3e4889f0f65e28a2bd1bff /src
parent250962b3d8e3840c3046ae3434fc3785ecacb7eb (diff)
downloademacs-4b75ffab30b7764136732398319d33d7104cae68.tar.gz
(ASET): Check bounds.
Diffstat (limited to 'src')
-rw-r--r--src/ChangeLog4
-rw-r--r--src/lisp.h6
2 files changed, 9 insertions, 1 deletions
diff --git a/src/ChangeLog b/src/ChangeLog
index afc6603da5e..b3ffe08f3ba 100644
--- a/src/ChangeLog
+++ b/src/ChangeLog
@@ -1,3 +1,7 @@
+2008-02-10 Stefan Monnier <monnier@iro.umontreal.ca>
+
+ * lisp.h (ASET): Check bounds.
+
2008-02-10 Glenn Morris <rgm@gnu.org>
* buffer.c (mode-name): Doc fix.
diff --git a/src/lisp.h b/src/lisp.h
index 5156386551a..daa45d3aad8 100644
--- a/src/lisp.h
+++ b/src/lisp.h
@@ -589,8 +589,12 @@ extern size_t pure_size;
/* Convenience macros for dealing with Lisp arrays. */
#define AREF(ARRAY, IDX) XVECTOR ((ARRAY))->contents[IDX]
-#define ASET(ARRAY, IDX, VAL) (AREF ((ARRAY), (IDX)) = (VAL))
#define ASIZE(ARRAY) XVECTOR ((ARRAY))->size
+/* The IDX==IDX tries to detect when the macro argument is side-effecting. */
+#define ASET(ARRAY, IDX, VAL) \
+ (eassert ((IDX) == (IDX)), \
+ eassert ((IDX) >= 0 && (IDX) < ASIZE (ARRAY)), \
+ ASLOT ((ARRAY), (IDX)) = (VAL))
/* Convenience macros for dealing with Lisp strings. */