Remove #include "kvp_frame.h" from qof.h and add it where it's legitimately used.

src/app-utils/test/test-option-util.c

@@ -26,6 +26,7 @@
 #include <glib.h>
 #include <unittest-support.h>
 #include <qofbookslots.h>
+#include <kvp_frame.h>
 
 #include "../option-util.h"
 

src/backend/dbi/test/test-backend-dbi-basic.c

@@ -12,6 +12,7 @@
 #include <glib/gstdio.h>
 
 #include <qof.h>
+#include <kvp_frame.h>
 #include <unittest-support.h>
 #include <test-stuff.h>
 #include <test-dbi-stuff.h>

src/backend/dbi/test/test-dbi-stuff.c

@@ -23,17 +23,18 @@
  *  02110-1301, USA.
  */
 
-#include "config.h"
+#include <config.h>
-#include "qof.h"
+#include <qof.h>
-#include "qofsession-p.h"
+#include <kvp_frame.h>
-#include "cashobjects.h"
+#include <qofsession-p.h>
-#include "test-dbi-stuff.h"
+#include <cashobjects.h>
+#include <test-dbi-stuff.h>
 #include <unittest-support.h>
 
-#include "Account.h"
+#include <Account.h>
-#include "Split.h"
+#include <Split.h>
-#include "Transaction.h"
+#include <Transaction.h>
-#include "gnc-commodity.h"
+#include <gnc-commodity.h>
 #include <SX-book.h>
 #include <gnc-lot.h>
 #include "../gnc-backend-dbi-priv.h"

src/backend/sql/gnc-backend-sql.c

@@ -34,17 +34,18 @@
 #include <glib/gi18n.h>
 #include <glib/gstdio.h>
 
-#include "qof.h"
+#include <qof.h>
-#include "qofquery-p.h"
+#include <qofquery-p.h>
-#include "qofquerycore-p.h"
+#include <qofquerycore-p.h>
-#include "Account.h"
+#include <kvp_frame.h>
-#include "TransLog.h"
+#include <Account.h>
-#include "gnc-engine.h"
+#include <TransLog.h>
-#include "SX-book.h"
+#include <gnc-engine.h>
-#include "Recurrence.h"
+#include <SX-book.h>
-#include "gncBillTerm.h"
+#include <Recurrence.h>
-#include "gncTaxTable.h"
+#include <gncBillTerm.h>
-#include "gncInvoice.h"
+#include <gncTaxTable.h>
+#include <gncInvoice.h>
 
 #include "gnc-backend-sql.h"
 

src/backend/sql/gnc-slots-sql.cpp

@@ -31,8 +31,9 @@ extern "C"
 
 #include <glib.h>
 
-#include "qof.h"
+#include <qof.h>
-#include "gnc-engine.h"
+#include <gnc-engine.h>
+#include <kvp_frame.h>
 
 #include "gnc-backend-sql.h"
 

src/backend/xml/io-gncxml-v1.cpp

@@ -33,16 +33,17 @@ extern "C"
 
 #include <glib.h>
 
-#include "gnc-xml-helper.h"
+#include <gnc-xml-helper.h>
-#include "Account.h"
+#include <Account.h>
-#include "AccountP.h"
+#include <AccountP.h>
-#include "Query.h"
+#include <Query.h>
-#include "Scrub.h"
+#include <Scrub.h>
-#include "Transaction.h"
+#include <Transaction.h>
-#include "TransactionP.h"
+#include <TransactionP.h>
-#include "TransLog.h"
+#include <TransLog.h>
-#include "gnc-pricedb.h"
+#include <gnc-pricedb.h>
-#include "gnc-pricedb-p.h"
+#include <gnc-pricedb-p.h>
+#include <kvp_frame.h>
 #include "io-gncxml.h"
 
 #include "sixtp.h"

src/backend/xml/sixtp-dom-generators.cpp

@@ -29,6 +29,7 @@ extern "C"
 
 #include "gnc-xml-helper.h"
 #include <gnc-date.h>
+#include <kvp_frame.h>
 
 #include "sixtp-dom-generators.h"
 #include "sixtp-utils.h"

src/backend/xml/sixtp-dom-parsers.cpp

@@ -28,7 +28,8 @@ extern "C"
 #include <string.h>
 
 #include "gnc-xml-helper.h"
-#include "gnc-engine.h"
+#include <gnc-engine.h>
+#include <kvp_frame.h>
 #include "sixtp-utils.h"
 #include "sixtp-dom-parsers.h"
 }

src/backend/xml/test/test-file-stuff.h

@@ -7,11 +7,12 @@
 
 #include <glib.h>
 
-#include "gnc-commodity.h"
+#include <kvp_frame.h>
-#include "gnc-engine.h"
+#include <gnc-commodity.h>
-#include "gnc-xml-helper.h"
+#include <gnc-engine.h>
-#include "io-gncxml-gen.h"
+#include <gnc-xml-helper.h>
-#include "sixtp.h"
+#include <io-gncxml-gen.h>
+#include <sixtp.h>
 
 
 

src/engine/engine-helpers.c

@@ -37,8 +37,10 @@
 #include "gnc-session.h"
 #include "guile-mappings.h"
 #include "gnc-guile-utils.h"
-#include "qof.h"
+#include <qof.h>
-#include "qofbookslots.h"
+#include <kvp_frame.h>
+#include <qofbookslots.h>
+
 /** \todo Code dependent on the private query headers
 qofquery-p.h and qofquerycore-p.h may need to be modified.
 These files are temporarily exported for QOF 0.6.0 but

src/engine/kvp-scm.c

@@ -1,6 +1,7 @@
 #include "config.h"
 
-#include "qof.h"
+#include <qof.h>
+#include <kvp_frame.h>
 #include <libguile.h>
 #include "engine-helpers-guile.h"
 

src/engine/test-core/test-engine-stuff.c

@@ -28,6 +28,7 @@
 #include <string.h>
 #include <sys/stat.h>
 #include <unistd.h>
+#include <qof.h>
 #include <qofinstance-p.h>
 
 #include "Account.h"

src/engine/test-core/test-engine-stuff.h

@@ -10,6 +10,7 @@
 #include <stdint.h>
 
 #include "qof.h"
+#include <kvp_frame.h>
 #include "Query.h"
 #include "gnc-pricedb.h"
 #include "SchedXaction.h"

src/engine/test/utest-Account.c

@@ -25,6 +25,7 @@
 #include <unittest-support.h>
 #include <gnc-event.h>
 #include <gnc-gdate-utils.h>
+#include <kvp_frame.h>
 #include <qofinstance-p.h>
 /* Add specific headers for this class */
 #include "../Account.h"

src/engine/test/utest-Split.cpp

@@ -38,6 +38,7 @@ extern "C"
 #include <TransactionP.h>
 #include <gnc-lot.h>
 #include <gnc-event.h>
+#include <kvp_frame.h>
 #include <qofinstance-p.h>
 
 #ifdef HAVE_GLIB_2_38

src/engine/test/utest-Transaction.c

@@ -33,6 +33,7 @@
 #include "../gnc-lot.h"
 #include "../gnc-event.h"
 #include <qof.h>
+#include <kvp_frame.h>
 #include <qofbackend-p.h>
 
 #ifdef HAVE_GLIB_2_38

src/libqof/qof/kvp-value.hpp

@@ -28,6 +28,7 @@ extern "C"
 {
 #include "config.h"
 #include "qof.h"
+#include "kvp_frame.h"
 }
 #include <boost/version.hpp>
 #if BOOST_VERSION == 105600

src/libqof/qof/qofbook.cpp

@@ -56,6 +56,7 @@ extern "C"
 #include "qofid-p.h"
 #include "qofobject-p.h"
 #include "qofbookslots.h"
+#include "kvp_frame.h"
 
 static QofLogModule log_module = QOF_MOD_ENGINE;
 #define AB_KEY "hbci"

src/libqof/qof/qofbook.h

@@ -49,9 +49,9 @@ extern "C"
 #ifndef SWIG
 
 typedef struct _QofBookClass  QofBookClass;
+typedef struct KvpValueImpl KvpValue;
 
 #include "qofid.h"
-#include "kvp_frame.h"
 #include "qofinstance.h"
 
 /* --- type macros --- */

src/libqof/qof/qofinstance.cpp

@@ -38,6 +38,7 @@ extern "C"
 #include "qof.h"
 #include "qofbook-p.h"
 #include "qofid-p.h"
+#include "kvp_frame.h"
 #include "qofinstance-p.h"
 
 static QofLogModule log_module = QOF_MOD_ENGINE;

src/libqof/qof/qofinstance.h

@@ -46,7 +46,6 @@ typedef struct _QofBook       QofBook;
 #include "qofid.h"
 #include "guid.h"
 #include "gnc-date.h"
-#include "kvp_frame.h"
 #include "qof-gobject.h"
 
 /* --- type macros --- */
@@ -62,6 +61,8 @@ typedef struct _QofBook       QofBook;
 #define QOF_INSTANCE_GET_CLASS(o)    \
      (G_TYPE_INSTANCE_GET_CLASS ((o), QOF_TYPE_INSTANCE, QofInstanceClass))
 
+typedef struct KvpFrameImpl KvpFrame;
+
 struct QofInstance_s
 {
     GObject object;

src/libqof/qof/test/test-kvp_frame.c

@@ -27,13 +27,14 @@ extern "C"
 #include "config.h"
 #include <string.h>
 #include <glib.h>
-#include "unittest-support.h"
+#include <unittest-support.h>
-
+#include <qof.h>
+#include <kvp_frame.h>
 #ifdef __cplusplus
 }
 #endif
 
-#include "qof.h"
+
 
 static const gchar *suitename = "/qof/kvp_frame";
 void test_suite_kvp_frame ( void );

src/libqof/qof/test/test-qofinstance.c

@@ -24,6 +24,7 @@
 #include <unittest-support.h>
 #include "../qof.h"
 #include "../qofbackend-p.h"
+#include "../kvp_frame.h"
 
 static const gchar *suitename = "/qof/qofinstance";
 void test_suite_qofinstance ( void );