WIP - more .h cleanup

addons/ofxAssimpModelLoader/src/ofxAssimpTexture.cpp

@@ -7,6 +7,7 @@
 
 #include "ofxAssimpTexture.h"
 #include "ofLog.h"
+#include "ofUtils.h"
 
 void ofxAssimpTexture::setup(const ofTexture & texture, const of::filesystem::path & texturePath, bool bTexRepeat) {
 	this->texture = texture;

libs/openFrameworks/gl/ofCubeMap.cpp

@@ -12,6 +12,7 @@
 #include "ofGLUtils.h"
 #include "ofGLProgrammableRenderer.h"
 #include "ofCubeMapShaders.h"
+#include "ofFbo.h"
 #include "ofConstants.h"
 
 #define GLM_FORCE_CTOR_INIT
@@ -31,6 +32,9 @@ ofVboMesh ofCubeMap::sCubeMesh;
 ofShader ofCubeMap::shaderBrdfLUT;
 ofTexture ofCubeMap::sBrdfLutTex;
 
+void ofCubeMap::setExposure(float aExposure) {
+	data->exposure=ofClamp(aExposure, 0.0f, 1.0f);
+}
 
 // texture management copied from ofTexture
 static std::map<GLuint,int> & getTexturesIndex(){

libs/openFrameworks/gl/ofCubeMap.h

@@ -6,8 +6,9 @@
 
 #pragma once
 
-#include "ofFbo.h"
-#include "ofVboMesh.h"
+#include "glm/mat4x4.hpp"
+
+class ofVboMesh;
 
 class ofShader;
 class ofGLProgrammableRenderer;
@@ -16,8 +17,8 @@ class ofCubeMap {
 public:
 
 	struct ofCubeMapSettings {
-		of::filesystem::path filePath = "";
-		of::filesystem::path cacheDirectory = "";
+		of::filesystem::path filePath { "" };
+		of::filesystem::path cacheDirectory { "" };
 
 		bool overwriteCache = false;
 		bool useCache = false;
@@ -105,7 +106,7 @@ class ofCubeMap {
 	GLuint getTextureId();
 	bool isHdr();
 
-	void setExposure(float aExposure) {data->exposure=ofClamp(aExposure, 0.0f, 1.0f);}
+	void setExposure(float aExposure);
 	float getExposure() { return data->exposure; }
 
 	void setUseBrdfLutTexture( bool ab );

libs/openFrameworks/gl/ofFbo.h

@@ -1,5 +1,6 @@
 #pragma once
 
+// depthBufferTex only
 #include "ofTexture.h"
 #include "ofGLBaseTypes.h"
 

libs/openFrameworks/gl/ofShader.h

@@ -7,7 +7,6 @@
  make sure to catch and report that error.
  */
 
-#include "ofFileUtils.h"
 #include "ofConstants.h"
 #include "glm/fwd.hpp"
 #include <unordered_map>

libs/openFrameworks/utils/ofFileUtils.cpp

@@ -1,5 +1,6 @@
 #include "ofFileUtils.h"
 #include "ofLog.h"
+#include "ofUtils.h"
 
 #ifndef TARGET_WIN32
 	#include <pwd.h>

libs/openFrameworks/utils/ofFpsCounter.h

@@ -1,5 +1,6 @@
 #pragma once
 
+// ofTime only
 #include "ofUtils.h"
 #include "ofConstants.h"
 #include <queue>

libs/openFrameworks/utils/ofLog.h

@@ -1,8 +1,8 @@
 #pragma once
 
 #include "ofFileUtils.h"
-#include "ofUtils.h"
 #include "ofConstants.h"
+#include "ofUtils.h" // ofVAArgsToString
 #include <sstream>
 
 /// \file

libs/openFrameworks/utils/ofSystemUtils.h

@@ -5,19 +5,19 @@
 /// \class ofFileDialogResult
 /// \brief file dialog box selection or save result
 class ofFileDialogResult{
-	public:
-		ofFileDialogResult();
-		
-		/// \return the name of the selected file or directory, if set
-		/// currently returns only 1 file, this may change in the future
-		std::string getName();
-	
-		/// \return the full path of the selected file or directory, if set
-		std::string getPath();
-	
-		std::string filePath; ///< full path to selected file or directory
-		std::string fileName; ///< selected file or directory name
-		bool bSuccess; ///< true if the dialog action was successful, aka file select not cancel
+public:
+	ofFileDialogResult();
+
+	/// \return the name of the selected file or directory, if set
+	/// currently returns only 1 file, this may change in the future
+	std::string getName();
+
+	/// \return the full path of the selected file or directory, if set
+	std::string getPath();
+
+	std::string filePath; ///< full path to selected file or directory
+	std::string fileName; ///< selected file or directory name
+	bool bSuccess; ///< true if the dialog action was successful, aka file select not cancel
 };
 
 /// \brief show an error message in an alert dialog box

libs/openFrameworks/utils/ofURLFileLoader.h

@@ -1,6 +1,7 @@
 #pragma once
 
 #include "ofEvents.h"
+// ofBuffer only
 #include "ofFileUtils.h"
 #include "ofConstants.h"
 #include <map>

libs/openFrameworks/utils/ofXml.h

@@ -1,7 +1,6 @@
 #pragma once
 
 #include "ofParameter.h"
-#include "ofConstants.h"
 #include "pugixml.hpp"
 
 template<typename It>