Merge pull request #390 from n7275/main

*Temporarily* Revert "Use std::filesystem instead of windows API in Orbiter core"

Sound/XRSound/src/Utils/FileList.cpp

@@ -53,47 +53,79 @@ void FileList::Scan(const char *pPath, const int recursionLevel)
 
     // This code was broken out from XRPayloadClassData::InitializeXRPayloadClassData().
 
-    for (auto& file : fs::directory_iterator(pPath)) {
-        if (file.path().stem().c_str()[0] != '.') {
-            if (clbkFilterNode(file))
+    WIN32_FIND_DATA findFileData;
+    char pConfigFilespecWildcard[MAX_PATH];
+    sprintf_s(pConfigFilespecWildcard, "%s\\*", pPath);  // iterate through all files and directories
+
+    HANDLE hFind = ::FindFirstFile(pConfigFilespecWildcard, &findFileData);
+    if (hFind == INVALID_HANDLE_VALUE)
+        return;    // directory is empty (should really never happen because "." and ".." are always present)
+
+    // found at least one file
+    goto process_file;   // a little goofy, but that's Windows' goofy FindFirstFile/FindNextFile for you...
+
+    // now loop through all remaining files
+    for (;;)
+    {
+        if (::FindNextFile(hFind, &findFileData) != 0)
+        {
+        process_file:   // entry point from FirstFirstFile
+                        // build the new configdir-relative path
+            char pNodeFilespec[MAX_PATH];
+            sprintf_s(pNodeFilespec, "%s\\%s", pPath, findFileData.cFileName);  // e.g., "Vessels\XRParts.cfg"
+
+            if (!STARTSWITH_DOT(findFileData))   // file not "." or ".."?
             {
-                // node should be included
-                if (file.is_directory())
-                {
-                    // this is a directory, so recurse down into it
-                    Scan(file.path().string().c_str(), recursionLevel + 1);
-                }
-                else if (file.file_size() > 0)  // it's a file node; is it not empty?
+                if (clbkFilterNode(pNodeFilespec, findFileData))
                 {
-                    // it's a file and it's not empty, so add it to our master list of nodes and invoke the callback for subclasses to hook
-                    m_allFiles.push_back(file.path().string().c_str());
-                    clbkProcessFile(file);
+                    // node should be included
+                    if (IS_DIRECTORY(findFileData))
+                    {
+                        // this is a directory, so recurse down into it
+                        Scan(pNodeFilespec, recursionLevel + 1);
+                    }
+                    else if (!IS_EMPTY(findFileData))  // it's a file node; is it not empty?
+                    {
+                        // it's a file and it's not empty, so add it to our master list of nodes and invoke the callback for subclasses to hook
+                        m_allFiles.push_back(pNodeFilespec);
+                        clbkProcessFile(pNodeFilespec, findFileData);
+                    }
                 }
             }
+        }  // if (::FindNextFile(hFind, &findFileData) != 0)
+        else   // FindNextFile failed
+        {
+            // check the error; continue parsing next file unless "no more files" reached
+            DWORD dwError = GetLastError();
+            if (dwError == ERROR_NO_MORE_FILES)
+                break;      // all done
+                            // else skip this file: fall through and parse the next file
         }
-    }
+    }  // for (;;)
+    FindClose(hFind);
 }
 
 // Invoked for each file or folder node found.  The default method here looks at bRecurseSubfolders (for folder nodes) and
 // pFileTypesToAccept (for file nodes) to decide whether accept a node or not.
 // Subclasses should override this method if they want more advanced filtering.
 //
 // Returns true if file node should be included or folder should be recursed into, or false if the node should be skipped.
-bool FileList::clbkFilterNode(const fs::directory_entry& entry)
+bool FileList::clbkFilterNode(const char *pPathOfNode, const WIN32_FIND_DATA &fd)
 {
-    if (entry.is_directory())
-        return m_bRecurseSubfolders;
+    if (IS_DIRECTORY(fd))
+        return m_bRecurseSubfolders; 
 
     // it's a file node
     bool bAcceptFile = false;
     if (!m_fileTypesToAccept.empty())
     {
-        if (entry.path().extension().string().length() > 0)     // e.g., ".flac"
+        const char *pFileExtension = strrchr(fd.cFileName, '.');
+        if (pFileExtension)     // e.g., ".flac"
         {
             // see if we have a case-insensitive match for this extension in our master list
-            for (auto it = m_fileTypesToAccept.begin(); it != m_fileTypesToAccept.end(); it++)
+            for (vector<CString>::const_iterator it = m_fileTypesToAccept.begin(); it != m_fileTypesToAccept.end(); it++)
             {
-                if (stricmp(entry.path().extension().string().c_str(), *it) == 0)
+                if (_stricmp(pFileExtension, *it) == 0)
                 {
                     bAcceptFile = true;
                     break;
@@ -108,7 +140,7 @@ bool FileList::clbkFilterNode(const fs::directory_entry& entry)
 }
 
 // Callback invoked for non-empty file nodes that passed the clbkFilterNode check; this is here for subclasses to hook.
-void FileList::clbkProcessFile(const fs::directory_entry& entry)
+void FileList::clbkProcessFile(const char *pFilespec, const WIN32_FIND_DATA &fd)
 {
     // no-op; this method is for subclasses to use
 }

Sound/XRSound/src/Utils/FileList.h

@@ -13,9 +13,6 @@
 
 using namespace std;
 
-#include <filesystem>
-namespace fs = std::filesystem;
-
 class FileList
 {
 public:
@@ -26,10 +23,9 @@ class FileList
 
     static bool DirectoryExists(const char *pPath)
     {
-        std::error_code ec;
-        auto status = fs::status(pPath, ec);
-     	if(ec) return false;
-    	return fs::is_directory(status);
+        DWORD dwAttrib = GetFileAttributes(pPath);
+
+        return (dwAttrib != INVALID_FILE_ATTRIBUTES && (dwAttrib & FILE_ATTRIBUTE_DIRECTORY));
     }
 
     // Scan (or rescan) file tree.
@@ -45,10 +41,10 @@ class FileList
 
     // Invoked for each file or folder node found; should return true if file node should be included or folder should be
     // recursed into, or false if the node should be skipped.
-    virtual bool clbkFilterNode(const fs::directory_entry &);
+    virtual bool clbkFilterNode(const char *pPathOfNode, const WIN32_FIND_DATA &fd);
 
     // Callback invoked for non-empty file nodes that passed the clbkFilterNode check; this is here for subclasses to hook.
-    virtual void clbkProcessFile(const fs::directory_entry &);
+    virtual void clbkProcessFile(const char *pFilespec, const WIN32_FIND_DATA &fd);
 
     int GetScannedFileCount() const { return static_cast<int>(m_allFiles.size()); }
     bool IsEmpty() const { return m_allFiles.empty(); }

Src/Orbiter/FlightRecorder.cpp

@@ -16,9 +16,11 @@
 #include "State.h"
 #include "MenuInfoBar.h"
 #include <fstream>
+#include <iomanip>
+#include <io.h>
+#include <direct.h>
+#include <errno.h>
 #include <string>
-#include <filesystem>
-namespace fs = std::filesystem;
 
 using namespace std;
 
@@ -774,17 +776,24 @@ void Orbiter::FRecorder_Reset ()
 
 bool Orbiter::FRecorder_PrepareDir (const char *fname, bool force)
 {
-	fs::path dir = fs::path("Flights") / fname;
-	std::error_code ec;
-	auto status = fs::status(dir, ec);
-
-	// don't overwrite existing recording
-	if (!ec) {
-		if(fs::is_directory(status) && !force) return false;
-		fs::remove_all(dir);
+	char cbuf[256];
+	strcpy (cbuf, "Flights\\"); strcat (cbuf, fname);
+	if (_mkdir (cbuf) == -1) {
+		if (errno == EEXIST && !force) return false;
+		// don't overwrite existing recording
+		struct _finddata_t fd;
+		char cb2[256], cb3[256];
+		strcpy (cb2, cbuf); strcat (cb2, "\\*");
+		intptr_t handle = _findfirst (cb2, &fd), res = handle;
+		while (res != -1) {
+			if (!(fd.attrib & _A_SUBDIR)) {
+				sprintf (cb3, "%s\\%s", cbuf, fd.name);
+				_unlink (cb3);
+			}
+			res = _findnext (handle, &fd);
+		}
+		_findclose (handle);
 	}
-	fs::create_directory(dir);
-
 	return true;
 }
 

Src/Orbiter/GraphicsAPI.cpp

@@ -17,8 +17,7 @@
 #include "Util.h"
 #include "resource.h"
 #include <wincodec.h>
-#include <filesystem>
-namespace fs = std::filesystem;
+#include <io.h>
 
 using std::min;
 
@@ -175,17 +174,24 @@ ScreenAnnotation *GraphicsClient::clbkCreateAnnotation ()
 
 bool GraphicsClient::TexturePath (const char *fname, char *path) const
 {
+	struct _finddata_t fd;
+	intptr_t fh;
+
 	// first try htex directory
 	strcpy (path, g_pOrbiter->Cfg()->CfgDirPrm.HightexDir);
 	strcat (path, fname);
-	if (fs::exists(path)) return true;
+	if ((fh = _findfirst (path, &fd)) != -1) {
+		_findclose (fh);
+		return true;
+	}
 
 	// try tex directory
 	strcpy (path, g_pOrbiter->Cfg()->CfgDirPrm.TextureDir);
 	strcat (path, fname);
-
-	if (fs::exists(path)) return true;
-
+	if ((fh = _findfirst (path, &fd)) != -1) {
+		_findclose (fh);
+		return true;
+	}
 	return false;
 }
 

Src/Orbiter/OptionsPages.cpp

@@ -6,6 +6,7 @@
 // ======================================================================
 
 #include <windows.h>
+#include <io.h>
 #include <array>
 #include "OptionsPages.h"
 #include "DlgCtrl.h"
@@ -1705,13 +1706,19 @@ void OptionsPage_Planetarium::RescanMarkerList(HWND hPage)
 	const std::vector< oapi::GraphicsClient::LABELLIST>& list = g_psys->LabelList();
 	if (!list.size()) return;
 
-	int n = 0;
-	g_psys->ForEach(FILETYPE_MARKER, [&](const fs::directory_entry& entry) {
-		SendDlgItemMessage(hPage, IDC_OPT_PLN_MKRLIST, LB_ADDSTRING, 0, (LPARAM)entry.path().stem().string().c_str());
-		if (n < list.size() && list[n].active)
-			SendDlgItemMessage(hPage, IDC_OPT_PLN_MKRLIST, LB_SETSEL, TRUE, n);
-		n++;
-	});
+	char cbuf[256];
+	_finddata_t fdata;
+	intptr_t fh = g_psys->FindFirst(FILETYPE_MARKER, &fdata, cbuf);
+	if (fh >= 0) {
+		int n = 0;
+		do {
+			SendDlgItemMessage(hPage, IDC_OPT_PLN_MKRLIST, LB_ADDSTRING, 0, (LPARAM)trim_string(cbuf));
+			if (n < list.size() && list[n].active)
+				SendDlgItemMessage(hPage, IDC_OPT_PLN_MKRLIST, LB_SETSEL, TRUE, n);
+			n++;
+		} while (!g_psys->FindNext(fh, &fdata, cbuf));
+		_findclose(fh);
+	}
 }
 
 // ======================================================================
@@ -1878,14 +1885,18 @@ void OptionsPage_Labels::UpdateFeatureList(HWND hPage)
 	if (planet->LabelFormat() < 2) {
 		oapi::GraphicsClient::LABELLIST* list = planet->LabelList(&nlist);
 		if (!nlist) return;
-
-		n = 0;
-		planet->ForEach(FILETYPE_MARKER, [&](const fs::directory_entry& entry) {
-				SendDlgItemMessage(hPage, IDC_OPT_MKR_FEATURELIST, LB_ADDSTRING, 0, (LPARAM)entry.path().stem().string().c_str());
+		_finddata_t fdata;
+		long fh = planet->FindFirst(FILETYPE_MARKER, &fdata, cpath, cbuf);
+		if (fh >= 0) {
+			n = 0;
+			do {
+				SendDlgItemMessage(hPage, IDC_OPT_MKR_FEATURELIST, LB_ADDSTRING, 0, (LPARAM)trim_string(cbuf));
 				if (n < nlist && list[n].active)
 					SendDlgItemMessage(hPage, IDC_OPT_MKR_FEATURELIST, LB_SETSEL, TRUE, n);
 				n++;
-			});
+			} while (!planet->FindNext(fh, &fdata, cbuf));
+			_findclose(fh);
+		}
 	}
 	else {
 		int nlabel = planet->NumLabelLegend();

Src/Orbiter/Orbiter.cpp

@@ -8,11 +8,17 @@
 #pragma comment(linker,"\"/manifestdependency:type='win32' name='Microsoft.Windows.Common-Controls' version='6.0.0.0' processorArchitecture='*' publicKeyToken='6595b64144ccf1df' language='*'\"")
 
 #include <windows.h>
+#include <zmouse.h>
+#include <commctrl.h>
+#include <mmsystem.h>
+#include <process.h>
 #include <direct.h>
 #include <stdio.h>
 #include <time.h>
 #include <fstream>
 #include <strstream>
+#include <iomanip>
+#include <io.h>
 #include "cmdline.h"
 #include "D3d7util.h"
 #include "D3dmath.h"
@@ -43,8 +49,6 @@
 #include "DlgCtrl.h"
 #include "GraphicsAPI.h"
 #include "ConsoleManager.h"
-#include <filesystem>
-namespace fs = std::filesystem;
 
 #ifdef INLINEGRAPHICS
 #include "OGraphics.h"
@@ -541,13 +545,18 @@ int Orbiter::GetVersion () const
 	return v;
 }
 
+static bool FileExists(const char* path)
+{
+	return access(path, 0) != -1;
+}
+
 //! Finds legacy module consisting of a single DLL
 //! @return true on success
 //! @param cbufOut returns path to the plugin DLL
 static bool FindStandaloneDll(const char *path, const char *name, char* cbufOut)
 {
 	sprintf (cbufOut, "%s\\%s.dll", path, name);
-	return fs::exists(cbufOut);
+	return FileExists(cbufOut);
 }
 
 //! Finds module consisting of a plugin DLL inside a plugin-specific folder
@@ -556,7 +565,7 @@ static bool FindStandaloneDll(const char *path, const char *name, char* cbufOut)
 static bool FindDllInPluginFolder(const char *path, const char *name, char* cbufOut)
 {
 	sprintf(cbufOut, "%s\\%s\\%s.dll", path, name, name);
-	return fs::exists(cbufOut);
+	return FileExists(cbufOut);
 }
 
 void Orbiter::LoadModules(const std::string& path, const std::list<std::string>& names)
@@ -565,14 +574,19 @@ void Orbiter::LoadModules(const std::string& path, const std::list<std::string>&
 		LoadModule(path.c_str(), name.c_str());
 }
 
+
 void Orbiter::LoadModules(const std::string& path)
 {
-	for (const auto& entry : fs::directory_iterator(path)) {
-		auto fpath = entry.path();
-		if (fpath.extension().string() == ".dll") {
-			LoadModule(path.c_str(), fpath.stem().string().c_str());
+	struct _finddata_t fdata;
+	intptr_t fh = _findfirst((path + std::string("\\*.dll")).c_str(), &fdata);
+	if (fh == -1) return; // no files found
+	do {
+		if (strlen(fdata.name) > 4) {
+			fdata.name[strlen(fdata.name) - 4] = '\0'; // cut off extension
+			LoadModule(path.c_str(), fdata.name);
 		}
-	}
+	} while (!_findnext(fh, &fdata));
+	_findclose(fh);
 }
 
 //-----------------------------------------------------------------------------