[gdb] Fix build error in macroexp.c
authorTom de Vries <tdevries@suse.de>
Sat, 16 Jun 2018 16:39:30 +0000 (18:39 +0200)
committerTom de Vries <tdevries@suse.de>
Thu, 5 Jul 2018 09:23:46 +0000 (11:23 +0200)
commitc530603c59efb5630ce67446be64d42c0522fee2
tree6a271a4b409231ad82e3e8c8dc6621a0fd698818
parent4a91d0ba307eb24eb87ad27f4ea8fcde823c3e61
[gdb] Fix build error in macroexp.c

When doing a combined build with the gcc and binutils-gdb repos, I run into
this build error in gdb:
...
gdb/macroexp.c: \
  In function ‘void get_next_token_for_substitution(macro_buffer*, \
  macro_buffer*, char**, macro_buffer*, char**, int*, bool*)’:
gdb/macroexp.c:925:17: error: \
  implicitly-declared ‘constexpr macro_buffer& \
  macro_buffer::operator=(const macro_buffer&)’ is deprecated \
  [-Werror=deprecated-copy]
       *token = *lookahead;
...

Wdeprecated-copy is a new gcc warning added after gcc 8.

This patch fixes the build error by adding an explicit copy operator to the
macro_buffer class.  I've added asserts to ensure that both the dest and src
of the copy are shared, in other words, neither is owner of the text pointer.

I've run the gdb testsuite on x86_64-linux and the asserts did not trigger.

2018-07-05  Tom de Vries  <tdevries@suse.de>

* macroexp.c (macro_buffer) <operator=>: New member function.
gdb/ChangeLog
gdb/macroexp.c