From patchwork Sat Aug 8 11:10:26 2009 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Kamil Dudka X-Patchwork-Id: 40114 Received: from vger.kernel.org (vger.kernel.org [209.132.176.167]) by demeter.kernel.org (8.14.2/8.14.2) with ESMTP id n78BB0Ur015231 for ; Sat, 8 Aug 2009 11:11:01 GMT Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756129AbZHHLK6 (ORCPT ); Sat, 8 Aug 2009 07:10:58 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S932101AbZHHLK6 (ORCPT ); Sat, 8 Aug 2009 07:10:58 -0400 Received: from mx2.redhat.com ([66.187.237.31]:57277 "EHLO mx2.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756129AbZHHLK5 (ORCPT ); Sat, 8 Aug 2009 07:10:57 -0400 Received: from int-mx2.corp.redhat.com (int-mx2.corp.redhat.com [172.16.27.26]) by mx2.redhat.com (8.13.8/8.13.8) with ESMTP id n78BAwI2008383 for ; Sat, 8 Aug 2009 07:10:58 -0400 Received: from ns3.rdu.redhat.com (ns3.rdu.redhat.com [10.11.255.199]) by int-mx2.corp.redhat.com (8.13.1/8.13.1) with ESMTP id n78BAwmm030559 for ; Sat, 8 Aug 2009 07:10:58 -0400 Received: from vpn1-4-54.ams2.redhat.com (vpn1-4-54.ams2.redhat.com [10.36.4.54]) by ns3.rdu.redhat.com (8.13.8/8.13.8) with ESMTP id n78BAuUi017322 for ; Sat, 8 Aug 2009 07:10:57 -0400 From: Kamil Dudka To: sparse Subject: Re: [PATCH] make sparse headers self-compilable... Date: Sat, 8 Aug 2009 13:10:26 +0200 User-Agent: KMail/1.9.7 References: <200908072227.08652.kdudka@redhat.com> In-Reply-To: <200908072227.08652.kdudka@redhat.com> MIME-Version: 1.0 X-Length: 5099 X-UID: 38391 Message-Id: <200908081310.27671.kdudka@redhat.com> X-Scanned-By: MIMEDefang 2.58 on 172.16.27.26 Sender: linux-sparse-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-sparse@vger.kernel.org On Friday 07 of August 2009 22:27:08 Kamil Dudka wrote: > 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. Attached are amended versions of the patch and the test. I didn't spot some warnings before. Kamil From 619088669e2333a8c5921d99befa8d72b322004d Mon Sep 17 00:00:00 2001 From: Kamil Dudka Date: Sat, 8 Aug 2009 12:58:41 +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 --- compile.h | 2 ++ flow.h | 2 ++ ptrlist.h | 2 ++ scope.h | 2 ++ storage.h | 3 +++ 5 files changed, 11 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 + #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..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