[Scummvm-devel] SCI file handling revamp

[Max Horn]

Hi folks,

I had a first go at revamping the SCI file handling code to use our  
portable file APIs, instead of the existing code which relied on stuff  
like chdir, getcwd, being able to fwrite to arbitrary files etc.

It is not yet feature complete, mainly because I don't know how to  
test all features, and in some cases it was not quite clear how to  
implement those. E.g. it is likely that character im-/export in QfG is  
broken with this patch. for starters, it is broken as the target name  
is prepended to all files im-/exported; for the QfG series the string  
should likely always be "qfg", but that's easy enough to change. Also,  
maybe we should prepend the gameid and not the targetname.

Furthemore, file "mirroring" is not implemented, nor is opening files  
for both reading and writing. While I could have worked on that (and  
there is a lengthy comment in the patch which explains on how to do  
this), I would prefer to first find an example where this is actually  
used. Mainly because (a) I didn't want to waste time on stuff that  
might not be needed, and (b) w/o any means to test this, it's no fun  
to implement this :-).


Everybody is welcome to look at this patch, comment on it (via the  
tracker, ideally), or take it and complete it :-).


Bye,
Max