I get it now, thanks for making it clear :)<br><br><div><span class="gmail_quote">On 10/31/07, <b class="gmail_sendername">Marcus Comstedt</b> <<a href="mailto:marcus@mc.pp.se">marcus@mc.pp.se</a>> wrote:</span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br>"David Corrales" <<a href="mailto:corrales.david@gmail.com">corrales.david@gmail.com</a>> writes:<br><br>> 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>
<br><br>Sorry, I was a bit unclear. The stubbing currently consists of the<br>following lines in file.cpp:<br><br>#ifdef __DC__<br> /* Can't remove files from CD-ROM... */<br> #define remove(name) ((errno = EROFS), -1)
<br>#endif<br><br>They are needed as long as a call to remove() exists in file.cpp,<br>because otherwise I get a link error because no real remove() function<br>exists (since it can't work anyway, it didn't make any sense to write
<br>one). With this #define, removeFile() becomes a stub, scummvm links,<br>but the removeFile() function will of course not work (since it can't<br>work on the DC). Hence the question if something actually relied on
<br>this function to work.<br><br>If the call to remove() is moved to the default savefile manager,<br>I don't need the stub anymore since default-saves.cpp is not included<br>in my build.<br><br><br> // Marcus<br><br>
<br></blockquote></div><br>