Author Topic: Save Game  (Read 1869 times)

Omega

  • MegaGlest Team
  • Dragon
  • ********
  • Posts: 6,167
  • Professional bug writer
    • View Profile
    • Personal site
Save Game
« on: 25 August 2009, 05:04:11 »
Doesn't save game normally work? Didn't it work in 0.2.11? Because apparently it isn't working for me now, in v0.2.12...

It makes the map, tileset, and tech paths wrong. For example, it puts the map as X, when it should be maps/X.gbm. The techtree should be techs/X, and the tileset should be tilesets/X. However, even then it crashes (but displays the load game screen right, at least).

What is wrong?!?

Why?
Edit the MegaGlest wiki: http://docs.megaglest.org/

My personal projects: http://github.com/KatrinaHoffert

silnarm

  • Local Moderator
  • Behemoth
  • ********
  • Posts: 1,373
    • View Profile
Re: Save Game
« Reply #1 on: 25 August 2009, 09:28:00 »
What is wrong?!?
The paths stored in the 'GameSettings' object have changed.  We didn't test game saving, we probably need a checklist of things to check before a release, to make sure we haven't broken stuff.

Quote
Why?
3.2.2 Merge.
Glest Advanced Engine - Code Monkey

Timeline | Downloads

Omega

  • MegaGlest Team
  • Dragon
  • ********
  • Posts: 6,167
  • Professional bug writer
    • View Profile
    • Personal site
Re: Save Game
« Reply #2 on: 26 August 2009, 05:30:13 »
Of course. I guess its a thing to do for the next version? Fixing the things you broke? :D
Edit the MegaGlest wiki: http://docs.megaglest.org/

My personal projects: http://github.com/KatrinaHoffert

daniel.santos

  • Guest
Re: Save Game
« Reply #3 on: 9 September 2009, 00:06:57 »
hehe =)  Typically, I have done bu fixes as a #.#.#a release, so 0.2.12a could contain such a bug fix. 

silnarm, I need to start digging into the new sources, so unless you've already started on this one, maybe I can fix it?

silnarm

  • Local Moderator
  • Behemoth
  • ********
  • Posts: 1,373
    • View Profile
Re: Save Game
« Reply #4 on: 9 September 2009, 07:22:28 »
silnarm, I need to start digging into the new sources, so unless you've already started on this one, maybe I can fix it?

I had not yet got to this one, all yours!  ;D

It shouldn't be a hard one, previously the full path was stored, now it's just the filename, or perhaps it was the other way around ;)  Hailstone fixed these up, I think he added 'convenience' functions to get the full path.
Glest Advanced Engine - Code Monkey

Timeline | Downloads

Omega

  • MegaGlest Team
  • Dragon
  • ********
  • Posts: 6,167
  • Professional bug writer
    • View Profile
    • Personal site
Re: Save Game
« Reply #5 on: 10 September 2009, 05:39:32 »
BTW: I feel it is worthy to mention that I manually tried to change the paths in the XML myself at one point, and got everything to display seemingly correct, except for the faction. It may have been right, but it was all lowercase, and usually the first char is automatically capitalized, so I think it needs a path too, but no amount of fumbling around could fix it.

Hopefully you'll get it up soon Daniel, along with the few others mentioned before.

PS: The error was something with lastDir() I believe in it. Can't say for sure because I can't currently check. Unless you know which that error message is (found in the util.cpp file [I think thats the shared]), I'll post it some other time.
Edit the MegaGlest wiki: http://docs.megaglest.org/

My personal projects: http://github.com/KatrinaHoffert

 

anything