Author Topic: Shadow of the Wyrm v1.7.2 "Michelagnolo Galilei"(formerly Savage Lands)  (Read 289926 times)

jcd748

  • Rogueliker
  • ***
  • Posts: 231
  • Karma: +0/-0
    • View Profile
Re: Shadow of the Wyrm 0.4.1 (formerly Savage Lands)
« Reply #45 on: September 19, 2015, 02:06:07 PM »
I'll see what I can do.  I can certainly try to add more logging and hope it gets hit when catching the exception.

In the meantime, can you install procdump (https://technet.microsoft.com/en-us/sysinternals/dd996900.aspx), start sotw.exe, and then run "procdump -mp -e sotw.exe" in another window?  This should create a dump that if you send to me I can open in Visual Studio to try to see what's wrong. (if you put it on Dropbox or something similar, that'd be perfect)

Aleksanderus

  • Rogueliker
  • ***
  • Posts: 179
  • Karma: +0/-0
  • Champion of the unknown
    • View Profile
Re: Shadow of the Wyrm 0.4.1 (formerly Savage Lands)
« Reply #46 on: September 19, 2015, 02:20:52 PM »
I'll see what I can do.  I can certainly try to add more logging and hope it gets hit when catching the exception.

In the meantime, can you install procdump (https://technet.microsoft.com/en-us/sysinternals/dd996900.aspx), start sotw.exe, and then run "procdump -mp -e sotw.exe" in another window?  This should create a dump that if you send to me I can open in Visual Studio to try to see what's wrong. (if you put it on Dropbox or something similar, that'd be perfect)
Unfoartunately I don't know how to use I did everything what you wrote in the cmd and it says: Dump count not reached
« Last Edit: September 19, 2015, 02:27:23 PM by Aleksanderus »

jcd748

  • Rogueliker
  • ***
  • Posts: 231
  • Karma: +0/-0
    • View Profile
Re: Shadow of the Wyrm 0.4.1 (formerly Savage Lands)
« Reply #47 on: September 19, 2015, 02:29:35 PM »
So I first run the game:



Then I run procdump (open a cmd prompt, then "c:\procdump\procdump.exe -mp -e sotw.exe", or wherever you installed procdump):



The window stays open as long as sotw.exe is running.

Aleksanderus

  • Rogueliker
  • ***
  • Posts: 179
  • Karma: +0/-0
  • Champion of the unknown
    • View Profile
Re: Shadow of the Wyrm 0.4.1 (formerly Savage Lands)
« Reply #48 on: September 19, 2015, 02:34:15 PM »
So I first run the game:



Then I run procdump (open a cmd prompt, then "c:\procdump\procdump.exe -mp -e sotw.exe", or wherever you installed procdump):



The window stays open as long as sotw.exe is running.
Oh ok I've got a dump log It's pretty big for a text file! And i need to wait becouse uploading from computer to dropbox is pretty long
Nevermind here's dropbox link: https://www.dropbox.com/s/t5o9zue1v0t7vuy/sotw.exe_150919_163148.dmp?dl=0
« Last Edit: September 19, 2015, 02:36:03 PM by Aleksanderus »

jcd748

  • Rogueliker
  • ***
  • Posts: 231
  • Karma: +0/-0
    • View Profile
Re: Shadow of the Wyrm 0.4.1 (formerly Savage Lands)
« Reply #49 on: September 19, 2015, 02:46:44 PM »
I'm able to open it, but can't seem to load things properly.

Can you do me a favour?  I've created a release build of the latest 0.4.2, with PDB:

https://www.dropbox.com/s/60w42akqmpvbvvv/ShadowOfTheWyrm-0.4.2-current-PDB.zip?dl=0

Can you repeat the process and send me the file again?  Hopefully then we're both working off the exact same version and then Visual Studio will give me more details.

Aleksanderus

  • Rogueliker
  • ***
  • Posts: 179
  • Karma: +0/-0
  • Champion of the unknown
    • View Profile
Re: Shadow of the Wyrm 0.4.1 (formerly Savage Lands)
« Reply #50 on: September 19, 2015, 02:53:54 PM »
I'm able to open it, but can't seem to load things properly.

Can you do me a favour?  I've created a release build of the latest 0.4.2, with PDB:

https://www.dropbox.com/s/60w42akqmpvbvvv/ShadowOfTheWyrm-0.4.2-current-PDB.zip?dl=0

Can you repeat the process and send me the file again?  Hopefully then we're both working off the exact same version and then Visual Studio will give me more details.
Still the same and here's the file and also I'm quite enjoying our cooperation ;D https://www.dropbox.com/s/x6z442aovwz0lqn/sotw.exe_150919_165057.dmp?dl=0

jcd748

  • Rogueliker
  • ***
  • Posts: 231
  • Karma: +0/-0
    • View Profile
Re: Shadow of the Wyrm 0.4.1 (formerly Savage Lands)
« Reply #51 on: September 19, 2015, 03:17:17 PM »
Made a few small tweaks - here's a new version:

https://www.dropbox.com/s/60w42akqmpvbvvv/ShadowOfTheWyrm-0.4.2-current-PDB.zip?dl=0

This time can you run "procdump -ma -e sotw.exe"?  The "-ma" option should include a larger memory dump, which should hopefully give me additional details...

Basically, all I'm getting right now from the dump is that you're accessing invalid memory.  I'm trying to figure out why exactly that is, and where, so I can investigate and make a proper fix for the next version.  Sorry this is turning into such a hassle, but your help is hugely appreciated!

Aleksanderus

  • Rogueliker
  • ***
  • Posts: 179
  • Karma: +0/-0
  • Champion of the unknown
    • View Profile
Re: Shadow of the Wyrm 0.4.1 (formerly Savage Lands)
« Reply #52 on: September 19, 2015, 03:37:58 PM »
Made a few small tweaks - here's a new version:

https://www.dropbox.com/s/60w42akqmpvbvvv/ShadowOfTheWyrm-0.4.2-current-PDB.zip?dl=0

This time can you run "procdump -ma -e sotw.exe"?  The "-ma" option should include a larger memory dump, which should hopefully give me additional details...

Basically, all I'm getting right now from the dump is that you're accessing invalid memory.  I'm trying to figure out why exactly that is, and where, so I can investigate and make a proper fix for the next version.  Sorry this is turning into such a hassle, but your help is hugely appreciated!
Ok here's dump file again this time I turned admin privilages both in dump program and game and maybe I see a problem with dump file
and link to new dump file https://www.dropbox.com/s/vi91zk1p7r29vxc/sotw.exe_150919_173025.dmp?dl=0
And also this dump program created PDB file what this file does?

jcd748

  • Rogueliker
  • ***
  • Posts: 231
  • Karma: +0/-0
    • View Profile
Re: Shadow of the Wyrm 0.4.1 (formerly Savage Lands)
« Reply #53 on: September 19, 2015, 04:15:16 PM »
PDB is the debug symbol database.  It's something I included in the latest build. 

I just went out for a bike ride and thought about your problem a bit.  You're able to get into the game, and select a race, class, etc.  So you're able to load the main game XML.  But the issue seems to be map-related.  So perhaps the maps aren't being loaded and the issue you're seeing with the memory violations is attempting to access a null MapPtr (shared pointer to a Map).

I noticed in your first two memory dumps that your SOTW directory had a space in the name.  Mine have never had that.  This is a shot in the dark, but can you try opening the latest build to e.g. "c:\sotw" and see if you're able to run the game and enter maps?  I'm wondering if the map loading code is not able to handle directories with spaces in them.

Thanks for the latest dump - I've saved it and will take a look at it tomorrow morning to see if I can get closer to figuring this thing out.  I really appreciate your patience.  I know this must be frustrating!

Aleksanderus

  • Rogueliker
  • ***
  • Posts: 179
  • Karma: +0/-0
  • Champion of the unknown
    • View Profile
Re: Shadow of the Wyrm 0.4.1 (formerly Savage Lands)
« Reply #54 on: September 19, 2015, 04:34:31 PM »
PDB is the debug symbol database.  It's something I included in the latest build. 

I just went out for a bike ride and thought about your problem a bit.  You're able to get into the game, and select a race, class, etc.  So you're able to load the main game XML.  But the issue seems to be map-related.  So perhaps the maps aren't being loaded and the issue you're seeing with the memory violations is attempting to access a null MapPtr (shared pointer to a Map).

I noticed in your first two memory dumps that your SOTW directory had a space in the name.  Mine have never had that.  This is a shot in the dark, but can you try opening the latest build to e.g. "c:\sotw" and see if you're able to run the game and enter maps?  I'm wondering if the map loading code is not able to handle directories with spaces in them.

Thanks for the latest dump - I've saved it and will take a look at it tomorrow morning to see if I can get closer to figuring this thing out.  I really appreciate your patience.  I know this must be frustrating!
It isn't frustrating I just want to play this awesome game ;D
And also I will send you email with something what I hope will be next feature.

Zireael

  • Rogueliker
  • ***
  • Posts: 604
  • Karma: +0/-0
    • View Profile
Re: Shadow of the Wyrm 0.4.1 (formerly Savage Lands)
« Reply #55 on: September 19, 2015, 04:58:42 PM »
Quote
I noticed in your first two memory dumps that your SOTW directory had a space in the name.

My directory also has a space in the name and I'm on a Polish locale. So it's not either of those.

Aleksanderus, do your Windows user name or the path SotW is in contain Polish diacritics?

Aleksanderus

  • Rogueliker
  • ***
  • Posts: 179
  • Karma: +0/-0
  • Champion of the unknown
    • View Profile
Re: Shadow of the Wyrm 0.4.1 (formerly Savage Lands)
« Reply #56 on: September 19, 2015, 05:07:15 PM »
Quote
I noticed in your first two memory dumps that your SOTW directory had a space in the name.

My directory also has a space in the name and I'm on a Polish locale. So it's not either of those.

Aleksanderus, do your Windows user name or the path SotW is in contain Polish diacritics?
No I checked it and no polish letters like ę or ó and other letters which are from poland.

jcd748

  • Rogueliker
  • ***
  • Posts: 231
  • Karma: +0/-0
    • View Profile
Re: Shadow of the Wyrm 0.4.1 (formerly Savage Lands)
« Reply #57 on: September 19, 2015, 05:35:32 PM »
Quote
I noticed in your first two memory dumps that your SOTW directory had a space in the name.

My directory also has a space in the name and I'm on a Polish locale. So it's not either of those.

Aleksanderus, do your Windows user name or the path SotW is in contain Polish diacritics?
No I checked it and no polish letters like ę or ó and other letters which are from poland.

Okay, I can open this in VS now and step through the stack.  This is super helpful - hopefully it will let me figure out what the issue is.  I don't have any more time today (have to do real-world adult stuff), but I will look at this first thing tomorrow morning.

And if you have feature requests, that's fantastic!  You can post them here, or PM me here or on reddit (I'm jcd748 there as well) and I'll add them to my suggestions file.

Aleksanderus

  • Rogueliker
  • ***
  • Posts: 179
  • Karma: +0/-0
  • Champion of the unknown
    • View Profile
Re: Shadow of the Wyrm 0.4.1 (formerly Savage Lands)
« Reply #58 on: September 19, 2015, 05:38:52 PM »
Quote
I noticed in your first two memory dumps that your SOTW directory had a space in the name.

My directory also has a space in the name and I'm on a Polish locale. So it's not either of those.

Aleksanderus, do your Windows user name or the path SotW is in contain Polish diacritics?
No I checked it and no polish letters like ę or ó and other letters which are from poland.

Okay, I can open this in VS now and step through the stack.  This is super helpful - hopefully it will let me figure out what the issue is.  I don't have any more time today (have to do real-world adult stuff), but I will look at this first thing tomorrow morning.

And if you have feature requests, that's fantastic!  You can post them here, or PM me here or on reddit (I'm jcd748 there as well) and I'll add them to my suggestions file.
Very good that you are open to features becouse I have got A LOT of them ;D

jcd748

  • Rogueliker
  • ***
  • Posts: 231
  • Karma: +0/-0
    • View Profile
Re: Shadow of the Wyrm 0.4.1 (formerly Savage Lands)
« Reply #59 on: September 19, 2015, 05:39:04 PM »
I think you have a large display - 94x25 according to the stack details.  Can you try reducing your prompt to 80x25 and seeing if that works?

Let me know if that works for you.  If it does, I'll get this fixed and included in the final v0.4.2.  This looks like a bug in my map display code.