Dennis Duda
Dennis Duda
I also can't get it to compile in the "obvious" way, and it certainly works without the `*` in the test code. Another argument in favor of reverting this -...
Warning: Big writeup ahead! :) As @mewmew wrote up over [here](https://github.com/diasurgical/devilution/pull/135#issuecomment-404386600), more often than not, the compiler we currently use (5.0 SP3) generated slightly different instructions than the original 1.09b...
https://github.com/seritools/devilution/commit/6f5f573bbd520db8dcda36db8bb52525b9bcd7f7 (please excuse my crappy makefile skills) Managed to link the exe with 5.10 while compiling with 6 :) However in this constellation, PDBs cannot be generated, since the linker...
A few more observations: - SP5 is definitely ruled out: The release binary is way too big, and one jump in `CheckSpell` (`.text:0045759A jz short loc_4575A0`) is generated as a...
Alright, SP2, SP4, SP5 are excluded now, since all three of those generate binaries that are too big. For some reason, with SP3, the filesize went down again. I reinstalled...
PDB info is nice while comparing functions, since you can just jump to the function in question in the new binary, without remembering adresses, and getting all the type info...
And here is the exe :) [devilution.zip](https://github.com/diasurgical/devilution/files/2207307/devilution.zip)
Yeah, I only got the size way down when using the VC6 libs, the file was quite a bit bigger with the VC5 ones. (Tbh it was surpriting to me...
https://github.com/diasurgical/devil-nightly/issues/15 We're pretty much done 🎉 In the end, Blizzard made sure to keep their software up to date it seems 😄