On 11/06/11 07:28, Torvald Riegel wrote:
On Fri, 2011-11-04 at 22:48 +0100, Torvald Riegel wrote:
On Thu, 2011-11-03 at 20:38 +0000, Joseph S. Myers wrote:
Make sure that you do need each #include present in this and any other new
file.  Since 2008 a lot of includes of tm.h and toplev.h have been removed
and diagnostic-core.h introduced as an alternative to diagnostic.h for
many users.  If tm.h is needed, it's good to have a comment on the
#include explaining why (which target macros are used, for example), since
we'd like to avoid unnecessary tm.h includes.

See attached patch. OK for branch?

I should have updated the build dependencies too, which this patch does.
OK for branch?

Yes.

Reply via email to