Hi all,<br><br>With the merging of the Broken Sword 2.5 engine into the trunk, yay :), I'd thought I'd raise a couple of points regarding the engine:<br><br>1. For those of you, like me, using Visual Studio to compile ScummVM, you currently need to go to the Project Options for the 'sword25' project, and under the C++ section, change the 'Disable Language Extensions (/za)' option to 'No'. This is needed to properly compile the Lua code module. I'm aware that there are some warnings currently in the module, such as the use of 'tmpnam' and various pointer conversions. Up until now we left them alone because we didn't want to start modifying a third party library. We could always discuss it further, if anyone has any opinion on the matter.<br>
<br>Maybe now that it's hit the trunk, someone could patch the create_msvc tool to automatically not set that flag for the sword25 project? I've mucked around with the tool myself, but I'm not really happy with my hacks, so it can probably be done cleaner by someone else, such that only sword25 doesn't have the setting.<br>
<br>2. Refer to the TODO list at:  <a href="http://wiki.scummvm.org/index.php/Sword25/TODO">http://wiki.scummvm.org/index.php/Sword25/TODO</a><br><br>Particularly what I consider to be the biggest remaining bug is that there's a memory corruption during savegame loading that I hadn't had a chance to track down yet. Maybe someone familiar with Valgrind can take a look into it. <br>
<br>3. There's still a lot of German comments in the code, so maybe someone interested in delving into the engine can spend some time with Google Translate to convert them. :). I don't think there's any easy automated way to convert all the comments, so it's probably going to have to be a gradual manual process.<br>
<br>Regards,<br><br>Paul.<br><div style="visibility: hidden; display: inline;" id="avg_ls_inline_popup"></div><style type="text/css">#avg_ls_inline_popup {  position:absolute;  z-index:9999;  padding: 0px 0px;  margin-left: 0px;  margin-top: 0px;  width: 240px;  overflow: hidden;  word-wrap: break-word;  color: black;  font-size: 10px;  text-align: left;  line-height: 13px;}</style>