Some more fixes

src/anki/core/App.cpp

@@ -496,7 +496,7 @@ Error App::initDirs(const ConfigSet& cfg)
 #if !ANKI_OS_ANDROID
 	// Settings path
 	StringAuto home(m_heapAlloc);
-	ANKI_CHECK(getHomeDirectory(m_heapAlloc, home));
+	ANKI_CHECK(getHomeDirectory(home));
 
 	m_settingsDir.sprintf(m_heapAlloc, "%s/.anki", &home[0]);
 

src/anki/physics/Common.h

@@ -16,6 +16,10 @@
 #	pragma GCC diagnostic ignored "-Wconversion"
 #	pragma GCC diagnostic ignored "-Wfloat-conversion"
 #endif
+#if ANKI_COMPILER_MSVC
+#	pragma warning(push)
+#	pragma warning(disable : 4305)
+#endif
 #define BT_THREADSAFE 0
 #define BT_NO_PROFILE 1
 #include <btBulletCollisionCommon.h>
@@ -26,6 +30,9 @@
 #if ANKI_COMPILER_GCC_COMPATIBLE
 #	pragma GCC diagnostic pop
 #endif
+#if ANKI_COMPILER_MSVC
+#	pragma warning(pop)
+#endif
 
 namespace anki
 {

src/anki/util/Filesystem.h

@@ -45,7 +45,7 @@ ANKI_USE_RESULT Error createDirectory(const CString& dir);
 /// Get the home directory.
 /// Write the home directory to @a buff. The @a buffSize is the size of the @a buff. If the @buffSize is not enough the
 /// function will throw an exception.
-ANKI_USE_RESULT Error getHomeDirectory(GenericMemoryPoolAllocator<U8> alloc, String& out);
+ANKI_USE_RESULT Error getHomeDirectory(StringAuto& out);
 /// @}
 
 } // end namespace anki

src/anki/util/FilesystemPosix.cpp

@@ -195,7 +195,7 @@ Error createDirectory(const CString& dir)
 	return err;
 }
 
-Error getHomeDirectory(GenericMemoryPoolAllocator<U8> alloc, String& out)
+Error getHomeDirectory(StringAuto& out)
 {
 	const char* home = getenv("HOME");
 	if(home == nullptr)
@@ -204,7 +204,7 @@ Error getHomeDirectory(GenericMemoryPoolAllocator<U8> alloc, String& out)
 		return Error::FUNCTION_FAILED;
 	}
 
-	out.create(alloc, home);
+	out.create(home);
 	return Error::NONE;
 }
 

src/anki/util/FilesystemWindows.cpp

@@ -7,6 +7,7 @@
 #include <anki/util/Assert.h>
 #include <anki/util/Logger.h>
 #include <windows.h>
+#include <Shlobj.h>
 #include <anki/util/CleanupWindows.h>
 
 namespace anki
@@ -70,28 +71,16 @@ Error createDirectory(const CString& dir)
 	return err;
 }
 
-Error getHomeDirectory(GenericMemoryPoolAllocator<U8> alloc, String& out)
+Error getHomeDirectory(StringAuto& out)
 {
-	const char* homed = getenv("HOMEDRIVE");
-	const char* homep = getenv("HOMEPATH");
-
-	if(homed == nullptr || homep == nullptr)
+	char path[MAX_PATH];
+	if(SHGetFolderPath(NULL, CSIDL_PROFILE, nullptr, 0, path) != S_OK)
 	{
-		ANKI_UTIL_LOGE("HOME environment variables not set");
+		ANKI_UTIL_LOGE("SHGetFolderPath() failed");
 		return Error::FUNCTION_FAILED;
 	}
 
-	out.sprintf(alloc, "%s/%s", homed, homep);
-
-	// Convert to Unix path
-	for(char& c : out)
-	{
-		if(c == '\\')
-		{
-			c = '/';
-		}
-	}
-
+	out.create(path);
 	return Error::NONE;
 }
 

tests/util/Filesystem.cpp

@@ -58,7 +58,7 @@ ANKI_TEST(Util, HomeDir)
 	HeapAllocator<char> alloc(allocAligned, nullptr);
 	StringAuto out(alloc);
 
-	ANKI_TEST_EXPECT_NO_ERR(getHomeDirectory(alloc, out));
+	ANKI_TEST_EXPECT_NO_ERR(getHomeDirectory(out));
 	printf("home dir %s\n", &out[0]);
 	ANKI_TEST_EXPECT_GT(out.getLength(), 0);
 }