[Scummvm-cvs-logs] SF.net SVN: scummvm: [21582] scummvm/trunk/backends/fs/posix/posix-fs.cpp

[fingolfin at users.sourceforge.net]

Revision: 21582
Author:   fingolfin
Date:     2006-04-03 15:06:42 -0700 (Mon, 03 Apr 2006)
ViewCVS:  http://svn.sourceforge.net/scummvm/?rev=21582&view=rev

Log Message:
-----------
Added comment on why we use getcwd in POSIXFilesystemNode at all

Modified Paths:
--------------
    scummvm/trunk/backends/fs/posix/posix-fs.cpp
Modified: scummvm/trunk/backends/fs/posix/posix-fs.cpp
===================================================================
--- scummvm/trunk/backends/fs/posix/posix-fs.cpp	2006-04-03 21:54:26 UTC (rev 21581)
+++ scummvm/trunk/backends/fs/posix/posix-fs.cpp	2006-04-03 22:06:42 UTC (rev 21582)
@@ -82,7 +82,13 @@
 
 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);
 


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.