Re: [patch/commit] Fix nested header include in infttrace.h

[Andrew Cagney <cagney at gnu dot org>]

> infttrace.h was including target.h, which caused a lot of compile
> warnings related to duplicate defines of PC_REQUIRES_RUN_BEFORE_USE.
>
> this fixes it. 
>
> commited as obvious. thx
> randolph
>
> 2004-05-26  Randolph Chung  <tausq@debian.org>
>
> 	* infttrace.h (target_waitkind): Forward declare type instead of pulling
> 	in header file.

Unfortunatly this is a GCC extension.  It sounds like a header file is 
missing:
	#ifndef FOO_H
	#define FOO_H
	#endif
wrappers?  Adding them is ``obvious''.

Andrew

> Index: infttrace.h
> ===================================================================
> RCS file: /cvs/src/src/gdb/infttrace.h,v
> retrieving revision 1.3
> diff -u -p -r1.3 infttrace.h
> --- infttrace.h	6 Apr 2004 18:29:10 -0000	1.3
> +++ infttrace.h	26 May 2004 15:34:24 -0000
> @@ -22,7 +22,7 @@
>  #ifndef INFTTRACE_H
>  #define INFTTRACE_H
>  
> -#include "target.h"		/* For enum target_waitkind.  */
> +enum target_waitkind;
>  
>  extern int parent_attach_all (int, PTRACE_ARG3_TYPE, int);
>  extern pid_t hppa_switched_threads (pid_t gdb_pid);
> -- Randolph Chung Debian GNU/Linux Developer, hppa/ia64 ports http://www.tausq.org/