[Scummvm-devel] fsnode-gsoc remove file method

David Corrales corrales.david at gmail.com
Wed Oct 31 06:30:39 CET 2007


I didn't quite get the stubbing part. I take that now, the code that is in
removeFile is to be moved to the default SaveFileManager, but I don't see
the stubbing part. Care to explain a bit further please?


> On So, Oktober 28, 2007 22:32, Marcus Comstedt wrote:
> >
> > "David Corrales" <corrales.david at gmail.com> writes:
> >
> >> The removeSaveFile() calls the common::removeFile() method to perform
> >> the file
> >> deletion :)
> >
> > removeSavefile() is pure virtual, so it depends entirely on the
> > SaveFileManager implementation what it does.  The SaveFileManager on
> > the Dreamcast does not call common::removeFile().  Are you saying that
> > this environmental dependency exists in Common just for the benefit of
> > backends/saves/default/default-saves.cpp?  That sounds backwards;
> > better have such dependencies in backends/saves/default in that case,
> > so that only platforms which uses it needs to deal with that
> > dependency.
>
> I agree. David, please have a look at that.
>
> >
> > But I take it then that it should be safe to simply stub the
> > removeFile function on systems which don't use the default save
> > manager, if this is indeed the only indented use...
> >
>
> Aye. Until we properly fix it, at least.
>
>
> Max
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.scummvm.org/pipermail/scummvm-devel/attachments/20071030/1935787e/attachment.html>


More information about the Scummvm-devel mailing list