All, While looking at the include dependency graph for objectaddress.h:
http://doxygen.postgresql.org/objectaddress_8h.html I saw that pg_list.h is both included and inherited (through multiple paths) by objectaddress.h. Perhaps this doesn't matter, but I thought I would at least bring it up and propose removing this redundant #include from objectaddress.h. If it makes sense to do so, I have attached a patch that removes it. Thanks, Adam -- Adam Brightwell - adam.brightw...@crunchydatasolutions.com Database Engineer - www.crunchydatasolutions.com
diff --git a/src/include/catalog/objectaddress.h b/src/include/catalog/objectaddress.h new file mode 100644 index 37808c0..432cbe8 *** a/src/include/catalog/objectaddress.h --- b/src/include/catalog/objectaddress.h *************** *** 13,19 **** #ifndef OBJECTADDRESS_H #define OBJECTADDRESS_H - #include "nodes/pg_list.h" #include "storage/lock.h" #include "utils/acl.h" #include "utils/relcache.h" --- 13,18 ----
-- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
