Don't include tm.h in intl.c

Joseph S. Myers joseph@codesourcery.com
Wed Dec 1 17:30:00 GMT 2010


In the course of testing a previous patch I found that intl.c includes
tm.h, although it has no need for that header and no business
depending on target configuration at all.  I've now committed this
patch to remove that include.  Bootstrapped with no regressions on
x86_64-unknown-linux-gnu.

2010-12-01  Joseph Myers  <joseph@codesourcery.com>

	* intl.c: Don't include tm.h.
	* Makefile.in (intl.o): Don't depend on $(TM_H).

Index: gcc/intl.c
===================================================================
--- gcc/intl.c	(revision 167331)
+++ gcc/intl.c	(working copy)
@@ -21,7 +21,6 @@ along with GCC; see the file COPYING3.  
 #include "config.h"
 #include "system.h"
 #include "coretypes.h"
-#include "tm.h"
 #include "intl.h"
 
 #ifdef HAVE_LANGINFO_CODESET
Index: gcc/Makefile.in
===================================================================
--- gcc/Makefile.in	(revision 167331)
+++ gcc/Makefile.in	(working copy)
@@ -4000,7 +4000,7 @@ gengtype-lex.c : gengtype-lex.l
 
 #
 # Remake internationalization support.
-intl.o: intl.c $(CONFIG_H) $(SYSTEM_H) coretypes.h $(TM_H) intl.h Makefile
+intl.o: intl.c $(CONFIG_H) $(SYSTEM_H) coretypes.h intl.h Makefile
 	$(COMPILER) $(ALL_COMPILERFLAGS) $(ALL_CPPFLAGS) \
 	  -DLOCALEDIR=\"$(localedir)\" \
 	  -c $(srcdir)/intl.c $(OUTPUT_OPTION)

-- 
Joseph S. Myers
joseph@codesourcery.com



More information about the Gcc-patches mailing list