Update.

	* locale/programs/ld-collate.c (output_weightwc): Return index
	measured in words, not bytes.  Don't write out words for ignored
	characters.
	(collate_output): Correct various problems with alignment in the
	output file.

diff --git a/ChangeLog b/ChangeLog
index f11c9c5a6e25eff78c59717927ba104ea07fb07a..b5e96efc2d0a3486f85aa77af2e273eabe66de34 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,11 @@
 2000-01-19  Ulrich Drepper  <drepper@cygnus.com>
 
+       * locale/programs/ld-collate.c (output_weightwc): Return index
+       measured in words, not bytes.  Don't write out words for ignored
+       characters.
+       (collate_output): Correct various problems with alignment in the
+       output file.
+
        * stdlib/strfmon.c: Don't report an error if final NUL is at the
        end of the buffer.  Set errno correctly if floating-point number
        would overflow buffer.

diff --git a/locale/programs/ld-collate.c b/locale/programs/ld-collate.c
index 2e3707d78fc7db0182e543412baad75c0143a7df..bd07f706129974138aa49200864aa1dc0af1d2e2 100644 (file)
--- a/locale/programs/ld-collate.c
+++ b/locale/programs/ld-collate.c
@@ -1833,25 +1833,26 @@ output_weightwc (struct obstack *pool, struct locale_collate_t *collate,
 
   /* This byte can start exactly one collation element and this is
      a single byte.  We can directly give the index to the weights.  */
-  retval = obstack_object_size (pool);
+  retval = obstack_object_size (pool) / sizeof (int32_t);
 
   /* Construct the weight.  */
   for (cnt = 0; cnt < nrules; ++cnt)
     {
       int32_t buf[elem->weights[cnt].cnt];
-      int32_t i;
+      int i;
+      int32_t j;
 
-      for (i = 0; i < elem->weights[cnt].cnt; ++i)
+      for (i = 0, j = 0; i < elem->weights[cnt].cnt; ++i)
        if (elem->weights[cnt].w[i] != NULL)
-         buf[i] = elem->weights[cnt].w[i]->wcorder;
+         buf[j++] = elem->weights[cnt].w[i]->wcorder;
 
       /* And add the buffer content.  */
       if (sizeof (int) == sizeof (int32_t))
-       obstack_int_grow (pool, i);
+       obstack_int_grow (pool, j);
       else
-       obstack_grow (pool, &i, sizeof (int32_t));
+       obstack_grow (pool, &j, sizeof (int32_t));
 
-      obstack_grow (pool, buf, i * sizeof (int32_t));
+      obstack_grow (pool, buf, j * sizeof (int32_t));
     }
 
   return retval | ((elem->section->ruleidx & 0x7f) << 24);
@@ -2044,6 +2045,8 @@ collate_output (struct localedef_t *locale, struct charmap_t *charmap,
                added = ((sizeof (int32_t) + 1 + 2 * (runp->nmbs - 1)
                          + __alignof__ (int32_t) - 1)
                         & ~(__alignof__ (int32_t) - 1));
+               assert ((obstack_object_size (&extrapool)
+                        & (__alignof__ (int32_t) - 1)) == 0);
                obstack_make_room (&extrapool, added);
 
                /* More than one consecutive entry.  We mark this by having
@@ -2128,11 +2131,9 @@ collate_output (struct localedef_t *locale, struct charmap_t *charmap,
              }
 
            /* Add alignment bytes if necessary.  */
-           i = added % __alignof__ (int32_t);
-           if (i > 0)
-             do
-               obstack_1grow_fast (&extrapool, '\0');
-             while (++i != __alignof__ (int32_t));
+           while ((obstack_object_size (&extrapool)
+                   & (__alignof__ (int32_t) - 1)) != 0)
+             obstack_1grow_fast (&extrapool, '\0');
 
            /* Next entry.  */
            lastp = runp;
@@ -2161,19 +2162,23 @@ collate_output (struct localedef_t *locale, struct charmap_t *charmap,
            obstack_1grow_fast (&extrapool, 0);
 
            /* Add alignment bytes if necessary.  */
-           i = added % __alignof__ (int32_t);
-           if (i > 0)
-             do
-               obstack_1grow_fast (&extrapool, '\0');
-             while (++i != __alignof__ (int32_t));
+           while ((obstack_object_size (&extrapool)
+                   & (__alignof__ (int32_t) - 1)) != 0)
+             obstack_1grow_fast (&extrapool, '\0');
          }
       }
 
+  /* Add padding to the tables if necessary.  */
+  while ((obstack_object_size (&weightpool) & (__alignof__ (int32_t) - 1))
+        != 0)
+    obstack_1grow (&weightpool, 0);
+
   /* Now add the four tables.  */
   assert (cnt == _NL_ITEM_INDEX (_NL_COLLATE_TABLEMB));
   iov[2 + cnt].iov_base = tablemb;
   iov[2 + cnt].iov_len = sizeof (tablemb);
   idx[1 + cnt] = idx[cnt] + iov[2 + cnt].iov_len;
+  assert ((iov[2 + cnt].iov_len & (__alignof__ (int32_t) - 1)) == 0);
   ++cnt;
 
   assert (cnt == _NL_ITEM_INDEX (_NL_COLLATE_WEIGHTMB));
@@ -2192,6 +2197,7 @@ collate_output (struct localedef_t *locale, struct charmap_t *charmap,
   iov[2 + cnt].iov_len = obstack_object_size (&indirectpool);
   iov[2 + cnt].iov_base = obstack_finish (&indirectpool);
   idx[1 + cnt] = idx[cnt] + iov[2 + cnt].iov_len;
+  assert ((iov[2 + cnt].iov_len & (__alignof__ (int32_t) - 1)) == 0);
   ++cnt;
 
 
@@ -2243,10 +2249,11 @@ collate_output (struct localedef_t *locale, struct charmap_t *charmap,
   /* Now insert the `UNDEFINED' value if it is used.  Since this value
      will probably be used more than once it is good to store the
      weights only once.  */
-  output_weightwc (&weightpool, collate, &collate->undefined);
+  if (output_weightwc (&weightpool, collate, &collate->undefined) != 0)
+    abort ();
 
-  /* Generate the table.  Walk through the lists of sequences
-     starting with the same byte and add them one after the other to
+  /* Generate the table.  Walk through the lists of sequences starting
+     with the same wide character and add them one after the other to
      the table.  In case we have more than one sequence starting with
      the same byte we have to use extra indirection.  */
   tablewc = (uint32_t *) alloca (table_size * sizeof (uint32_t));
@@ -2257,7 +2264,7 @@ collate_output (struct localedef_t *locale, struct charmap_t *charmap,
        tablewc[ch] = 0;
       }
     else if (collate->wcheads[ch]->wcnext == NULL
-       && collate->wcheads[ch]->nwcs == 1)
+            && collate->wcheads[ch]->nwcs == 1)
       {
        tablewc[ch] = output_weightwc (&weightpool, collate,
                                       collate->wcheads[ch]);
@@ -2269,7 +2276,7 @@ collate_output (struct localedef_t *locale, struct charmap_t *charmap,
        struct element_t *runp = collate->wcheads[ch];
        struct element_t *lastp;
 
-       tablewc[ch] = -obstack_object_size (&extrapool);
+       tablewc[ch] = -(obstack_object_size (&extrapool) / sizeof (uint32_t));
 
        do
          {
@@ -2397,26 +2404,31 @@ collate_output (struct localedef_t *locale, struct charmap_t *charmap,
   /* Now add the four tables.  */
   assert (cnt == _NL_ITEM_INDEX (_NL_COLLATE_TABLEWC));
   iov[2 + cnt].iov_base = tablewc;
-  iov[2 + cnt].iov_len = table_size * sizeof (int32_t);
+  iov[2 + cnt].iov_len = table_size * sizeof (uint32_t);
   idx[1 + cnt] = idx[cnt] + iov[2 + cnt].iov_len;
+  assert (iov[2 + cnt].iov_len % sizeof (int32_t) == 0);
   ++cnt;
 
   assert (cnt == _NL_ITEM_INDEX (_NL_COLLATE_WEIGHTWC));
   iov[2 + cnt].iov_len = obstack_object_size (&weightpool);
   iov[2 + cnt].iov_base = obstack_finish (&weightpool);
   idx[1 + cnt] = idx[cnt] + iov[2 + cnt].iov_len;
+  assert (iov[2 + cnt].iov_len % sizeof (int32_t) == 0);
   ++cnt;
 
   assert (cnt == _NL_ITEM_INDEX (_NL_COLLATE_EXTRAWC));
   iov[2 + cnt].iov_len = obstack_object_size (&extrapool);
   iov[2 + cnt].iov_base = obstack_finish (&extrapool);
   idx[1 + cnt] = idx[cnt] + iov[2 + cnt].iov_len;
+  assert (iov[2 + cnt].iov_len % sizeof (int32_t) == 0);
+  assert (iov[2 + cnt].iov_len % sizeof (int32_t) == 0);
   ++cnt;
 
   assert (cnt == _NL_ITEM_INDEX (_NL_COLLATE_INDIRECTWC));
   iov[2 + cnt].iov_len = obstack_object_size (&indirectpool);
   iov[2 + cnt].iov_base = obstack_finish (&indirectpool);
   idx[1 + cnt] = idx[cnt] + iov[2 + cnt].iov_len;
+  assert (iov[2 + cnt].iov_len % sizeof (int32_t) == 0);
   ++cnt;