[RFA] Remove defs.h #include in rs6000-tdep.h
Joel Brobecker
brobecker@adacore.com
Thu Dec 20 15:47:00 GMT 2007
This is the only other #include of defs.h in a .h file that I could find,
and I wouldn't mind a second look.
I noticed that it was also missing an opaque declaration of struct
frame_info, so I added it as well.
2007-12-20 Joel Brobecker <brobecker@adacore.com>
* rs6000-tdep.h: Remove defs.h #include.
(struct frame_info): Add opaque declaration.
* Makefile.in (rs6000_tdep_h): Update dependencies.
Tested by rebuilding on powerpc-aix.
Does it look OK?
Thanks,
--
Joel
-------------- next part --------------
Index: rs6000-tdep.h
===================================================================
RCS file: /cvs/src/src/gdb/rs6000-tdep.h,v
retrieving revision 1.7
diff -u -p -r1.7 rs6000-tdep.h
--- rs6000-tdep.h 23 Aug 2007 18:08:37 -0000 1.7
+++ rs6000-tdep.h 20 Dec 2007 06:01:23 -0000
@@ -17,7 +17,8 @@
You should have received a copy of the GNU General Public License
along with this program. If not, see <http://www.gnu.org/licenses/>. */
-#include "defs.h"
+/* Opaque declarations. */
+struct frame_info;
extern int rs6000_software_single_step (struct frame_info *frame);
Index: Makefile.in
===================================================================
RCS file: /cvs/src/src/gdb/Makefile.in,v
retrieving revision 1.968
diff -u -p -r1.968 Makefile.in
--- Makefile.in 20 Dec 2007 05:36:14 -0000 1.968
+++ Makefile.in 20 Dec 2007 05:59:46 -0000
@@ -855,7 +855,7 @@ reggroups_h = reggroups.h
regset_h = regset.h
remote_fileio_h = remote-fileio.h
remote_h = remote.h
-rs6000_tdep_h = rs6000-tdep.h $(defs_h)
+rs6000_tdep_h = rs6000-tdep.h
s390_tdep_h = s390-tdep.h
scm_lang_h = scm-lang.h $(scm_tags_h)
scm_tags_h = scm-tags.h
More information about the Gdb-patches
mailing list