From e197dfae62d741b8cd147bab888183c7c90f2a88 Mon Sep 17 00:00:00 2001 From: Simon Marchi Date: Thu, 29 Apr 2021 16:24:19 -0400 Subject: [PATCH] gdb: don't use C++17 namespace declaration style In a review, I suggested to use "namespace gdb::observers" instead of two separate namespace declarations. I didn't realize that this was a C++17 feature, which breaks compilers that default to an earlier version of the language, like g++ 4.8. Change it back to two separate declarations. gdb/ChangeLog: * auto-load.h: Split namespace declaration. Change-Id: I701537161967fbd9fcc298ff600bd072aab1251d --- gdb/ChangeLog | 4 ++++ gdb/auto-load.h | 6 ++++-- 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index ea65327e6e0..8e77fa677e9 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,7 @@ +2021-04-29 Simon Marchi + + * auto-load.h: Split namespace declaration. + 2021-04-29 Simon Marchi * infrun.c (save_waitstatus): Move variables to inner scope. diff --git a/gdb/auto-load.h b/gdb/auto-load.h index 4372ec4f4dd..c10787f873b 100644 --- a/gdb/auto-load.h +++ b/gdb/auto-load.h @@ -25,9 +25,11 @@ struct program_space; struct auto_load_pspace_info; struct extension_language_defn; -namespace gdb::observers { +namespace gdb { +namespace observers { struct token; -} +} /* namespace observers */ +} /* namespace gdb */ /* Value of the 'set debug auto-load' configuration variable. */ -- 2.30.2