Cleanup includes to reflect actual dependencies

src/AudioReaderSource.cpp

@@ -30,6 +30,10 @@
 
 #include "AudioReaderSource.h"
 #include "Exceptions.h"
+#include "Frame.h"
+#include "ZmqLogger.h"
+
+#include <OpenShotAudio.h>
 
 using namespace std;
 using namespace openshot;

src/AudioReaderSource.h

@@ -31,8 +31,8 @@
 #ifndef OPENSHOT_AUDIOREADERSOURCE_H
 #define OPENSHOT_AUDIOREADERSOURCE_H
 
-#include <iomanip>
 #include "ReaderBase.h"
+
 #include <OpenShotAudio.h>
 
 /// This namespace is the default namespace for all code in the openshot library

src/CacheBase.h

@@ -32,11 +32,13 @@
 #define OPENSHOT_CACHE_BASE_H
 
 #include <memory>
-#include <cstdlib>
-#include "Frame.h"
+
 #include "Json.h"
 
+#include <OpenShotAudio.h>
+
 namespace openshot {
+	class Frame;
 
 	/**
 	 * @brief All cache managers in libopenshot are based on this CacheBase class

src/CacheDisk.cpp

@@ -30,7 +30,9 @@
 
 #include "CacheDisk.h"
 #include "Exceptions.h"
+#include "Frame.h"
 #include "QtUtilities.h"
+
 #include <Qt>
 #include <QString>
 #include <QTextStream>

src/CacheDisk.h

@@ -34,11 +34,13 @@
 #include <map>
 #include <deque>
 #include <memory>
+
 #include "CacheBase.h"
-#include "Frame.h"
+
 #include <QDir>
 
 namespace openshot {
+	class Frame;
 
 	/**
 	 * @brief This class is a disk-based cache manager for Frame objects.

src/CacheMemory.cpp

@@ -30,6 +30,7 @@
 
 #include "CacheMemory.h"
 #include "Exceptions.h"
+#include "Frame.h"
 
 using namespace std;
 using namespace openshot;

src/CacheMemory.h

@@ -34,10 +34,11 @@
 #include <map>
 #include <deque>
 #include <memory>
+
 #include "CacheBase.h"
-#include "Frame.h"
 
 namespace openshot {
+	class Frame;
 
 	/**
 	 * @brief This class is a memory-based cache manager for Frame objects.

src/ChunkReader.h

@@ -31,16 +31,16 @@
 #ifndef OPENSHOT_CHUNK_READER_H
 #define OPENSHOT_CHUNK_READER_H
 
-#include "ReaderBase.h"
 #include <string>
 #include <memory>
 
-#include "Frame.h"
+#include "ReaderBase.h"
 #include "Json.h"
 #include "CacheMemory.h"
 
 namespace openshot
 {
+	class Frame;
 
 	/**
 	 * @brief This struct holds the location of a frame within a chunk.
@@ -137,7 +137,7 @@ namespace openshot
 		void SetChunkSize(int64_t new_size) { chunk_size = new_size; };
 
 		/// Get the cache object used by this reader (always return NULL for this reader)
-		openshot::CacheMemory* GetCache() override { return NULL; };
+		openshot::CacheMemory* GetCache() override { return nullptr; };
 
 		/// @brief Get an openshot::Frame object for a specific frame number of this reader.
 		/// @returns				The requested frame (containing the image and audio)

src/DummyReader.cpp

@@ -30,6 +30,7 @@
 
 #include "DummyReader.h"
 #include "Exceptions.h"
+#include "Frame.h"
 
 using namespace openshot;
 

src/ImageReader.cpp

@@ -33,6 +33,7 @@
 
 #include "ImageReader.h"
 #include "Exceptions.h"
+#include "Frame.h"
 
 using namespace openshot;
 

src/Qt/VideoCacheThread.h

@@ -34,6 +34,7 @@
 #include "../OpenMPUtilities.h"
 #include "../ReaderBase.h"
 #include "../RendererBase.h"
+#include "../CacheBase.h"
 
 namespace openshot
 {

src/QtHtmlReader.cpp

@@ -32,6 +32,8 @@
 
 #include "QtHtmlReader.h"
 #include "Exceptions.h"
+#include "Frame.h"
+
 #include <QImage>
 #include <QPainter>
 #include <QTextDocument>

src/QtImageReader.cpp

@@ -34,11 +34,12 @@
 #include "Clip.h"
 #include "CacheMemory.h"
 #include "Timeline.h"
-#include <QtCore/QString>
-#include <QtGui/QImage>
-#include <QtGui/QPainter>
-#include <QtGui/QIcon>
-#include <QtGui/QImageReader>
+
+#include <QString>
+#include <QImage>
+#include <QPainter>
+#include <QIcon>
+#include <QImageReader>
 
 #if USE_RESVG == 1
 	// If defined and found in CMake, utilize the libresvg for parsing

src/QtImageReader.h

@@ -31,15 +31,14 @@
 #ifndef OPENSHOT_QIMAGE_READER_H
 #define OPENSHOT_QIMAGE_READER_H
 
-#include <cmath>
-#include <ctime>
-#include <iostream>
-#include <omp.h>
-#include <stdio.h>
 #include <memory>
 
 #include "ReaderBase.h"
 
+#include <QImage>
+#include <QSize>
+#include <QString>
+
 namespace openshot
 {
 	// Forward decl

src/QtTextReader.cpp

@@ -32,6 +32,8 @@
 
 #include "QtTextReader.h"
 #include "Exceptions.h"
+#include "Frame.h"
+
 #include <QImage>
 #include <QPainter>
 

src/QtTextReader.h

@@ -35,15 +35,12 @@
 
 #include "ReaderBase.h"
 
-#include <cmath>
-#include <ctime>
-#include <iostream>
-#include <omp.h>
-#include <stdio.h>
 #include <memory>
+
 #include "CacheMemory.h"
 #include "Enums.h"
 
+#include <QFont>
 
 class QImage;
 

src/ReaderBase.cpp

@@ -29,6 +29,8 @@
  */
 
 #include "ReaderBase.h"
+#include "ClipBase.h"
+#include "Frame.h"
 
 using namespace openshot;
 

src/ReaderBase.h

@@ -31,26 +31,23 @@
 #ifndef OPENSHOT_READER_BASE_H
 #define OPENSHOT_READER_BASE_H
 
-#include <iostream>
-#include <iomanip>
+#include <map>
 #include <memory>
-#include <cstdlib>
-#include <sstream>
-#include "CacheMemory.h"
+#include <string>
+#include <vector>
+
 #include "ChannelLayouts.h"
-#include "ClipBase.h"
 #include "Fraction.h"
-#include "Frame.h"
 #include "Json.h"
-#include "ZmqLogger.h"
-#include <QString>
-#include <QGraphicsItem>
-#include <QGraphicsScene>
-#include <QGraphicsPixmapItem>
-#include <QPixmap>
+
+#include <OpenShotAudio.h>
 
 namespace openshot
 {
+	class ClipBase;
+	class CacheBase;
+	class Frame;
+
 	/**
 	 * @brief This struct contains info about a media file, such as height, width, frames per second, etc...
 	 *

src/TextReader.cpp

@@ -33,6 +33,7 @@
 
 #include "TextReader.h"
 #include "Exceptions.h"
+#include "Frame.h"
 
 using namespace openshot;
 

src/effects/Caption.cpp

@@ -38,6 +38,8 @@
 #include <QRect>
 #include <QPen>
 #include <QBrush>
+#include <QPainter>
+#include <QPainterPath>
 
 using namespace openshot;