Patchwork Don't include tm.h in intl.c

login
register
mail settings
Submitter Joseph S. Myers
Date Dec. 1, 2010, 5:29 p.m.
Message ID <Pine.LNX.4.64.1012011729350.5152@digraph.polyomino.org.uk>
Download mbox | patch
Permalink /patch/73850/
State New
Headers show

Comments

Joseph S. Myers - Dec. 1, 2010, 5:29 p.m.
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).

Patch

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)