Include rearrangement.

diff --git a/hacks/check_includes b/hacks/check_includes
new file mode 100755 (executable)
index 0000000..bd0ea63
--- /dev/null
+++ b/hacks/check_includes
@@ -0,0 +1,40 @@
+#!/usr/bin/python
+
+import sys
+
+for a in sys.argv[1:]:
+    includes = []
+    for line in open(a, 'r'):
+        if line.startswith('#include'):
+            includes.append(line.strip()[9:])
+
+    std = ['<cstdio>', '<unistd.h>', '<stdexcept>', '<iostream>', '<algorithm>', '<fstream>', '<cstdlib>', '<iomanip>', '<stdint.h>', '<cmath>', '<cassert>', '<cstring>', '<mntent.h>', '<windows.h>', '<shlwapi.h>', '<sys/sysctl.h>', '<mach-o/dyld.h>', '<IOKit/pwr_mgt/IOPMLib.h>', '<sys/types.h>', '<ifaddrs.h>', '<netinet/in.h>', '<arpa/inet.h>']
+
+    current_group = 0
+    for i in includes:
+        if i == '"i18n.h"':
+            continue
+
+        if i in std:
+            group = 5
+        elif i.find("<boost/") != -1:
+            group = 4
+        elif i.find("<libxml++/") != -1 or i == '<glib.h>':
+            group = 3
+        elif i.find("<libcxml/") != -1 or i.find("<dcp/") != -1:
+            group = 2
+        elif i.find("\"wx/") != -1:
+            group = 1
+        else:
+            group = 0
+
+        print '%s: %d' % (i, group)
+
+        if group < current_group:
+            print '%s: first wrong order is %s' % (a, i)
+            sys.exit(1)
+
+        current_group = group
+
+
+

diff --git a/src/lib/audio_analysis.cc b/src/lib/audio_analysis.cc
index 9f92bdb503be802bcdee6aa400d0a1738b0defc3..681917fcf4c80272449ae73aad0447ba4bd57fb7 100644 (file)
--- a/src/lib/audio_analysis.cc
+++ b/src/lib/audio_analysis.cc
@@ -17,14 +17,14 @@
 
 */
 
+#include "audio_analysis.h"
+#include "cross.h"
+#include <boost/filesystem.hpp>
 #include <stdint.h>
 #include <cmath>
 #include <cassert>
 #include <cstdio>
 #include <iostream>
-#include <boost/filesystem.hpp>
-#include "audio_analysis.h"
-#include "cross.h"
 
 using std::ostream;
 using std::istream;

diff --git a/src/lib/audio_buffers.cc b/src/lib/audio_buffers.cc
index 56ca7a94b18c5860e3800f03740baa2c09480d59..d1003cc8cb32a32b4c5530e99e45e90d37a0aea1 100644 (file)
--- a/src/lib/audio_buffers.cc
+++ b/src/lib/audio_buffers.cc
@@ -17,11 +17,11 @@
 
 */
 
+#include "audio_buffers.h"
 #include <cassert>
 #include <cstring>
 #include <cmath>
 #include <stdexcept>
-#include "audio_buffers.h"
 
 using std::bad_alloc;
 using boost::shared_ptr;

diff --git a/src/lib/audio_content.cc b/src/lib/audio_content.cc
index 8458e3314c7be41b36aaccfdb8fdde437fe7b8da..c0e99b24cb5902ad9dbdd8d142c429f1fb7f6a7b 100644 (file)
--- a/src/lib/audio_content.cc
+++ b/src/lib/audio_content.cc
@@ -17,8 +17,6 @@
 
 */
 
-#include <libcxml/cxml.h>
-#include <dcp/raw_convert.h>
 #include "audio_content.h"
 #include "analyse_audio_job.h"
 #include "job_manager.h"
@@ -27,6 +25,8 @@
 #include "config.h"
 #include "frame_rate_change.h"
 #include "audio_processor.h"
+#include <libcxml/cxml.h>
+#include <dcp/raw_convert.h>
 
 #include "i18n.h"
 

diff --git a/src/lib/audio_decoder.cc b/src/lib/audio_decoder.cc
index f3251f306bf47284a71bc1d53917631e917c827b..31dcf2ef910289899b6776b3bfad4209698822a2 100644 (file)
--- a/src/lib/audio_decoder.cc
+++ b/src/lib/audio_decoder.cc
@@ -17,12 +17,12 @@
 
 */
 
-#include <iostream>
 #include "audio_decoder.h"
 #include "audio_buffers.h"
 #include "audio_processor.h"
 #include "resampler.h"
 #include "util.h"
+#include <iostream>
 
 #include "i18n.h"
 

diff --git a/src/lib/audio_filter.cc b/src/lib/audio_filter.cc
index 59b5684ea62e2cd7e447599b215c6d1b6a44fc0e..dbef25201c642f29f25ad2f8e6336cffa3b5b072 100644 (file)
--- a/src/lib/audio_filter.cc
+++ b/src/lib/audio_filter.cc
@@ -17,9 +17,9 @@
 
 */
 
-#include <cmath>
 #include "audio_filter.h"
 #include "audio_buffers.h"
+#include <cmath>
 
 using std::vector;
 using std::min;

diff --git a/src/lib/audio_mapping.cc b/src/lib/audio_mapping.cc
index e86e2e2ac6820499cd551ddaed17ffe8e7f30019..5b3e36f25046e0b7e3a1a83cf6fe29e971eefb1c 100644 (file)
--- a/src/lib/audio_mapping.cc
+++ b/src/lib/audio_mapping.cc
@@ -17,12 +17,12 @@
 
 */
 
-#include <libxml++/libxml++.h>
-#include <libcxml/cxml.h>
-#include <dcp/raw_convert.h>
 #include "audio_mapping.h"
 #include "util.h"
 #include "md5_digester.h"
+#include <libcxml/cxml.h>
+#include <dcp/raw_convert.h>
+#include <libxml++/libxml++.h>
 
 using std::list;
 using std::cout;

diff --git a/src/lib/cinema.cc b/src/lib/cinema.cc
index 62023618676a6f83de20bfde5f6be6acd651e1b7..282f5427fae472b859101c3d76dfbb30f224ad7d 100644 (file)
--- a/src/lib/cinema.cc
+++ b/src/lib/cinema.cc
@@ -17,9 +17,9 @@
 
 */
 
-#include <libxml++/libxml++.h>
-#include <libcxml/cxml.h>
 #include "cinema.h"
+#include <libcxml/cxml.h>
+#include <libxml++/libxml++.h>
 
 using std::list;
 using boost::shared_ptr;

diff --git a/src/lib/cinema_sound_processor.cc b/src/lib/cinema_sound_processor.cc
index 6a79051145e81e33ff998cacf1f4f7e21cb6c7d7..485af3fd66168848ab2b50734f00d1d6dfe30079 100644 (file)
--- a/src/lib/cinema_sound_processor.cc
+++ b/src/lib/cinema_sound_processor.cc
@@ -21,10 +21,10 @@
  *  @brief CinemaSoundProcessor class.
  */
 
-#include <iostream>
-#include <cassert>
 #include "cinema_sound_processor.h"
 #include "dolby_cp750.h"
+#include <iostream>
+#include <cassert>
 
 using namespace std;
 

diff --git a/src/lib/colour_conversion.cc b/src/lib/colour_conversion.cc
index 5ecbf4021ce4efb30f8ccb39de37eea6f127820b..6dd4ab87675008abee80a9f517049e351ce75f16 100644 (file)
--- a/src/lib/colour_conversion.cc
+++ b/src/lib/colour_conversion.cc
@@ -17,14 +17,14 @@
 
 */
 
-#include <libxml++/libxml++.h>
-#include <dcp/colour_matrix.h>
-#include <dcp/raw_convert.h>
-#include <libcxml/cxml.h>
 #include "config.h"
 #include "colour_conversion.h"
 #include "util.h"
 #include "md5_digester.h"
+#include <dcp/colour_matrix.h>
+#include <dcp/raw_convert.h>
+#include <libcxml/cxml.h>
+#include <libxml++/libxml++.h>
 
 #include "i18n.h"
 

diff --git a/src/lib/config.cc b/src/lib/config.cc
index 5f48262fbfb783a4e87c285ff9239685b0efc22b..a15825dc7e0f56fb3b511afe37d43d8d1ca3e0e8 100644 (file)
--- a/src/lib/config.cc
+++ b/src/lib/config.cc
@@ -17,16 +17,6 @@
 
 */
 
-#include <cstdlib>
-#include <fstream>
-#include <glib.h>
-#include <boost/filesystem.hpp>
-#include <boost/algorithm/string.hpp>
-#include <dcp/colour_matrix.h>
-#include <dcp/raw_convert.h>
-#include <dcp/signer.h>
-#include <dcp/certificate_chain.h>
-#include <libcxml/cxml.h>
 #include "config.h"
 #include "server.h"
 #include "scaler.h"
@@ -38,6 +28,16 @@
 #include "cinema.h"
 #include "util.h"
 #include "cross.h"
+#include <dcp/colour_matrix.h>
+#include <dcp/raw_convert.h>
+#include <dcp/signer.h>
+#include <dcp/certificate_chain.h>
+#include <libcxml/cxml.h>
+#include <glib.h>
+#include <boost/filesystem.hpp>
+#include <boost/algorithm/string.hpp>
+#include <cstdlib>
+#include <fstream>
 
 #include "i18n.h"
 

diff --git a/src/lib/content.cc b/src/lib/content.cc
index ec16b961015ee19a8eb6d1b487e297879adf53e5..bcfedb1f9e6e9019f788236eb0c440ef4f83d5f1 100644 (file)
--- a/src/lib/content.cc
+++ b/src/lib/content.cc
@@ -21,10 +21,6 @@
  *  @brief Content class.
  */
 
-#include <boost/thread/mutex.hpp>
-#include <libxml++/libxml++.h>
-#include <libcxml/cxml.h>
-#include <dcp/raw_convert.h>
 #include "content.h"
 #include "util.h"
 #include "content_factory.h"
@@ -33,6 +29,10 @@
 #include "film.h"
 #include "safe_stringstream.h"
 #include "job.h"
+#include <libcxml/cxml.h>
+#include <dcp/raw_convert.h>
+#include <libxml++/libxml++.h>
+#include <boost/thread/mutex.hpp>
 
 #include "i18n.h"
 

diff --git a/src/lib/content_factory.cc b/src/lib/content_factory.cc
index 16340adb43b0e74f8f14d4b2bc039e258d8e58a9..de940d0252f1651a53e143bdfefffa1e9dc2ef9d 100644 (file)
--- a/src/lib/content_factory.cc
+++ b/src/lib/content_factory.cc
@@ -21,7 +21,6 @@
  *  @brief Methods to create content objects.
  */
 
-#include <libcxml/cxml.h>
 #include "ffmpeg_content.h"
 #include "image_content.h"
 #include "sndfile_content.h"
@@ -29,6 +28,7 @@
 #include "dcp_content.h"
 #include "dcp_subtitle_content.h"
 #include "util.h"
+#include <libcxml/cxml.h>
 
 using std::string;
 using std::list;

diff --git a/src/lib/cross.cc b/src/lib/cross.cc
index d84c17c55b25318ef5b32a0a06eda8ee23ac1d16..e189222faeb94c031bb0a6a9d55744766de0aafe 100644 (file)
--- a/src/lib/cross.cc
+++ b/src/lib/cross.cc
@@ -17,11 +17,11 @@
 
 */
 
-#include <fstream>
-#include <boost/algorithm/string.hpp>
 #include "cross.h"
 #include "compose.hpp"
 #include "log.h"
+#include "exceptions.h"
+#include <boost/algorithm/string.hpp>
 #ifdef DCPOMATIC_LINUX
 #include <unistd.h>
 #include <mntent.h>
@@ -42,7 +42,7 @@
 #include <netinet/in.h>
 #include <arpa/inet.h>
 #endif
-#include "exceptions.h"
+#include <fstream>
 
 #include "i18n.h"
 

diff --git a/src/lib/dcp_content.cc b/src/lib/dcp_content.cc
index 28b7ac8620138ebda61caaac00c4d6b364046b6c..cd82ee6e9e811b54a6b7538d1a8a712d9ca22947 100644 (file)
--- a/src/lib/dcp_content.cc
+++ b/src/lib/dcp_content.cc
@@ -17,14 +17,14 @@
 
 */
 
-#include <dcp/dcp.h>
-#include <dcp/exceptions.h>
 #include "dcp_content.h"
 #include "dcp_examiner.h"
 #include "job.h"
 #include "film.h"
 #include "config.h"
 #include "compose.hpp"
+#include <dcp/dcp.h>
+#include <dcp/exceptions.h>
 
 #include "i18n.h"
 

diff --git a/src/lib/dcp_content_type.cc b/src/lib/dcp_content_type.cc
index e5466e1398936c1e4917efe11e95fa1cf5b12472..e659a9b88a7b7ce3027b09393b317f5ce7ebcf90 100644 (file)
--- a/src/lib/dcp_content_type.cc
+++ b/src/lib/dcp_content_type.cc
@@ -21,8 +21,8 @@
  *  @brief A description of the type of content for a DCP (e.g. feature, trailer etc.)
  */
 
-#include <cassert>
 #include "dcp_content_type.h"
+#include <cassert>
 
 #include "i18n.h"
 

diff --git a/src/lib/dcp_decoder.cc b/src/lib/dcp_decoder.cc
index adeadd98a3d8fb19d826e4ce663af2ad2a84a30c..74affe857e5f058133224a0e87be24c73dd5848d 100644 (file)
--- a/src/lib/dcp_decoder.cc
+++ b/src/lib/dcp_decoder.cc
@@ -17,6 +17,11 @@
 
 */
 
+#include "dcp_decoder.h"
+#include "dcp_content.h"
+#include "j2k_image_proxy.h"
+#include "image.h"
+#include "config.h"
 #include <dcp/dcp.h>
 #include <dcp/cpl.h>
 #include <dcp/reel.h>
@@ -27,11 +32,6 @@
 #include <dcp/mono_picture_frame.h>
 #include <dcp/stereo_picture_frame.h>
 #include <dcp/sound_frame.h>
-#include "dcp_decoder.h"
-#include "dcp_content.h"
-#include "j2k_image_proxy.h"
-#include "image.h"
-#include "config.h"
 
 using std::list;
 using std::cout;

diff --git a/src/lib/dcp_examiner.cc b/src/lib/dcp_examiner.cc
index 1e4cc899df96df614fd6a9b1413d3101245247a4..052c8bd76c9b35a827665a4ca92de069a7f4e8cc 100644 (file)
--- a/src/lib/dcp_examiner.cc
+++ b/src/lib/dcp_examiner.cc
@@ -17,6 +17,11 @@
 
 */
 
+#include "dcp_examiner.h"
+#include "dcp_content.h"
+#include "exceptions.h"
+#include "image.h"
+#include "config.h"
 #include <dcp/dcp.h>
 #include <dcp/cpl.h>
 #include <dcp/reel.h>
@@ -27,11 +32,6 @@
 #include <dcp/stereo_picture_mxf.h>
 #include <dcp/stereo_picture_frame.h>
 #include <dcp/sound_mxf.h>
-#include "dcp_examiner.h"
-#include "dcp_content.h"
-#include "exceptions.h"
-#include "image.h"
-#include "config.h"
 
 #include "i18n.h"
 

diff --git a/src/lib/dcp_subtitle_content.cc b/src/lib/dcp_subtitle_content.cc
index 1935a874f7870dadfd8a6517d9acb13f9b6d09cf..7be96ceeb7d1127b35dfb88a71e35389abffde25 100644 (file)
--- a/src/lib/dcp_subtitle_content.cc
+++ b/src/lib/dcp_subtitle_content.cc
@@ -17,9 +17,9 @@
 
 */
 
+#include "dcp_subtitle_content.h"
 #include <dcp/subtitle_content.h>
 #include <dcp/raw_convert.h>
-#include "dcp_subtitle_content.h"
 
 #include "i18n.h"
 

diff --git a/src/lib/dcp_subtitle_decoder.cc b/src/lib/dcp_subtitle_decoder.cc
index 20a9f32fee031ddd99fc03f19ab1d5182614e6f7..f61c69a7bd23783be3b9e9122227aa6e4a5bcc2a 100644 (file)
--- a/src/lib/dcp_subtitle_decoder.cc
+++ b/src/lib/dcp_subtitle_decoder.cc
@@ -17,9 +17,9 @@
 
 */
 
-#include <dcp/subtitle_content.h>
 #include "dcp_subtitle_decoder.h"
 #include "dcp_subtitle_content.h"
+#include <dcp/subtitle_content.h>
 
 using std::list;
 using std::cout;

diff --git a/src/lib/dcp_video.cc b/src/lib/dcp_video.cc
index cacba190f87603dfa5d12a1a27c953229af959a5..62340a4683eda8dbdeede0d3facbd6aedb6942c6 100644 (file)
--- a/src/lib/dcp_video.cc
+++ b/src/lib/dcp_video.cc
@@ -28,26 +28,6 @@
  *  of images that require encoding.
  */
 
-#include <stdint.h>
-#include <cstring>
-#include <cstdlib>
-#include <stdexcept>
-#include <cstdio>
-#include <iomanip>
-#include <iostream>
-#include <fstream>
-#include <unistd.h>
-#include <errno.h>
-#include <boost/array.hpp>
-#include <boost/asio.hpp>
-#include <boost/filesystem.hpp>
-#include <boost/lexical_cast.hpp>
-#include <dcp/gamma_lut.h>
-#include <dcp/xyz_frame.h>
-#include <dcp/rgb_xyz.h>
-#include <dcp/colour_matrix.h>
-#include <dcp/raw_convert.h>
-#include <libcxml/cxml.h>
 #include "film.h"
 #include "dcp_video.h"
 #include "config.h"
@@ -60,6 +40,26 @@
 #include "cross.h"
 #include "player_video.h"
 #include "encoded_data.h"
+#include <libcxml/cxml.h>
+#include <dcp/gamma_lut.h>
+#include <dcp/xyz_frame.h>
+#include <dcp/rgb_xyz.h>
+#include <dcp/colour_matrix.h>
+#include <dcp/raw_convert.h>
+#include <boost/array.hpp>
+#include <boost/asio.hpp>
+#include <boost/filesystem.hpp>
+#include <boost/lexical_cast.hpp>
+#include <stdint.h>
+#include <cstring>
+#include <cstdlib>
+#include <stdexcept>
+#include <cstdio>
+#include <iomanip>
+#include <iostream>
+#include <fstream>
+#include <unistd.h>
+#include <errno.h>
 
 #define LOG_GENERAL(...) _log->log (String::compose (__VA_ARGS__), Log::TYPE_GENERAL);
 

diff --git a/src/lib/encoder.cc b/src/lib/encoder.cc
index 96fda7cc243a859b9c5785c70eb33e44f997f570..da6c969052f4a4d8e59f37043263b2c3fa258851 100644 (file)
--- a/src/lib/encoder.cc
+++ b/src/lib/encoder.cc
@@ -21,9 +21,6 @@
  *  @brief Parent class for classes which can encode video and audio frames.
  */
 
-#include <iostream>
-#include <boost/lambda/lambda.hpp>
-#include <libcxml/cxml.h>
 #include "encoder.h"
 #include "util.h"
 #include "film.h"
@@ -36,6 +33,9 @@
 #include "server_finder.h"
 #include "player.h"
 #include "player_video.h"
+#include <libcxml/cxml.h>
+#include <boost/lambda/lambda.hpp>
+#include <iostream>
 
 #include "i18n.h"
 

diff --git a/src/lib/film.cc b/src/lib/film.cc
index f0ddd14a8a941810e06dc8c8905fd39b2e975fb4..cc75a05585fa35625d4b121cee5963215ae7851c 100644 (file)
--- a/src/lib/film.cc
+++ b/src/lib/film.cc
@@ -56,7 +56,6 @@
 #include <cstdlib>
 #include <iomanip>
 
-
 #include "i18n.h"
 
 using std::string;