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?<br><br><div>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><br>On So, Oktober 28, 2007 22:32, Marcus Comstedt wrote:<br>><br>> "David Corrales" <
<a href="mailto:corrales.david@gmail.com">corrales.david@gmail.com</a>> writes:<br>><br>>> The removeSaveFile() calls the common::removeFile() method to perform<br>>> the file<br>>> deletion :)<br>
><br>> removeSavefile() is pure virtual, so it depends entirely on the<br>> SaveFileManager implementation what it does.  The SaveFileManager on<br>> the Dreamcast does not call common::removeFile().  Are you saying that
<br>> this environmental dependency exists in Common just for the benefit of<br>> backends/saves/default/default-saves.cpp?  That sounds backwards;<br>> better have such dependencies in backends/saves/default in that case,
<br>> so that only platforms which uses it needs to deal with that<br>> dependency.<br><br>I agree. David, please have a look at that.<br><br>><br>> But I take it then that it should be safe to simply stub the
<br>> removeFile function on systems which don't use the default save<br>> manager, if this is indeed the only indented use...<br>><br><br>Aye. Until we properly fix it, at least.<br><br><br>Max<br></blockquote>
</div><br>