[Scummvm-devel] Log file support

[Max Horn]

Am 25.11.2010 um 01:25 schrieb Michael Madsen:

>> -----Original Message-----
>> From: Johannes Schickel [mailto:lordhoto at scummvm.org]
>> Sent: Wednesday, November 24, 2010 11:50 PM
>> To: Max Horn
>> Cc: ScummVM devel
>> Subject: Re: [Scummvm-devel] Log file support
>> 
>> Agreed I guess using %APPDATA%\ScummVM\logs does fine. I am not sure
>> whether we still support Win9x systems, but in the past there had been
>> discussions that %APPDATA% might not work on every Windows version out
>> there, thus it might be required to have some fallback location. Maybe
>> the CWD would be a "sensible" fallback in that case?
> 
> According to MSDN, there has been an AppData folder since the release of IE4, but before Windows 98, it was possible to install IE4 without the required DLLs. In other words, only NT4 and Win95 can be a problem, and even there, it's not all systems that are affected.

I think we don't really need to worry about those versions, but nevertheless, getDefaultConfigFileName() in sdl.cpp already "solves" this problem; we can simply factor the windows specific code in there which locates the app data dir (and creates a ScummVM subdir of it, if necessary), into a separate function which both the config file and the log file code could use.
Oh, and all this will become nicely separated once / if we merge the gsoc2010-opengl branch, did I mention that? ;)

Cheers,
Max