* ada-lang.c (grow_vect): Return the new array instead of

	expecting pointer to pointer to avoid violation of aliasing rules.
	* ada-lang.h (GROW_VECT): Adapt call to grow_vect.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 12485fa0ef702f512f83bb8dce942fc99018bc6e..0df3786c7007763a3ffb74385e638ab7568e9bb9 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,9 @@
+2004-10-23  Andreas Schwab  <schwab@suse.de>
+
+       * ada-lang.c (grow_vect): Return the new array instead of               
+       expecting pointer to pointer to avoid violation of aliasing rules.      
+       * ada-lang.h (GROW_VECT): Adapt call to grow_vect.
+
 2004-10-23  Ulrich Weigand  <uweigand@de.ibm.com>
 
        * s390-tdep.c (enum pv_boolean): Remove.

diff --git a/gdb/ada-lang.c b/gdb/ada-lang.c
index 275e3a0a96e8a4d017b18acb2438b87dc867390d..00e61a47467be92be8e839cc493fccaacf253de1 100644 (file)
--- a/gdb/ada-lang.c
+++ b/gdb/ada-lang.c
@@ -319,20 +319,21 @@ extract_string (CORE_ADDR addr, char *buf)
   while (buf[char_index - 1] != '\000');
 }
 
-/* Assuming *OLD_VECT points to an array of *SIZE objects of size
+/* Assuming VECT points to an array of *SIZE objects of size
    ELEMENT_SIZE, grow it to contain at least MIN_SIZE objects,
-   updating *OLD_VECT and *SIZE as necessary.  */
+   updating *SIZE as necessary and returning the (new) array.  */
 
-void
-grow_vect (void **old_vect, size_t * size, size_t min_size, int element_size)
+void *
+grow_vect (void *vect, size_t *size, size_t min_size, int element_size)
 {
   if (*size < min_size)
     {
       *size *= 2;
       if (*size < min_size)
         *size = min_size;
-      *old_vect = xrealloc (*old_vect, *size * element_size);
+      vect = xrealloc (vect, *size * element_size);
     }
+  return vect;
 }
 
 /* True (non-zero) iff TARGET matches FIELD_NAME up to any trailing

diff --git a/gdb/ada-lang.h b/gdb/ada-lang.h
index 18a662a0730cdc30526b0b8a9a0c83239729eac8..3df4aa496c8cffdb5bbdc89c804d6e00a1e237f9 100644 (file)
--- a/gdb/ada-lang.h
+++ b/gdb/ada-lang.h
@@ -171,9 +171,9 @@ extern struct task_entry *task_list;
    least M objects, updating V and S as necessary. */
 
 #define GROW_VECT(v, s, m)                                              \
-   if ((s) < (m)) grow_vect ((void**) &(v), &(s), (m), sizeof(*(v)));
+   if ((s) < (m)) (v) = grow_vect (v, &(s), m, sizeof *(v));
 
-extern void grow_vect (void **, size_t *, size_t, int);
+extern void *grow_vect (void *, size_t *, size_t, int);
 
 extern int ada_get_field_index (const struct type *type,
                                 const char *field_name,