From ebeded36a198fea5d86721be859e4540f64624d3 Mon Sep 17 00:00:00 2001
From: Max Horn
Date: Mon, 3 Apr 2006 22:06:42 +0000
Subject: Added comment on why we use getcwd in POSIXFilesystemNode at all

svn-id: r21582
---
 backends/fs/posix/posix-fs.cpp | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

(limited to 'backends')

diff --git a/backends/fs/posix/posix-fs.cpp b/backends/fs/posix/posix-fs.cpp
index 925d06486b..f32f30d8ba 100644
--- a/backends/fs/posix/posix-fs.cpp
+++ b/backends/fs/posix/posix-fs.cpp
@@ -82,7 +82,13 @@ AbstractFilesystemNode *FilesystemNode::getNodeForPath(const String &path) {
 
 POSIXFilesystemNode::POSIXFilesystemNode() {
 #ifndef __DC__
-	// FIXME: It is evil & slow to always call getcwd here
+	// FIXME: It is evil & slow to always call getcwd here.
+	// The intention behind this hack was/is to be more user friendly
+	// in our save/load dialogs: Instead of starting at the FS root,
+	// we start at the current directory. However, that's just a hack.
+	// Proper solution would be to extend FilesystemNode by the concept
+	// of 'current' or 'default' directory, and then modify the
+	// save/load dialogs to explicitly use that as starting point.
 	char buf[MAXPATHLEN];
 	getcwd(buf, MAXPATHLEN);
 
-- 
cgit v1.2.3