<html>
<head>
<style><!--
.hmmessage P
{
margin:0px;
padding:0px
}
body.hmmessage
{
font-size: 10pt;
font-family:Verdana
}
--></style>
</head>
<body class='hmmessage'>
> For SCI, only the fallback detection code uses the SCI<br>> ResourceManager. I've always been iffy with this in the first place:<br>> We have a chance of erroring out on detection. I don't think we really<br>> need all of this fallback code. The filename fallback (as done with<br>> several other engines) should be good enough for most cases. The only<br>> time it would fail would be for other Sierra games with similar file<br>> names.<br><br><div id="">The reason why the fallback detection code was written is because of the</div><div id="">multitude of SCI games and versions out there. We could revert to an</div><div id="">"Unknown SCI game" or something along these lines, but I fail to see the </div><div id="">reasoning behind this, since Sierra itself embedded IDs in all of its games, </div><div id="">thus we can find the real name of the game from its data, together with other </div><div id="">information, such as the language it uses. If there is a good reason why this </div><div id="">detection scheme should be removed, please explain it, and I'll be quite happy </div><div id="aeaoofnhgocdbnbeljkmbjdmhbcokfdb-mousedown">to remove it, if you feel it's not needed. With this reasoning though, we could</div><div id="aeaoofnhgocdbnbeljkmbjdmhbcokfdb-mousedown">remove and simplify the detection code of the SCUMM engine, too (which</div><div id="aeaoofnhgocdbnbeljkmbjdmhbcokfdb-mousedown">uses a more advanced detection scheme too), but I find this idea to be a</div><div id="aeaoofnhgocdbnbeljkmbjdmhbcokfdb-mousedown">step backwards myself. If you do feel that the detection code is iffy, we</div><div id="aeaoofnhgocdbnbeljkmbjdmhbcokfdb-mousedown">should have a policy not to have complex detection code in all of our engines,</div><div id="aeaoofnhgocdbnbeljkmbjdmhbcokfdb-mousedown">and simplify this task (this includes SCUMM, SCI and AGI, afaik). If not, a</div><div id="aeaoofnhgocdbnbeljkmbjdmhbcokfdb-mousedown">different plan for detection needs to be devised, one that doesn't require</div><div id="aeaoofnhgocdbnbeljkmbjdmhbcokfdb-mousedown">splitting the detection code separately from the engine.</div><div id="aeaoofnhgocdbnbeljkmbjdmhbcokfdb-mousedown"><br></div><div id="aeaoofnhgocdbnbeljkmbjdmhbcokfdb-mousedown">Regards</div><div id="aeaoofnhgocdbnbeljkmbjdmhbcokfdb-mousedown">Filippos</div> <br /><hr />The New Busy is not the old busy. Search, chat and e-mail from your inbox. <a href='http://www.windowslive.com/campaign/thenewbusy?ocid=PID28326::T:WLMTAGL:ON:WL:en-US:WM_HMP:042010_3' target='_new'>Get started.</a></body>
</html>