diff options
| author | Paul Eggert | 2011-07-04 20:09:36 -0700 |
|---|---|---|
| committer | Paul Eggert | 2011-07-04 20:09:36 -0700 |
| commit | 6089c5670b18a02fc2caca3e665d2bb7799dc4c8 (patch) | |
| tree | 16b5f9dc30401413bead09c0005415d6c081adb0 /src | |
| parent | d8ed26bd07abb23acc9c1879776f5afc44ed4874 (diff) | |
| download | emacs-6089c5670b18a02fc2caca3e665d2bb7799dc4c8.tar.gz emacs-6089c5670b18a02fc2caca3e665d2bb7799dc4c8.zip | |
* lisp.h (struct vectorlike_header, struct Lisp_Subr): Signed sizes.
Use EMACS_INT, not EMACS_UINT, for sizes. The code works equally
well either way, and we prefer signed to unsigned.
Diffstat (limited to 'src')
| -rw-r--r-- | src/ChangeLog | 4 | ||||
| -rw-r--r-- | src/lisp.h | 4 |
2 files changed, 6 insertions, 2 deletions
diff --git a/src/ChangeLog b/src/ChangeLog index 9ad7da46ecf..e4702b4316b 100644 --- a/src/ChangeLog +++ b/src/ChangeLog | |||
| @@ -1,5 +1,9 @@ | |||
| 1 | 2011-07-05 Paul Eggert <eggert@cs.ucla.edu> | 1 | 2011-07-05 Paul Eggert <eggert@cs.ucla.edu> |
| 2 | 2 | ||
| 3 | * lisp.h (struct vectorlike_header, struct Lisp_Subr): Signed sizes. | ||
| 4 | Use EMACS_INT, not EMACS_UINT, for sizes. The code works equally | ||
| 5 | well either way, and we prefer signed to unsigned. | ||
| 6 | |||
| 3 | Random fixes. E.g., (random) never returned negative values. | 7 | Random fixes. E.g., (random) never returned negative values. |
| 4 | * fns.c (Frandom): Use GET_EMACS_TIME for random seed, and add the | 8 | * fns.c (Frandom): Use GET_EMACS_TIME for random seed, and add the |
| 5 | subseconds part to the entropy, as that's a bit more random. | 9 | subseconds part to the entropy, as that's a bit more random. |
diff --git a/src/lisp.h b/src/lisp.h index 762d34abb9c..2835302947f 100644 --- a/src/lisp.h +++ b/src/lisp.h | |||
| @@ -833,7 +833,7 @@ struct Lisp_String | |||
| 833 | <http://debbugs.gnu.org/cgi/bugreport.cgi?bug=8546>. */ | 833 | <http://debbugs.gnu.org/cgi/bugreport.cgi?bug=8546>. */ |
| 834 | struct vectorlike_header | 834 | struct vectorlike_header |
| 835 | { | 835 | { |
| 836 | EMACS_UINT size; | 836 | EMACS_INT size; |
| 837 | 837 | ||
| 838 | /* Pointer to the next vector-like object. It is generally a buffer or a | 838 | /* Pointer to the next vector-like object. It is generally a buffer or a |
| 839 | Lisp_Vector alias, so for convenience it is a union instead of a | 839 | Lisp_Vector alias, so for convenience it is a union instead of a |
| @@ -1028,7 +1028,7 @@ struct Lisp_Bool_Vector | |||
| 1028 | 1028 | ||
| 1029 | struct Lisp_Subr | 1029 | struct Lisp_Subr |
| 1030 | { | 1030 | { |
| 1031 | EMACS_UINT size; | 1031 | EMACS_INT size; |
| 1032 | union { | 1032 | union { |
| 1033 | Lisp_Object (*a0) (void); | 1033 | Lisp_Object (*a0) (void); |
| 1034 | Lisp_Object (*a1) (Lisp_Object); | 1034 | Lisp_Object (*a1) (Lisp_Object); |