* charset.h (struct charset.code_space): Now has 15 elements, not 16.

* charset.c (Fdefine_charset_internal): Don't initialize
charset.code_space[15].  The value was garbage, on hosts with
32-bit int.

diff --git a/src/ChangeLog b/src/ChangeLog
index 7b54b1e521c51d459addd8d024fa5bf03d148309..93af816947cecb77316bb83cf446383a8fda38ac 100644 (file)
--- a/src/ChangeLog
+++ b/src/ChangeLog
@@ -1,3 +1,10 @@
+2011-05-01  Paul Eggert  <eggert@cs.ucla.edu>
+
+       * charset.h (struct charset.code_space): Now has 15 elements, not 16.
+       * charset.c (Fdefine_charset_internal): Don't initialize
+       charset.code_space[15].  The value was garbage, on hosts with
+       32-bit int.
+
 2011-04-30  Eli Zaretskii  <eliz@gnu.org>
 
        * dosfns.c (Fint86, Fdos_memget, Fdos_memput): Use `ASIZE (FOO)'

diff --git a/src/charset.c b/src/charset.c
index 52c2ebdcc4e18aba62e64b310e95b5a1c89baaba..55fd57031acea7a21c70e0a7f85feb1f4d205e68 100644 (file)
--- a/src/charset.c
+++ b/src/charset.c
@@ -869,7 +869,7 @@ usage: (define-charset-internal ...)  */)
   ASET (attrs, charset_name, args[charset_arg_name]);
 
   val = args[charset_arg_code_space];
-  for (i = 0, dimension = 0, nchars = 1; i < 4; i++)
+  for (i = 0, dimension = 0, nchars = 1; ; i++)
     {
       int min_byte, max_byte;
 
@@ -880,10 +880,12 @@ usage: (define-charset-internal ...)  */)
       charset.code_space[i * 4] = min_byte;
       charset.code_space[i * 4 + 1] = max_byte;
       charset.code_space[i * 4 + 2] = max_byte - min_byte + 1;
-      nchars *= charset.code_space[i * 4 + 2];
-      charset.code_space[i * 4 + 3] = nchars;
       if (max_byte > 0)
        dimension = i + 1;
+      if (i == 3)
+       break;
+      nchars *= charset.code_space[i * 4 + 2];
+      charset.code_space[i * 4 + 3] = nchars;
     }
 
   val = args[charset_arg_dimension];

diff --git a/src/charset.h b/src/charset.h
index 74d55a31b43e8441105619fea2a70478409c33fb..53784bf8455a1c0194c7241068c8cffd45ec9b99 100644 (file)
--- a/src/charset.h
+++ b/src/charset.h
@@ -155,10 +155,11 @@ struct charset
      byte code of the (N+1)th dimension, <code_space>[4N+1] is a
      maximum byte code of the (N+1)th dimension, <code_space>[4N+2] is
      (<code_space>[4N+1] - <code_space>[4N] + 1), <code_space>[4N+3]
-     is a number of characters containd in the first to (N+1)th
-     dismesions.  We get `char-index' of a `code-point' from this
+     is the number of characters contained in the first through (N+1)th
+     dimensions, except that there is no <code_space>[15].
+     We get `char-index' of a `code-point' from this
      information.  */
-  int code_space[16];
+  int code_space[15];
 
   /* If B is a byte of Nth dimension of a code-point, the (N-1)th bit
      of code_space_mask[B] is set.  This array is used to quickly