[Scummvm-devel] PORTERS: New OSystem API for loading/storing default config file

[Schrijvers Luc]

Hi,

Tried applying the patch against the branch tree (with didn't work
ofcourse *bangs head*), but when apply'd to the trunk tree the build is
ok here, no errors while compiling/installing/running (lure).

Greetz,

Luc

On di, 2008-07-29 at 19:52 +0100, Max Horn wrote:
> Hi everybody,
> 
> based on the "File handling" thread, I think it would be a good idea
> to add two new OSystem methods for accessing the *default* confg file:
> 
>  virtual Common::SeekableReadStream *openConfigFileForReading();
>  virtual Common::WriteStream *openConfigFileForWriting();
> 
> (the latter method may return 0 to indicate that writing to the
> default config file is not possible).
> 
> The attached patch does just that. Initially, this change should not
> have any impact on anybody (besides the usual regressions caused by my
> coding mistakes ;). However, this should enable us to gradually get
> rid of the #ifdef mess and other evil tricks we currently use to load
> the default config file from the correct location. E.g. the PS2 & PSP
> hacks using fake paths could be removed.
> 
> I am interested in comments on this patch. Right now I am tempted to
> just commit it as-is, it just feels the "right" way to approach
> things. But I figure it's best to wait a bit to hear whether there is
> loud criticism or *gasp* even alternate proposals.
> 
> Cheers,
> Max
> -------------------------------------------------------------------------
> This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
> Build the coolest Linux based applications with Moblin SDK & win great prizes
> Grand prize is a trip for two to an Open Source event anywhere in the world
> http://moblin-contest.org/redirect.php?banner_id=100&url=/
> _______________________________________________ Scummvm-devel mailing list Scummvm-devel at lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/scummvm-devel