Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/fh1-multitexture into lp:widelands

2017-05-12 Thread kaputtnik
Got a compiler warning: /home/kaputtnik/Quellcode/widelands-repo/fh1-multitexture/src/graphic/text/rendered_text.cc:129:14: Warnung: unused variable »maximum_size« [-Wunused-variable] const int maximum_size = kMinimumSizeForTextures; ^~~~ And some crashes, but i am not

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/fh1-winconditions into lp:widelands

2017-05-12 Thread bunnybot
Continuous integration builds have changed state: Travis build 2179. State: passed. Details: https://travis-ci.org/widelands/widelands/builds/231575243. Appveyor build 2014. State: success. Details: https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_fh1_wincon

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/fh1-winconditions into lp:widelands

2017-05-12 Thread Klaus Halfmann
Uhm, this is all lua, what would be the way to test this? I cannot judge much about that code. I will however compile the code and play around with some winconditions -- https://code.launchpad.net/~widelands-dev/widelands/fh1-winconditions/+merge/323987 Your team Widelands Developers is requested

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/fh1-winconditions into lp:widelands

2017-05-12 Thread GunChleoc
GunChleoc has proposed merging lp:~widelands-dev/widelands/fh1-winconditions into lp:widelands. Commit message: Converted win condition scripts to new font renderer. Formatting functions for the new renderer live in data/scripting/richtext.lua Requested reviews: Widelands Developers (wideland

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/empire-mission-3 into lp:widelands

2017-05-12 Thread GunChleoc
Code refactoring is done, except for 1 NOCOM in the code that I need Hessenfarmer's opinion on. Once that is dealt with, the scenario should be finished and we need to playtest again - to save some time, I haven't tested my changes yet. -- https://code.launchpad.net/~widelands-dev/widelands/emp

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/empire-mission-3 into lp:widelands

2017-05-12 Thread GunChleoc
Yep, I already fixed those two, but I didn't upload yet. I'm going through the code with a fine-tooth comb at the moment. -- https://code.launchpad.net/~widelands-dev/widelands/empire-mission-3/+merge/321434 Your team Widelands Developers is requested to review the proposed merge of lp:~wideland