of::filesystem PR

addons/ofxAndroid/src/ofAppAndroidWindow.cpp

@@ -266,7 +266,7 @@ Java_cc_openframeworks_OFAndroid_setAppDataDir( JNIEnv*  env, jobject  thiz, jst
 	jboolean iscopy;
 	const char *mfile = env->GetStringUTFChars(data_dir, &iscopy);
 	__android_log_print(ANDROID_LOG_INFO,"ofAppAndroidWindow",("setting app dir name to: \"" + string(mfile) + "\"").c_str());
-    ofSetDataPathRoot(string(mfile)+"/");
+    ofSetDataPathRoot({ string(mfile)+"/" });
     env->ReleaseStringUTFChars(data_dir, mfile);
 }
 

addons/ofxAndroid/src/ofxAndroidSoundPlayer.cpp

@@ -43,7 +43,7 @@ ofxAndroidSoundPlayer::~ofxAndroidSoundPlayer(){
 
 
 //------------------------------------------------------------
-bool ofxAndroidSoundPlayer::load(const std::filesystem::path& fileName, bool stream){
+bool ofxAndroidSoundPlayer::load(const of::filesystem::path& fileName, bool stream){
 	if(!javaSoundPlayer){
 		ofLogError("ofxAndroidSoundPlayer") << "loadSound(): java SoundPlayer not loaded";
 		return false;

addons/ofxAndroid/src/ofxAndroidSoundPlayer.h

@@ -8,7 +8,7 @@ class ofxAndroidSoundPlayer: public ofBaseSoundPlayer{
 	ofxAndroidSoundPlayer();
 	virtual ~ofxAndroidSoundPlayer();
 
-	bool load(const std::filesystem::path& fileName, bool stream = false);
+	bool load(const of::filesystem::path& fileName, bool stream = false);
 	void unload();
 	void play();
 	void stop();

addons/ofxAssimpModelLoader/src/ofxAssimpModelLoader.cpp

@@ -388,20 +388,20 @@ void ofxAssimpModelLoader::loadGLResources(){
 				auto ogPath = texPathStr;
 				bool bHasEmbeddedTexture = false;
 
-				string modelFolder = ofFilePath::getEnclosingDirectory( file.path() );
-				string relTexPath = ofFilePath::getEnclosingDirectory(texPathStr,false);
-				string texFile = ofFilePath::getFileName(texPathStr);
-				string realPath = ofFilePath::join(ofFilePath::join(modelFolder, relTexPath), texFile);
+				auto modelFolder = ofFilePath::getEnclosingDirectory( file.path() );
+				auto relTexPath = ofFilePath::getEnclosingDirectory(texPathStr,false);
+				auto realPath = modelFolder / of::filesystem::path{ texPathStr };
+
 
 #ifndef TARGET_LINUX_ARM
 				if(bTryEmbed || ofFile::doesFileExist(realPath) == false) {
 					auto embeddedTexture = scene->GetEmbeddedTexture(ogPath.c_str());
 					if( embeddedTexture ){
 						bHasEmbeddedTexture = true;
-						ofLogVerbose("ofxAssimpModelLoader") << "loadGLResource() texture " << texFile << " is embedded ";
+						ofLogVerbose("ofxAssimpModelLoader") << "loadGLResource() texture " << realPath.filename() << " is embedded ";
 					}else{
 						ofLogError("ofxAssimpModelLoader") << "loadGLResource(): texture doesn't exist: \""
-						<< file.getFileName() + "\" in \"" << realPath << "\"";
+						<< file.getFileName() + "\" in \"" << realPath.string() << "\"";
 					}
 				}
 #endif
@@ -419,7 +419,7 @@ void ofxAssimpModelLoader::loadGLResources(){
 					meshHelper.addTexture(assimpTexture);
 
 					ofLogVerbose("ofxAssimpModelLoader") << "loadGLResource(): texture already loaded: \""
-					<< file.getFileName() + "\" from \"" << realPath << "\"" << " adding texture as " << assimpTexture.getTextureTypeAsString() ;
+					<< file.getFileName() + "\" from \"" << realPath.string() << "\"" << " adding texture as " << assimpTexture.getTextureTypeAsString() ;
 				} else {
 
 					shared_ptr<ofTexture> texture = std::make_shared<ofTexture>();
@@ -463,7 +463,7 @@ void ofxAssimpModelLoader::loadGLResources(){
 						ofLogVerbose("ofxAssimpModelLoader") << "loadGLResource(): texture " << tmpTex.getTextureTypeAsString() << " loaded, dimensions: " << texture->getWidth() << "x" << texture->getHeight();
 					}else{
 						ofLogError("ofxAssimpModelLoader") << "loadGLResource(): couldn't load texture: \""
-						<< file.getFileName() + "\" from \"" << realPath << "\"";
+						<< file.getFileName() + "\" from \"" << realPath.string() << "\"";
 					}
 				}
 			}

addons/ofxAssimpModelLoader/src/ofxAssimpModelLoader.h

@@ -19,6 +19,7 @@
 #include "ofxAssimpTexture.h"
 #include "ofMesh.h"
 #include "ofMath.h"
+#include "ofConstants.h"
 
 #include <assimp/Importer.hpp>
 
@@ -177,7 +178,10 @@ class ofxAssimpModelLoader{
 		glm::mat4 modelMatrix;
 
 		std::vector<ofLight> lights;
-		std::map<std::string,std::shared_ptr<ofTexture>> textures;
+		std::map<
+			of::filesystem::path,
+			std::shared_ptr<ofTexture>
+		> textures;
 		std::vector<ofxAssimpMeshHelper> modelMeshes;
 		std::vector<ofxAssimpAnimation> animations;
 		int currentAnimation; // DEPRECATED - to be removed with deprecated animation functions.

addons/ofxAssimpModelLoader/src/ofxAssimpTexture.cpp

@@ -8,7 +8,7 @@
 #include "ofxAssimpTexture.h"
 #include "ofLog.h"
 
-void ofxAssimpTexture::setup(const ofTexture & texture, std::string texturePath, bool bTexRepeat) {
+void ofxAssimpTexture::setup(const ofTexture & texture, const of::filesystem::path & texturePath, bool bTexRepeat) {
 	this->texture = texture;
 	if( bTexRepeat ){
 		this->texture.setTextureWrap(GL_REPEAT, GL_REPEAT);
@@ -42,7 +42,7 @@ ofTexture & ofxAssimpTexture::getTextureRef() {
 	return texture;
 }
 
-std::string ofxAssimpTexture::getTexturePath() {
+of::filesystem::path ofxAssimpTexture::getTexturePath() {
 	return texturePath;
 }
 

addons/ofxAssimpModelLoader/src/ofxAssimpTexture.h

@@ -14,10 +14,10 @@ class ofxAssimpTexture {
 
 public:
 
-	void setup(const ofTexture & texture, std::string texturePath, bool bTexRepeat = true);
+	void setup(const ofTexture & texture, const of::filesystem::path & texturePath, bool bTexRepeat = true);
 
 	ofTexture & getTextureRef();
-	std::string getTexturePath();
+	of::filesystem::path getTexturePath();
 	bool hasTexture();
 
 	void setTextureType(aiTextureType aTexType);
@@ -27,7 +27,7 @@ class ofxAssimpTexture {
 private:
 
 	ofTexture texture;
-	std::string texturePath;
+	of::filesystem::path texturePath;
 	aiTextureType textureType;
 	std::string mTexTypeStr;
 };

addons/ofxEmscripten/src/ofxEmscriptenURLFileLoader.cpp

@@ -22,20 +22,20 @@ ofHttpResponse ofxEmscriptenURLFileLoader::get(const string & url){
 	return ofHttpResponse();
 }
 
-int ofxEmscriptenURLFileLoader::getAsync(const string &  url, const string &  name){
+int ofxEmscriptenURLFileLoader::getAsync(const string & url, const string & name){
 	ofHttpRequest * req = new ofHttpRequest(url,name,false);
 #if __EMSCRIPTEN_major__>1 || (__EMSCRIPTEN_major__==1 && __EMSCRIPTEN_minor__>22)
 	emscripten_async_wget2_data(url.c_str(), "GET", "", req, true, &onload_cb, &onerror_cb, NULL);
 #endif
 	return req->getId();
 }
 
-ofHttpResponse ofxEmscriptenURLFileLoader::saveTo(const string &  url, const std::filesystem::path &  path){
+ofHttpResponse ofxEmscriptenURLFileLoader::saveTo(const string & url, const of::filesystem::path & path){
 	saveAsync(url,path);
 	return ofHttpResponse();
 }
 
-int ofxEmscriptenURLFileLoader::saveAsync(const string &  url, const std::filesystem::path &  path){
+int ofxEmscriptenURLFileLoader::saveAsync(const string & url, const of::filesystem::path & path){
 	ofHttpRequest * req = new ofHttpRequest(url,url,true);
 #if __EMSCRIPTEN_major__>1 || (__EMSCRIPTEN_major__==1 && __EMSCRIPTEN_minor__>22)
 	emscripten_async_wget2(url.c_str(), path.c_str(), "GET", "", req, &onload_file_cb, &onerror_file_cb, NULL);

addons/ofxEmscripten/src/ofxEmscriptenURLFileLoader.h

@@ -11,10 +11,10 @@ class ofxEmscriptenURLFileLoader: public ofBaseURLFileLoader {
 public:
 	ofxEmscriptenURLFileLoader();
 	virtual ~ofxEmscriptenURLFileLoader();
-	ofHttpResponse get(const std::string &  url);
-	int getAsync(const std::string &  url, const std::string &  name=""); // returns id
-	ofHttpResponse saveTo(const std::string &  url, const std::filesystem::path &  path);
-	int saveAsync(const std::string &  url, const std::filesystem::path &  path);
+	ofHttpResponse get(const std::string & url);
+	int getAsync(const std::string & url, const std::string & name=""); // returns id
+	ofHttpResponse saveTo(const std::string & url, const of::filesystem::path & path);
+	int saveAsync(const std::string & url, const of::filesystem::path & path);
 	ofHttpResponse handleRequest(const ofHttpRequest & request);
 	int handleRequestAsync(const ofHttpRequest & request);
 	void remove(int id);

addons/ofxOpenCv/src/ofxCvHaarFinder.cpp

@@ -49,6 +49,7 @@ void ofxCvHaarFinder::setNeighbors(unsigned neighbors) {
 	this->neighbors = neighbors;
 }
 
+// FIXME: convert to of::filesystem::path
 void ofxCvHaarFinder::setup(std::string haarFile) {
 
 	this->haarFile = haarFile;

addons/ofxPoco/src/ofxXmlPoco.cpp

@@ -42,8 +42,8 @@ ofxXmlPoco::ofxXmlPoco(){
 }
 
 
-bool ofxXmlPoco::load(const std::filesystem::path & path){
-	ofFile file(path, ofFile::ReadOnly);
+bool ofxXmlPoco::load(const of::filesystem::path & filePath){
+	ofFile file(filePath, ofFile::ReadOnly);
 	if(!file.exists()){
 		ofLogError("ofxXmlPoco") << "couldn't load, \"" << file.getFileName() << "\" not found";
 		return false;
@@ -53,10 +53,10 @@ bool ofxXmlPoco::load(const std::filesystem::path & path){
 }
 
 
-bool ofxXmlPoco::save(const std::filesystem::path & path){
+bool ofxXmlPoco::save(const of::filesystem::path & filePath){
     ofBuffer buffer;
     buffer.set(toString());
-    ofFile file(path, ofFile::WriteOnly);
+    ofFile file(filePath, ofFile::WriteOnly);
     return file.writeFromBuffer(buffer);
 }
 

addons/ofxPoco/src/ofxXmlPoco.h

@@ -41,8 +41,8 @@ class ofxXmlPoco{
     ofxXmlPoco( const ofxXmlPoco& rhs );
     const ofxXmlPoco& operator =( const ofxXmlPoco& rhs );
 
-    bool load(const std::filesystem::path & path);
-    bool save(const std::filesystem::path & path);
+    bool load(const of::filesystem::path & filePath);
+    bool save(const of::filesystem::path & filePath);
 
     bool            addChild( const std::string& path );
     void            addXml( ofxXmlPoco& xml, bool copyAll = false);

addons/ofxSvg/src/ofxSvg.cpp

@@ -2,7 +2,8 @@
 #include "ofConstants.h"
 #include <locale>
 
-using namespace std;
+using std::string;
+using std::vector;
 
 extern "C"{
 	#include "svgtiny.h"
@@ -26,18 +27,30 @@ ofPath & ofxSvg::getPathAt(int n){
 	return paths[n];
 }
 
-void ofxSvg::load(std::string path){
-	path = ofToDataPath(path);
+void ofxSvg::load(of::filesystem::path fileName){
+	// fileName = ofToDataPath(fileName);
+	std::string file = ofToDataPath(fileName);
 
-	if(path.compare("") == 0){
-		ofLogError("ofxSVG") << "load(): path does not exist: \"" << path << "\"";
+	// FIXME: I think this is the equivalent of .empty() which is simpler.
+	// maybe use file exists to check instead?
+	// if(fileName.compare("") == 0){
+	// 	ofLogError("ofxSVG") << "load(): path does not exist: \"" << fileName << "\"";
+	// 	return;
+	// }
+
+	// ofBuffer buffer = ofBufferFromFile(fileName);
+
+	// loadFromString(buffer.getText(), fileName);
+
+	if(file.compare("") == 0){
+		ofLogError("ofxSVG") << "load(): path does not exist: \"" << file << "\"";
 		return;
 	}
 
-	ofBuffer buffer = ofBufferFromFile(path);
-
-	loadFromString(buffer.getText(), path);
+	ofBuffer buffer = ofBufferFromFile(file);
 
+	loadFromString(buffer.getText(), file);
+
 }
 
 void ofxSvg::loadFromString(std::string stringdata, std::string urlstring){

addons/ofxSvg/src/ofxSvg.h

@@ -25,7 +25,7 @@ class ofxSvg {
 		/// \brief Loads an SVG file from the provided filename.
 		///
 		/// ~~~~
-		void load(std::string path);
+		void load(of::filesystem::path fileName);
 
 		/// \brief Loads an SVG from a text string.
 		///

addons/ofxUnitTests/src/ofxUnitTests.h

@@ -228,9 +228,9 @@ class ofxUnitTestsApp: public ofBaseApp{
         const std::string APPVEYOR_API_URL = "APPVEYOR_API_URL";
         if(ofGetEnv(APPVEYOR_API_URL)!=""){
             //ofSystem("appveyor AddTest -Name " + projectName.string() + " -Framework ofxUnitTests -FileName " + exeName.string() + " -Outcome " + (passed?"Passed":"Failed") + " -Duration " + ofToString(now-then));
-            auto projectDir = std::filesystem::canonical(std::filesystem::path(ofFilePath::getCurrentExeDir()) / "..");
+            auto projectDir = of::filesystem::canonical(of::filesystem::path(ofFilePath::getCurrentExeDir()) / "..");
             auto projectName = projectDir.stem();
-            auto exeName = std::filesystem::path(ofFilePath::getCurrentExePath()).filename();
+            auto exeName = of::filesystem::path(ofFilePath::getCurrentExePath()).filename();
             auto stdOut = logger->getStdOut();
             ofStringReplace(stdOut, "\\", "\\\\");
             ofStringReplace(stdOut, "\"", "\\\"");

addons/ofxXmlSettings/src/ofxXmlSettings.cpp

@@ -72,8 +72,8 @@ void ofxXmlSettings::clear(){
 
 //---------------------------------------------------------
 bool ofxXmlSettings::load(const string& xmlFile){
-	string fullXmlFile = ofToDataPath(xmlFile);
-	bool loadOkay = doc.LoadFile(fullXmlFile);
+	auto fullXmlFile = ofToDataPath(xmlFile);
+	bool loadOkay = doc.LoadFile(fullXmlFile.c_str());
 
 	//theo removed bool check as it would
 	//return false if the file exists but was
@@ -88,8 +88,8 @@ bool ofxXmlSettings::load(const string& xmlFile){
 
 //---------------------------------------------------------
 bool ofxXmlSettings::save(const string& xmlFile){
-	string fullXmlFile = ofToDataPath(xmlFile);
-	return doc.SaveFile(fullXmlFile);
+	auto fullXmlFile = ofToDataPath(xmlFile);
+	return doc.SaveFile(fullXmlFile.c_str());
 }
 
 //---------------------------------------------------------

addons/ofxXmlSettings/src/ofxXmlSettings.h

@@ -8,7 +8,6 @@
 #include "tinyxml.h"
 #endif
 
-using namespace std;
 
 /*
 	Q: what is the which = 0 argument?

addons/ofxiOS/src/core/ofxiOSAppDelegate.mm

@@ -62,7 +62,7 @@ - (void)applicationDidFinishLaunching:(UIApplication *)application {
     currentScreenIndex = 0;
 
     // set the root application path
-    ofSetDataPathRoot([[NSString stringWithFormat:@"%@/", [[NSBundle mainBundle] resourcePath]] cStringUsingEncoding:NSUTF8StringEncoding]);
+    ofSetDataPathRoot(of::filesystem::path([[NSString stringWithFormat:@"%@/", [[NSBundle mainBundle] resourcePath]] cStringUsingEncoding:NSUTF8StringEncoding]));
 
 	// show or hide status bar depending on OF_WINDOW or OF_FULLSCREEN
     [[UIApplication sharedApplication] setStatusBarHidden:(ofxiOSGetOFWindow()->getWindowMode() == OF_FULLSCREEN)];

addons/ofxiOS/src/sound/ofxOpenALSoundPlayer.cpp

@@ -93,7 +93,7 @@ ofxOpenALSoundPlayer::~ofxOpenALSoundPlayer() {
 
 //--------------------------------------------------------------
 
-bool ofxOpenALSoundPlayer::load(const std::filesystem::path& filePath, bool stream) {
+bool ofxOpenALSoundPlayer::load(const of::filesystem::path& filePath, bool stream) {
 
 	if(!SoundEngineInitialized) {
 		ofxOpenALSoundPlayer::initializeSoundEngine();

addons/ofxiOS/src/sound/ofxOpenALSoundPlayer.h

@@ -62,7 +62,7 @@ class ofxOpenALSoundPlayer : public ofBaseSoundPlayer{
 	ofxOpenALSoundPlayer();
 	~ofxOpenALSoundPlayer();
 
-	bool	load(const std::filesystem::path& fileName, bool stream=false);
+	bool	load(const of::filesystem::path& fileName, bool stream=false);
 	void	unload();
 
 	void	play();

addons/ofxiOS/src/sound/ofxiOSSoundPlayer.h

@@ -15,7 +15,7 @@ class ofxiOSSoundPlayer : public ofBaseSoundPlayer {
     ofxiOSSoundPlayer();
     ~ofxiOSSoundPlayer();
 
-    bool load(const std::filesystem::path& fileName, bool stream = false);
+    bool load(const of::filesystem::path& fileName, bool stream = false);
     void unload();
     void play();
     void stop();

addons/ofxiOS/src/sound/ofxiOSSoundPlayer.mm

@@ -19,7 +19,7 @@
     unload();
 }
 
-bool ofxiOSSoundPlayer::load(const std::filesystem::path& fileName, bool stream) {
+bool ofxiOSSoundPlayer::load(const of::filesystem::path& fileName, bool stream) {
     if(soundPlayer != NULL) {
         unload();
     }

libs/openFrameworks/3d/ofMesh.h

@@ -604,7 +604,7 @@ class ofMesh_{
 	///
 	/// It expects that the file will be in the [PLY Format](http://en.wikipedia.org/wiki/PLY_(file_format)).
 	/// It will only load meshes saved in the PLY ASCII format; the binary format is not supported.
-    void load(const std::filesystem::path& path);
+    void load(const of::filesystem::path& path);
 
 	///  \brief Saves the mesh at the passed path in the [PLY Format](http://en.wikipedia.org/wiki/PLY_(file_format)).
 	///
@@ -615,7 +615,7 @@ class ofMesh_{
 	///  If you're planning on reloading the mesh into ofMesh, ofMesh currently only supports loading the ASCII format.
 	///
 	///  For more information, see the [PLY format specification](http://paulbourke.net/dataformats/ply/).
-    void save(const std::filesystem::path& path, bool useBinary = false) const;
+    void save(const of::filesystem::path& path, bool useBinary = false) const;
 
 	/// \}
 

libs/openFrameworks/3d/ofMesh.inl

@@ -1086,11 +1086,10 @@ void ofMesh_<V,N,C,T>::append(const ofMesh_<V,N,C,T> & mesh){
 
 //--------------------------------------------------------------
 template<class V, class N, class C, class T>
-void ofMesh_<V,N,C,T>::load(const std::filesystem::path& path){
-	ofFile is(path, ofFile::ReadOnly);
+void ofMesh_<V,N,C,T>::load(const of::filesystem::path& path){
+	ofFile is = {path, ofFile::ReadOnly};
 	auto & data = *this;
 
-
 	std::string error;
 	ofBuffer buffer(is);
 	auto backup = data;
@@ -1327,7 +1326,7 @@ void ofMesh_<V,N,C,T>::load(const std::filesystem::path& path){
 
 //--------------------------------------------------------------
 template<class V, class N, class C, class T>
-void ofMesh_<V,N,C,T>::save(const std::filesystem::path& path, bool useBinary) const{
+void ofMesh_<V,N,C,T>::save(const of::filesystem::path& path, bool useBinary) const{
 	ofFile os(path, ofFile::WriteOnly);
 	const auto & data = *this;