Hello,
attached is another fix for SPARSE headers improving their sanity. I am
also attaching a simple test for the dependency tracking of headers. It's
not enough generic and portable and therefore not really useful. Maybe
someone skilled in writing makefiles might want to include something like
that to the SPARSE Makefile as part of the 'check' target.
Kamil
From 8ffbb46652f69e662ce39c135f1867599c0f5477 Mon Sep 17 00:00:00 2001
From: Kamil Dudka <kdudka@xxxxxxxxxx>
Date: Fri, 7 Aug 2009 21:48:44 +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 | 2 ++
ptrlist.h | 2 ++
scope.h | 2 ++
storage.h | 2 ++
5 files changed, 10 insertions(+), 0 deletions(-)
diff --git a/compile.h b/compile.h
index 0db8fdb..6c5260d 100644
--- a/compile.h
+++ b/compile.h
@@ -1,6 +1,8 @@
#ifndef COMPILE_H
#define COMPILE_H
+#include "symbol.h"
+
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 9f2e165..241f081 100644
--- a/flow.h
+++ b/flow.h
@@ -1,6 +1,8 @@
#ifndef FLOW_H
#define FLOW_H
+#include "symbol.h"
+
extern unsigned long bb_generation;
#define REPEAT_CSE 1
diff --git a/ptrlist.h b/ptrlist.h
index dae0906..85b23fc 100644
--- a/ptrlist.h
+++ b/ptrlist.h
@@ -7,6 +7,8 @@
* (C) Copyright Linus Torvalds 2003-2005
*/
+#include <stdlib.h>
+
#define container(ptr, type, member) \
(type *)((void *)(ptr) - offsetof(type, member))
diff --git a/scope.h b/scope.h
index 5f1f232..aff7b6b 100644
--- a/scope.h
+++ b/scope.h
@@ -9,6 +9,8 @@
* Licensed under the Open Software License version 1.1
*/
+#include "symbol.h"
+
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..0d3bbc5 100644
--- a/storage.h
+++ b/storage.h
@@ -1,6 +1,8 @@
#ifndef STORAGE_H
#define STORAGE_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.3.3
# all SPARSE headers (except ident-list.h which is not an ordinary header)
HEADERS ?= \
allocate.h \
bitmap.h \
compat.h \
compile.h \
dissect.h \
expression.h \
flow.h \
lib.h \
linearize.h \
parse.h \
ptrlist.h \
scope.h \
storage.h \
symbol.h \
target.h \
token.h \
SINK ?= /dev/null
.PHONY: check $(HEADERS)
# check whether each header is self-compilable
check: $(HEADERS)
$(HEADERS):
$(CC) -o $(SINK) -c $@
