Version 0.4.2

Post by **Dave** » August 30th, 2003, 7:57 am

http://wesnoth.whitevine.net/files/wesnoth-0.4.2.tar.gz

Changelog:

ADDITIONAL ITEMS THAT I FORGOT TO RECORD IN THE GAME CHANGELOG:

- Saved games are now stored in a .wesnoth/saves directory which is located under $HOME (or under the directory the program is run from is $HOME is not defined). There is now a user preferences file in .wesnoth/preferences -- for this reason, saved games will NOT be compatible, although you can restore your previously saved games if you know what you're doing.
- Added facility to change languages in-game. The language used by default will be obtained from the $LANG environment variable
- Recall is now sorted and in a table

Version 0.4.2:
- If you hold shift, the game won't scroll at all, it'll jump between locations
- Added attack animations for necromancer and mage
- Added in Lohari's images with corrected shadows
- Added in Lohari's crossed daggers for battles, instead of the cross
- Added in Paladin's patch to highlight the hex of the unit that is currently selected
- Used new Makefile provided by zas
- Added setting of window title as suggested by zas
- Changed so that holding shift skips fading in recruiting units
- Increased cost of Naga from 8 -> 11 gold to make scenario 3 easier
- Fixed bug with recalling in mid-level saved games - should save alot of game corruption issues
- Made it so attack sounds do not play while loading game
- Added animations for Goblin Knight, Wolf Rider, and Troll Whelp
- Made it so Glordorf in Scenario 2 doesn't join the player
- Added in new missile images done by fmunoz
- Fixed bug where merman's storm trident would run out when the merman advances, or at the end of the level
- Page up and page down can be used to maneuver through menus
- Fixed bug reported by Jaramir, where exiting a multiplayer game would cause the save state to be remembered
- Fixed bug reported by Jaramir, where recruiting or recalling a unit wouldn't update your gold immediately
- Added better guarantees that when an AI attacks a unit, the unit being attacked won't be mostly off the screen
- Changed Necromancer to level 2
- Fixed bug where quitting the game by pressing escape during opening dialog sequence would cause the game to crash
- Added facility suggested by miyo where an entire directory can be scanned to look for configuration files. Re-arranged files in data/ to utilize this.
- Changed configuration files to allow [/tagname] to end a tag instead of [end]. Added better error handling for bad configuration files
- Added utility make_translation which when run, will construct a sample translation with all the strings that can be translated in it
- Removed Outrider's spear/charge attack and replaced it with a sword

miyo · Post by **miyo** » August 30th, 2003, 8:19 am

Recall sorting?

Post by **Dave** » August 30th, 2003, 8:36 am

I thought I added that in to the changelog, apparently not!

I've edited and added it.

David

miyo · Post by **miyo** » August 30th, 2003, 8:46 am

Warnings when building (these have been around also in previous releases - but now they are reported)...

OS: Debian GNU/Linux (Sarge)
GCC: 3.3.1

g++ `sdl-config --cflags` `freetype-config --cflags` -c display.cpp -o display.o
display.cpp: In member function `void display::scroll_to_tile(int, int, display::SCROLL_TYPE)':
display.cpp:213: warning: passing `const double' for argument passing 1 of `int abs(int)'
display.cpp:213: warning: argument to `int' from `const double'
display.cpp:213: warning: passing `const double' for argument passing 1 of `int abs(int)'
display.cpp:213: warning: argument to `int' from `const double'
display.cpp:214: warning: passing `const double' for argument passing 1 of `int abs(int)'
display.cpp:214: warning: argument to `int' from `const double'
display.cpp:214: warning: passing `const double' for argument passing 1 of `int abs(int)'
display.cpp:214: warning: argument to `int' from `const double'
display.cpp: In member function `void display::scroll_to_tiles(int, int, int, int, display::SCROLL_TYPE)':
display.cpp:256: warning: passing `double' for argument passing 1 of `int abs(int)'
display.cpp:256: warning: argument to `int' from `double'
display.cpp:257: warning: passing `double' for argument passing 1 of `int abs(int)'
display.cpp:257: warning: argument to `int' from `double'

But those are no show-stoppers.

- Miyo

Post by **Dave** » August 30th, 2003, 8:55 am

Yes I am planning to fix them soon. I've upgraded my compiler recently and it now reports them....

But what is it you mean about them being 'show stoppers' exactly? They're just warnings right?

David

Post by **Dave** » August 30th, 2003, 9:00 am

Okay, patch to display.cpp to make the warnings go away:

Code: Select all

213,214c213,214
<       int num_moves = static_cast<int>((fabs(xmove) > fabs(ymove) ?
<                                         fabs(xmove):fabs(ymove))/speed);
---
>       int num_moves = static_cast<int>((abs(xmove) > abs(ymove) ?
>                                         abs(xmove):abs(ymove))/speed);
256,257c256,257
<       const double diffx = fabs(xpos1 - xpos2);
<       const double diffy = fabs(ypos1 - ypos2);
---
>       const double diffx = abs(xpos1 - xpos2);
>       const double diffy = abs(ypos1 - ypos2);

There are quite a few warnings still if one uses the -Wall gcc flag, but since they are all 'comparison between unsigned integer and signed integer', which I don't consider to be a problem, I'm not going to fix them.

David

miyo · Post by **miyo** » August 30th, 2003, 9:01 am

Ooops... <syntax error>

fixed: now -> no

- Miyo

Post by **Dave** » August 30th, 2003, 9:02 am

Well they're fixed now anyway

Post by **Dave** » August 30th, 2003, 12:22 pm

There is a problem that now the translations/ and units/ and scenarios/ directories are scanned for *all* files in the directory. This can be a problem for editors that leave swap files in directories being edited, such as vim. The following patch to config.cpp will fix this problem by only retrieving files with the .cfg extension:

Code: Select all

37d36
<       //that end in .cfg
45,47c44
<                       if(f->size() > 4 && std::equal(f->end()-4,f->end(),".cfg")) {
<                               stream << preprocess_file(*f,defines_map,depth);<                       }
---
>                       stream << preprocess_file(*f,defines_map,depth);
211,212d207
<       clear();
<

miyo · Post by **miyo** » August 30th, 2003, 12:29 pm

If you change language to french from main menu... untranslated texts are empty... shouldn't untranslated text be shown in english as fallback?

- Miyo

Post by **Dave** » August 30th, 2003, 12:33 pm

miyo wrote:If you change language to french from main menu... untranslated texts are empty... shouldn't untranslated text be shown in english as fallback?

- Miyo

Possibly.

I'll look into it.

David

miyo · Post by **miyo** » August 31st, 2003, 12:04 am

Swordman -> Swordsman

- Miyo

miyo · Post by **miyo** » August 31st, 2003, 2:21 pm

Playtesting: scenario 1 and scenario 2 (this far I got before 0.4.3 was released) without troubles.

- Miyo