fix include order for off_t

include/qpdf/Pl_Count.hh

@@ -11,8 +11,8 @@
 // This pipeline is reusable; i.e., it is safe to call write() after
 // calling finish().
 
-#include <qpdf/Pipeline.hh>
 #include <qpdf/Types.h>
+#include <qpdf/Pipeline.hh>
 
 class Pl_Count: public Pipeline
 {

include/qpdf/QPDF.hh

@@ -8,15 +8,15 @@
 #ifndef __QPDF_HH__
 #define __QPDF_HH__
 
+#include <qpdf/DLL.h>
+#include <qpdf/Types.h>
+
 #include <stdio.h>
 #include <string>
 #include <map>
 #include <list>
 #include <iostream>
 
-#include <qpdf/DLL.h>
-#include <qpdf/Types.h>
-
 #include <qpdf/QPDFXRefEntry.hh>
 #include <qpdf/QPDFObjectHandle.hh>
 #include <qpdf/QPDFTokenizer.hh>

include/qpdf/QPDFExc.hh

@@ -9,8 +9,9 @@
 #define __QPDFEXC_HH__
 
 #include <qpdf/DLL.h>
-#include <qpdf/Constants.h>
 #include <qpdf/Types.h>
+
+#include <qpdf/Constants.h>
 #include <stdexcept>
 
 class QPDFExc: public std::runtime_error

include/qpdf/QPDFObjectHandle.hh

@@ -8,14 +8,14 @@
 #ifndef __QPDFOBJECTHANDLE_HH__
 #define __QPDFOBJECTHANDLE_HH__
 
+#include <qpdf/DLL.h>
+#include <qpdf/Types.h>
+
 #include <string>
 #include <vector>
 #include <set>
 #include <map>
 
-#include <qpdf/DLL.h>
-#include <qpdf/Types.h>
-
 #include <qpdf/PointerHolder.hh>
 #include <qpdf/Buffer.hh>
 

include/qpdf/QPDFWriter.hh

@@ -12,6 +12,9 @@
 #ifndef __QPDFWRITER_HH__
 #define __QPDFWRITER_HH__
 
+#include <qpdf/DLL.h>
+#include <qpdf/Types.h>
+
 #include <stdio.h>
 #include <string>
 #include <list>
@@ -19,9 +22,7 @@
 #include <set>
 #include <map>
 
-#include <qpdf/DLL.h>
 #include <qpdf/Constants.h>
-#include <qpdf/Types.h>
 
 #include <qpdf/QPDFXRefEntry.hh>
 

include/qpdf/QUtil.hh

@@ -10,6 +10,7 @@
 
 #include <qpdf/DLL.h>
 #include <qpdf/Types.h>
+
 #include <string>
 #include <list>
 #include <stdexcept>

include/qpdf/Types.h

@@ -1,6 +1,10 @@
 #ifndef __QPDFTYPES_H__
 #define __QPDFTYPES_H__
 
+/* This file must be included before any system files.  It should be
+ * included right after <qpdf/DLL.h> within the library.
+ */
+
 /* Attempt to provide off_t and size_t on any recent platform.  To
  * make cross compilation easier and to be more portable across
  * platforms, QPDF avoids having any public header files use the

include/qpdf/qpdf-c.h

@@ -70,8 +70,8 @@
  */
 
 #include <qpdf/DLL.h>
-#include <qpdf/Constants.h>
 #include <qpdf/Types.h>
+#include <qpdf/Constants.h>
 
 #ifdef __cplusplus
 extern "C" {

libqpdf/qpdf/MD5.hh

@@ -1,7 +1,6 @@
 #ifndef __MD5_HH__
 #define __MD5_HH__
 
-#include <string>
 #include <qpdf/DLL.h>
 #include <qpdf/qpdf-config.h>
 #ifdef HAVE_INTTYPES_H
@@ -10,6 +9,7 @@
 #ifdef HAVE_STDINT_H
 # include <stdint.h>
 #endif
+#include <string>
 
 class MD5
 {

libqpdf/qpdf/PCRE.hh

@@ -5,6 +5,8 @@
 #ifndef __PCRE_HH__
 #define __PCRE_HH__
 
+#include <qpdf/DLL.h>
+
 #ifdef _WIN32
 # define PCRE_STATIC
 #endif
@@ -12,8 +14,6 @@
 #include <string>
 #include <stdexcept>
 
-#include <qpdf/DLL.h>
-
 // Note: this class does not encapsulate all features of the PCRE
 // package -- only those that I actually need right now are here.
 

libqpdf/qpdf/QPDF_Stream.hh

@@ -1,8 +1,9 @@
 #ifndef __QPDF_STREAM_HH__
 #define __QPDF_STREAM_HH__
 
-#include <qpdf/QPDFObject.hh>
+#include <qpdf/Types.h>
 
+#include <qpdf/QPDFObject.hh>
 #include <qpdf/QPDFObjectHandle.hh>
 
 class Pipeline;