partial header cleanup - Mailing list pgsql-patches

From Zdenek Kotala
Subject partial header cleanup
Date
Msg-id 48402849.2050809@sun.com
Whole thread Raw
Responses Re: partial header cleanup
Re: partial header cleanup
List pgsql-patches
This replace xlog.h with xlogdefs.h in bufpage.h. All other changes are
forgotten include somewhere. It reduce e.g. bloat to half in itup.h. But, There
are still unresolved problems. htup should include bufpage.h, because it needs
PageHeader size, but there is still unnecessary bufmgr.h include in bufpage
which generates bloat.

See itup.h bloating:

http://doxygen.postgresql.org/itup_8h.html

this patch reduce xlog side. But there still about 18 unnecessary includes.



        Zdenek

PS: Thanks to Stefan K. He enabled graphs.
Index: src/backend/nodes/print.c
===================================================================
RCS file: /projects/cvsroot/pgsql/src/backend/nodes/print.c,v
retrieving revision 1.87
diff -c -r1.87 print.c
*** src/backend/nodes/print.c    1 Jan 2008 19:45:50 -0000    1.87
--- src/backend/nodes/print.c    30 May 2008 15:13:42 -0000
***************
*** 20,25 ****
--- 20,26 ----
  #include "postgres.h"

  #include "access/printtup.h"
+ #include "lib/stringinfo.h"
  #include "nodes/print.h"
  #include "optimizer/clauses.h"
  #include "parser/parsetree.h"
Index: src/backend/postmaster/postmaster.c
===================================================================
RCS file: /projects/cvsroot/pgsql/src/backend/postmaster/postmaster.c,v
retrieving revision 1.557
diff -c -r1.557 postmaster.c
*** src/backend/postmaster/postmaster.c    4 May 2008 21:13:35 -0000    1.557
--- src/backend/postmaster/postmaster.c    30 May 2008 15:13:42 -0000
***************
*** 93,98 ****
--- 93,99 ----
  #endif

  #include "access/transam.h"
+ #include "access/xlog.h"
  #include "bootstrap/bootstrap.h"
  #include "catalog/pg_control.h"
  #include "lib/dllist.h"
Index: src/backend/utils/adt/domains.c
===================================================================
RCS file: /projects/cvsroot/pgsql/src/backend/utils/adt/domains.c,v
retrieving revision 1.6
diff -c -r1.6 domains.c
*** src/backend/utils/adt/domains.c    1 Jan 2008 19:45:52 -0000    1.6
--- src/backend/utils/adt/domains.c    30 May 2008 15:13:42 -0000
***************
*** 33,38 ****
--- 33,39 ----

  #include "commands/typecmds.h"
  #include "executor/executor.h"
+ #include "lib/stringinfo.h"
  #include "utils/builtins.h"
  #include "utils/lsyscache.h"

Index: src/backend/utils/fmgr/fmgr.c
===================================================================
RCS file: /projects/cvsroot/pgsql/src/backend/utils/fmgr/fmgr.c,v
retrieving revision 1.119
diff -c -r1.119 fmgr.c
*** src/backend/utils/fmgr/fmgr.c    15 May 2008 00:17:40 -0000    1.119
--- src/backend/utils/fmgr/fmgr.c    30 May 2008 15:13:42 -0000
***************
*** 19,24 ****
--- 19,25 ----
  #include "catalog/pg_language.h"
  #include "catalog/pg_proc.h"
  #include "executor/functions.h"
+ #include "lib/stringinfo.h"
  #include "miscadmin.h"
  #include "parser/parse_expr.h"
  #include "pgstat.h"
Index: src/include/access/gin.h
===================================================================
RCS file: /projects/cvsroot/pgsql/src/include/access/gin.h,v
retrieving revision 1.20
diff -c -r1.20 gin.h
*** src/include/access/gin.h    16 May 2008 16:31:01 -0000    1.20
--- src/include/access/gin.h    30 May 2008 15:13:43 -0000
***************
*** 14,19 ****
--- 14,20 ----

  #include "access/itup.h"
  #include "access/relscan.h"
+ #include "access/xlog.h"
  #include "fmgr.h"
  #include "nodes/tidbitmap.h"
  #include "storage/block.h"
Index: src/include/access/heapam.h
===================================================================
RCS file: /projects/cvsroot/pgsql/src/include/access/heapam.h,v
retrieving revision 1.134
diff -c -r1.134 heapam.h
*** src/include/access/heapam.h    12 May 2008 00:00:53 -0000    1.134
--- src/include/access/heapam.h    30 May 2008 15:13:43 -0000
***************
*** 17,22 ****
--- 17,23 ----
  #include "access/htup.h"
  #include "access/relscan.h"
  #include "access/sdir.h"
+ #include "access/xlog.h"
  #include "nodes/primnodes.h"
  #include "storage/lock.h"
  #include "utils/snapshot.h"
Index: src/include/access/nbtree.h
===================================================================
RCS file: /projects/cvsroot/pgsql/src/include/access/nbtree.h,v
retrieving revision 1.118
diff -c -r1.118 nbtree.h
*** src/include/access/nbtree.h    16 Apr 2008 23:59:40 -0000    1.118
--- src/include/access/nbtree.h    30 May 2008 15:13:43 -0000
***************
*** 17,22 ****
--- 17,23 ----
  #include "access/itup.h"
  #include "access/relscan.h"
  #include "access/sdir.h"
+ #include "access/xlog.h"
  #include "access/xlogutils.h"


Index: src/include/storage/bufpage.h
===================================================================
RCS file: /projects/cvsroot/pgsql/src/include/storage/bufpage.h,v
retrieving revision 1.79
diff -c -r1.79 bufpage.h
*** src/include/storage/bufpage.h    12 May 2008 16:06:10 -0000    1.79
--- src/include/storage/bufpage.h    30 May 2008 15:13:43 -0000
***************
*** 14,20 ****
  #ifndef BUFPAGE_H
  #define BUFPAGE_H

! #include "access/xlog.h"
  #include "storage/bufmgr.h"
  #include "storage/item.h"
  #include "storage/off.h"
--- 14,20 ----
  #ifndef BUFPAGE_H
  #define BUFPAGE_H

! #include "access/xlogdefs.h"
  #include "storage/bufmgr.h"
  #include "storage/item.h"
  #include "storage/off.h"

pgsql-patches by date:

Previous
From: Tom Lane
Date:
Subject: Re: minor change to replace function comment
Next
From: Teodor Sigaev
Date:
Subject: GIN improvements