gdb: don't use C++17 namespace declaration style
authorSimon Marchi <simon.marchi@efficios.com>
Thu, 29 Apr 2021 20:24:19 +0000 (16:24 -0400)
committerSimon Marchi <simon.marchi@polymtl.ca>
Thu, 29 Apr 2021 20:24:49 +0000 (16:24 -0400)
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
gdb/auto-load.h

index ea65327e6e0c949db824403fd160244ac2d4e4b4..8e77fa677e96f7553483bdad40edaec7b5288b93 100644 (file)
@@ -1,3 +1,7 @@
+2021-04-29  Simon Marchi  <simon.marchi@efficios.com>
+
+       * auto-load.h: Split namespace declaration.
+
 2021-04-29  Simon Marchi  <simon.marchi@polymtl.ca>
 
        * infrun.c (save_waitstatus): Move variables to inner scope.
index 4372ec4f4dd71c8f7b9fd8644bec3d8ad38ac1e2..c10787f873bbe0343b0166eabd7aab93ed5834bb 100644 (file)
@@ -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.  */