On Tuesday 11 of August 2009 14:48:39 Hannes Eder wrote:
> What about just using some forward decls instead of including the header
> files?
>
> e.g. for compile.h it is enough to have a "struct symbol;" there
> instead of the "#include <symbol.h>" to make it self-compilable.
Thanks for the hint! It's definitely better, though not sufficient in all
cases. The amended patch is attached.
Kamil
From 93455c6fe7dc84e1519bec70a9fd8625a879d791 Mon Sep 17 00:00:00 2001
From: Kamil Dudka <kdudka@xxxxxxxxxx>
Date: Tue, 11 Aug 2009 14:56:53 +0200
Subject: [PATCH] make sparse headers self-compilable...
... and thus possible to include them in arbitrary order and without any
external dependencies.
Signed-off-by: Kamil Dudka <kdudka@xxxxxxxxxx>
---
compile.h | 2 ++
flow.h | 5 +++++
ptrlist.h | 2 ++
scope.h | 2 ++
storage.h | 3 +++
5 files changed, 14 insertions(+), 0 deletions(-)
diff --git a/compile.h b/compile.h
index 0db8fdb..177363a 100644
--- a/compile.h
+++ b/compile.h
@@ -1,6 +1,8 @@
#ifndef COMPILE_H
#define COMPILE_H
+struct symbol;
+
extern void emit_one_symbol(struct symbol *);
extern void emit_unit_begin(const char *);
extern void emit_unit_end(void);
diff --git a/flow.h b/flow.h
index fbc9505..370aadd 100644
--- a/flow.h
+++ b/flow.h
@@ -1,11 +1,16 @@
#ifndef FLOW_H
#define FLOW_H
+#include "lib.h"
+
extern unsigned long bb_generation;
#define REPEAT_CSE 1
#define REPEAT_SYMBOL_CLEANUP 2
+struct entrypoint;
+struct instruction;
+
extern int simplify_flow(struct entrypoint *ep);
extern void simplify_symbol_usage(struct entrypoint *ep);
diff --git a/ptrlist.h b/ptrlist.h
index dae0906..fbfc080 100644
--- a/ptrlist.h
+++ b/ptrlist.h
@@ -1,6 +1,8 @@
#ifndef PTR_LIST_H
#define PTR_LIST_H
+#include <stdlib.h>
+
/*
* Generic pointer list manipulation code.
*
diff --git a/scope.h b/scope.h
index 5f1f232..0fab286 100644
--- a/scope.h
+++ b/scope.h
@@ -9,6 +9,8 @@
* Licensed under the Open Software License version 1.1
*/
+struct symbol;
+
struct scope {
struct token *token; /* Scope start information */
struct symbol_list *symbols; /* List of symbols in this scope */
diff --git a/storage.h b/storage.h
index 610cbfd..e049e23 100644
--- a/storage.h
+++ b/storage.h
@@ -1,6 +1,9 @@
#ifndef STORAGE_H
#define STORAGE_H
+#include "allocate.h"
+#include "lib.h"
+
/*
* The "storage" that underlies an incoming/outgoing pseudo. It's
* basically the backing store for a pseudo, and may be a real hardware
--
1.6.4
