This is the mail archive of the gdb-cvs@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[binutils-gdb] Silence obstack.c -Wc++compat warning


https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=78d14d2b9325fb4d25034eccf5650ef967d19640

commit 78d14d2b9325fb4d25034eccf5650ef967d19640
Author: Alan Modra <amodra@gmail.com>
Date:   Mon Nov 9 15:15:51 2015 +1030

    Silence obstack.c -Wc++compat warning
    
    Fixes
    warning: request for implicit conversion from â??void *â?? to â??struct _obstack_chunk *â?? not permitted in C++ [-Wc++-compat]
    
    I moved the assignment to h->chunk to fix an overlong line, then
    decided it would be better after the alloc failure check just to do
    things the same way as in _obstack_newchunk.
    
    	* obstack.c (_obstack_newchunk): Silence -Wc++compat warning.
    	(_obstack_begin_worker): Likewise.  Move assignment to h->chunk
    	after alloc failure check.

Diff:
---
 libiberty/ChangeLog | 6 ++++++
 libiberty/obstack.c | 5 +++--
 2 files changed, 9 insertions(+), 2 deletions(-)

diff --git a/libiberty/ChangeLog b/libiberty/ChangeLog
index 05b60d8..6d3b295 100644
--- a/libiberty/ChangeLog
+++ b/libiberty/ChangeLog
@@ -1,5 +1,11 @@
 2015-11-09  Alan Modra  <amodra@gmail.com>
 
+	* obstack.c (_obstack_newchunk): Silence -Wc++compat warning.
+	(_obstack_begin_worker): Likewise.  Move assignment to h->chunk
+	after alloc failure check.
+
+2015-11-09  Alan Modra  <amodra@gmail.com>
+
 	PR gdb/17133
 	* obstack.c (__alignof__): Expand alignof_type from alignof.h.
 	(obstack_exit_failure): Don't use exitfail.h.
diff --git a/libiberty/obstack.c b/libiberty/obstack.c
index 9f34da1..6d8d672 100644
--- a/libiberty/obstack.c
+++ b/libiberty/obstack.c
@@ -138,9 +138,10 @@ _obstack_begin_worker (struct obstack *h,
   h->chunk_size = size;
   h->alignment_mask = alignment - 1;
 
-  chunk = h->chunk = call_chunkfun (h, h->chunk_size);
+  chunk = (struct _obstack_chunk *) call_chunkfun (h, h->chunk_size);
   if (!chunk)
     (*obstack_alloc_failed_handler) ();
+  h->chunk = chunk;
   h->next_free = h->object_base = __PTR_ALIGN ((char *) chunk, chunk->contents,
                                                alignment - 1);
   h->chunk_limit = chunk->limit = (char *) chunk + h->chunk_size;
@@ -202,7 +203,7 @@ _obstack_newchunk (struct obstack *h, _OBSTACK_SIZE_T length)
 
   /* Allocate and initialize the new chunk.  */
   if (obj_size <= sum1 && sum1 <= sum2)
-    new_chunk = call_chunkfun (h, new_size);
+    new_chunk = (struct _obstack_chunk *) call_chunkfun (h, new_size);
   if (!new_chunk)
     (*obstack_alloc_failed_handler)();
   h->chunk = new_chunk;


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]