[Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1252625-statistics-window-update into lp:widelands

2016-03-19 Thread bunnybot
Continuous integration builds have changed state: Travis build 866. State: passed. Details: https://travis-ci.org/widelands/widelands/builds/117151810. Appveyor build 701. State: success. Details: https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_bug_1252625_

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1395278-graphic into lp:widelands

2016-03-19 Thread GunChleoc
GunChleoc has proposed merging lp:~widelands-dev/widelands/bug-1395278-graphic into lp:widelands. Commit message: Refactored member variables in src/graphic. Requested reviews: Widelands Developers (widelands-dev) Related bugs: Bug #1395278 in widelands: "Consolidate naming of member variabl

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

2016-03-19 Thread TiborB
The proposal to merge lp:~widelands-dev/widelands/ship_DBG_window into lp:widelands has been updated. Commit Message changed to: Ships get debug window. Also its content are extended. For more details, see: https://code.launchpad.net/~widelands-dev/widelands/ship_DBG_window/+merge/289110 -- Yo

Re: [Widelands-dev] Widelands::OPtr

2016-03-19 Thread Fòram na Gàidhlig
Glad you found it :) Sgrìobh Tibor Bamhor na leanas 16/03/2016 aig 10:18: > OK, I found that right syntax is > > destination_.get(egbase)->get_dock_position().x > > 2016-03-16 10:28 GMT+01:00 Tibor Bamhor >: > > I run into this compilation error: > > erro

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1395278-scripting into lp:widelands

2016-03-19 Thread Tino
Seems some pre installation scripts failed (downloading InnoSetup) on Appveyor. I've triggered a re-build of this commit: https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_bug_1395278_scripting-686 -- https://code.launchpad.net/~widelands-dev/widelands/bug-139

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/improve-restool into lp:widelands

2016-03-19 Thread Miroslav Remák
This basically fixes bug #1550568 in a better way in terms of performance. -- https://code.launchpad.net/~widelands-dev/widelands/improve-restool/+merge/289305 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/improve-restool into lp:widelands

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1395278-editor into lp:widelands

2016-03-19 Thread Klaus Halfmann
Played this for a while now, found no anomalies. -- https://code.launchpad.net/~widelands-dev/widelands/bug-1395278-editor/+merge/289494 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/bug-1395278-editor. ___ Mailing

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

2016-03-19 Thread GunChleoc
OK, let's merge this then :) @bunnybot merge -- https://code.launchpad.net/~widelands-dev/widelands/ship_DBG_window/+merge/289110 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/ship_DBG_window. ___ Mailing list: htt

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/improve-restool into lp:widelands

2016-03-19 Thread kaputtnik
Review: Approve testing Works fine :-) -- https://code.launchpad.net/~widelands-dev/widelands/improve-restool/+merge/289305 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/improve-restool. ___ Mailing list: https:/

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/improve-restool into lp:widelands

2016-03-19 Thread Miroslav Remák
@tiborb95 Yes, an empty undo action is created. Unfortunately, it doesn't seem to be trivial to fix because of all the "draw tool" mess. -- https://code.launchpad.net/~widelands-dev/widelands/improve-restool/+merge/289305 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/watchwindow-fixes into lp:widelands

2016-03-19 Thread Miroslav Remák
Miroslav Remák has proposed merging lp:~widelands-dev/widelands/watchwindow-fixes into lp:widelands. Requested reviews: Widelands Developers (widelands-dev) For more details, see: https://code.launchpad.net/~widelands-dev/widelands/watchwindow-fixes/+merge/289573 - Fixes some oddities such as

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/improve-restool into lp:widelands

2016-03-19 Thread TiborB
No problem, I understand this is out of scope of this fix -- https://code.launchpad.net/~widelands-dev/widelands/improve-restool/+merge/289305 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/improve-restool. ___ Maili

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1395278-editor into lp:widelands

2016-03-19 Thread GunChleoc
I think we should go through all the clang warnings on Travis sometime and fix. Added replies to your comments. Diff comments: > > === modified file 'src/editor/tools/editor_increase_height_tool.h' > --- src/editor/tools/editor_increase_height_tool.h2016-01-28 05:24:34 > + > +++ src/ed

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1395278-scripting into lp:widelands

2016-03-19 Thread TiborB
Review: Approve LGTM I see appveyor reports failed... -- https://code.launchpad.net/~widelands-dev/widelands/bug-1395278-scripting/+merge/288975 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/bug-1395278-scripting. ___

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

2016-03-19 Thread GunChleoc
Sounds good to me then - I can take care of 1. when you're finished with reworking the info. -- https://code.launchpad.net/~widelands-dev/widelands/ship_DBG_window/+merge/289110 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/ship_DBG_window. _

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1395278-graphic into lp:widelands

2016-03-19 Thread noreply
The proposal to merge lp:~widelands-dev/widelands/bug-1395278-graphic into lp:widelands has been updated. Status: Needs review => Merged For more details, see: https://code.launchpad.net/~widelands-dev/widelands/bug-1395278-graphic/+merge/289402 -- Your team Widelands Developers is subscrib

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1395278-network-io-wui into lp:widelands

2016-03-19 Thread noreply
The proposal to merge lp:~widelands-dev/widelands/bug-1395278-network-io-wui into lp:widelands has been updated. Status: Needs review => Merged For more details, see: https://code.launchpad.net/~widelands-dev/widelands/bug-1395278-network-io-wui/+merge/289557 -- Your team Widelands Develope

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1395278-scripting into lp:widelands

2016-03-19 Thread noreply
The proposal to merge lp:~widelands-dev/widelands/bug-1395278-scripting into lp:widelands has been updated. Status: Needs review => Merged For more details, see: https://code.launchpad.net/~widelands-dev/widelands/bug-1395278-scripting/+merge/288975 -- Your team Widelands Developers is subs

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1395278-network-io-wui into lp:widelands

2016-03-19 Thread GunChleoc
Thanks! @bunnybot merge Diff comments: > === modified file 'src/io/filesystem/layered_filesystem.cc' > --- src/io/filesystem/layered_filesystem.cc 2016-02-18 18:27:52 + > +++ src/io/filesystem/layered_filesystem.cc 2016-03-19 10:27:47 + > @@ -89,7 +89,7 @@ >

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1252625-statistics-window-update into lp:widelands

2016-03-19 Thread GunChleoc
GunChleoc has proposed merging lp:~widelands-dev/widelands/bug-1252625-statistics-window-update into lp:widelands. Commit message: Plot areas now update their data less often. Requested reviews: Widelands Developers (widelands-dev) Related bugs: Bug #1252625 in widelands: "General statistic

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1395278-network-io-wui into lp:widelands

2016-03-19 Thread Klaus Halfmann
Review: Approve Found only renamings that are fine with me. Copiled this and played it a while, found no anomalies. Diff comments: > === modified file 'src/io/filesystem/layered_filesystem.cc' > --- src/io/filesystem/layered_filesystem.cc 2016-02-18 18:27:52 + > +++ src/io/filesystem/layer

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/memleaks-and-uninit-vars into lp:widelands

2016-03-19 Thread noreply
The proposal to merge lp:~widelands-dev/widelands/memleaks-and-uninit-vars into lp:widelands has been updated. Status: Needs review => Merged For more details, see: https://code.launchpad.net/~widelands-dev/widelands/memleaks-and-uninit-vars/+merge/289561 -- Your team Widelands Developers i

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/improve-restool into lp:widelands

2016-03-19 Thread TiborB
Review: Approve Exactly how I meant it! BTW, how the game behaves if no fields are changed? Is "empty" undo action created? I have not tested it -- https://code.launchpad.net/~widelands-dev/widelands/improve-restool/+merge/289305 Your team Widelands Developers is subscribed to branch lp:~wide

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/memleaks-and-uninit-vars into lp:widelands

2016-03-19 Thread GunChleoc
@bunnybot merge -- https://code.launchpad.net/~widelands-dev/widelands/memleaks-and-uninit-vars/+merge/289561 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/memleaks-and-uninit-vars. ___ Mailing list: https://launchp

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1395278-network-io-wui into lp:widelands

2016-03-19 Thread bunnybot
Continuous integration builds have changed state: Travis build 859. State: passed. Details: https://travis-ci.org/widelands/widelands/builds/117091384. Appveyor build 694. State: success. Details: https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_bug_1395278_

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands-website/fix-installation into lp:widelands-website

2016-03-19 Thread GunChleoc
Yes, I was unable to find current repositories for everything. Our only solution for this is to get the update to Django 1.8 working. -- https://code.launchpad.net/~widelands-dev/widelands-website/fix-installation/+merge/286797 Your team Widelands Developers is subscribed to branch lp:widelands-w

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1395278-scripting into lp:widelands

2016-03-19 Thread GunChleoc
Thanks for the review :) AppVeyor fails a lot on the downloads - the service is quite new, and I guess they're still working out the kinks. @bunnybot merge -- https://code.launchpad.net/~widelands-dev/widelands/bug-1395278-scripting/+merge/288975 Your team Widelands Developers is subscribed to

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/improve-restool into lp:widelands

2016-03-19 Thread TiborB
And similarly, for increasing resources - do nothing if resources are at max and for decreasing do nothing if resources are currently at 0. ? -- https://code.launchpad.net/~widelands-dev/widelands/improve-restool/+merge/289305 Your team Widelands Developers is requested to review the proposed mer

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/show-ship-names into lp:widelands

2016-03-19 Thread GunChleoc
GunChleoc has proposed merging lp:~widelands-dev/widelands/show-ship-names into lp:widelands. Commit message: Added census/statistics strings to ships and ship construction. Ship states are now an enum class. Requested reviews: Widelands Developers (widelands-dev) For more details, see: https

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/memleaks-and-uninit-vars into lp:widelands

2016-03-19 Thread GunChleoc
Review: Approve LGTM :) -- https://code.launchpad.net/~widelands-dev/widelands/memleaks-and-uninit-vars/+merge/289561 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/memleaks-and-uninit-vars. ___ Mailing list: https:

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/memleaks-and-uninit-vars into lp:widelands

2016-03-19 Thread Miroslav Remák
Miroslav Remák has proposed merging lp:~widelands-dev/widelands/memleaks-and-uninit-vars into lp:widelands. Requested reviews: Widelands Developers (widelands-dev) For more details, see: https://code.launchpad.net/~widelands-dev/widelands/memleaks-and-uninit-vars/+merge/289561 This also fixes

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/improve-restool into lp:widelands

2016-03-19 Thread noreply
The proposal to merge lp:~widelands-dev/widelands/improve-restool into lp:widelands has been updated. Status: Needs review => Merged For more details, see: https://code.launchpad.net/~widelands-dev/widelands/improve-restool/+merge/289305 -- Your team Widelands Developers is subscribed to br

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1395278-network-io-wui into lp:widelands

2016-03-19 Thread GunChleoc
GunChleoc has proposed merging lp:~widelands-dev/widelands/bug-1395278-network-io-wui into lp:widelands. Commit message: Refactored remaining member variables. Requested reviews: Widelands Developers (widelands-dev) Related bugs: Bug #1395278 in widelands: "Consolidate naming of member varia

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/improve-restool into lp:widelands

2016-03-19 Thread Miroslav Remák
Miroslav Remák has proposed merging lp:~widelands-dev/widelands/improve-restool into lp:widelands. Requested reviews: Widelands Developers (widelands-dev) For more details, see: https://code.launchpad.net/~widelands-dev/widelands/improve-restool/+merge/289305 -- Your team Widelands Developers

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/improve-restool into lp:widelands

2016-03-19 Thread bunnybot
Continuous integration builds have changed state: Travis build 850. State: passed. Details: https://travis-ci.org/widelands/widelands/builds/116499709. Appveyor build 684. State: success. Details: https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_improve_rest

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

2016-03-19 Thread kaputtnik
Review: Approve testing I think all is fine now :-) -- https://code.launchpad.net/~widelands-dev/widelands/about/+merge/288697 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/about. ___ Mailing list: https://launchp

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

2016-03-19 Thread TiborB
Gun, I am done, it is your turn now -- https://code.launchpad.net/~widelands-dev/widelands/ship_DBG_window/+merge/289110 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/ship_DBG_window. ___ Mailing list: https://launc

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

2016-03-19 Thread GunChleoc
Review: Approve I'm done - I prettied things up a bit. I noticed that the information on the bottom just keeps expanding rather than being replaces. Is this on purpose? If it is on purpose, please feel free to merge. Otherwise, please fix. -- https://code.launchpad.net/~widelands-dev/widelands

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1395278-scripting into lp:widelands

2016-03-19 Thread bunnybot
Continuous integration builds have changed state: Travis build 842. State: passed. Details: https://travis-ci.org/widelands/widelands/builds/116056744. Appveyor build 686. State: success. Details: https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_bug_1395278_

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1395278-graphic into lp:widelands

2016-03-19 Thread TiborB
Review: Approve LGTM -- https://code.launchpad.net/~widelands-dev/widelands/bug-1395278-graphic/+merge/289402 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/bug-1395278-graphic. ___ Mailing list: https://launchpad.n

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/improve-restool into lp:widelands

2016-03-19 Thread TiborB
It looks good to me, just look at small question in diff Diff comments: > > === modified file 'src/editor/tools/editor_set_resources_tool.cc' > --- src/editor/tools/editor_set_resources_tool.cc 2016-01-14 22:09:24 > + > +++ src/editor/tools/editor_set_resources_tool.cc 2016-03-16 21

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1395278-editor into lp:widelands

2016-03-19 Thread noreply
The proposal to merge lp:~widelands-dev/widelands/bug-1395278-editor into lp:widelands has been updated. Status: Needs review => Merged For more details, see: https://code.launchpad.net/~widelands-dev/widelands/bug-1395278-editor/+merge/289494 -- Your team Widelands Developers is subscribed

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/improve-restool into lp:widelands

2016-03-19 Thread Miroslav Remák
Done. -- https://code.launchpad.net/~widelands-dev/widelands/improve-restool/+merge/289305 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/improve-restool into lp:widelands. ___ Mailing list: http

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1395278-editor into lp:widelands

2016-03-19 Thread GunChleoc
Thanks! @bunnybot merge -- https://code.launchpad.net/~widelands-dev/widelands/bug-1395278-editor/+merge/289494 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/bug-1395278-editor. ___ Mailing list: https://launchpad.