11th hour localized file issue & fix

[lolovo]

Hi,

I've faced with an issue described here - ​https://forums.scummvm.org/viewtopic.php?t=16661. As it was mentioned it is a specific of localized files. I've noticed that in roq.cpp:547 for seek of end position endpos is used and this leeds to crash, but maybe appropriate try will be using file->pos in this case (I'm not the spicialist in scummvm code, as well as groovie engine). Anyway when file->pos is used the issue is solved (no crash anymore). The fix is the following:

...
	if (endpos != _file->pos()) {
		warning("Groovie::ROQ: BLOCK %04x Should have ended at %d, and has ended at %d", blockHeader.type, endpos, (int)_file->pos());
		warning("Ensure you've copied the files correctly according to the wiki.");
		_file->seek(_file->pos());//was _file->seek(endpos);
	}
...

Could you please verify this and maybe integrate this in project. With this fix my issue is solved. Thanks!

[antoniou79]

You could issue a Pull Request with this fix. Although, if it's specific to this fan made translation maybe it would be better to restrict it so that it will only apply there -- this would need also to update the code in detection to recongnize this fan made localization.

I cannot say if it's the correct fix, but a few days ago on Discord I mentioned the issue to Die4Ever, who is the main developer working on the Groovie engine, so he might be available to provide a review for a potential PR.

[lolovo]

Thanks for an answer! Done!
P.S. Appropriate fix would be getting MIN(_file->pos(),endpos)

[Die4Ever]

merged the pull request, thank you! sorry I took so long, been busy