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

2018-04-13 Thread GunChleoc
OK, I have set it back to "Work in Progress" -- https://code.launchpad.net/~widelands-dev/widelands/bug-1746481/+merge/337125 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/bug-1746481 into lp

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

2018-04-13 Thread GunChleoc
The proposal to merge lp:~widelands-dev/widelands/bug-1746481 into lp:widelands has been updated. Status: Needs review => Work in progress For more details, see: https://code.launchpad.net/~widelands-dev/widelands/bug-1746481/+merge/337125 -- Your team Widelands Developers is reques

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

2018-04-12 Thread TiborB
:~widelands-dev/widelands/bug-1746481 into lp:widelands. ___ Mailing list: https://launchpad.net/~widelands-dev Post to : widelands-dev@lists.launchpad.net Unsubscribe : https://launchpad.net/~widelands-dev More help : https://help.launchpad.net

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

2018-04-12 Thread GunChleoc
Is this branch ready for review? We should get rid of the extra lines in src/main.cc in any case. -- https://code.launchpad.net/~widelands-dev/widelands/bug-1746481/+merge/337125 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/bug-1746481

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

2018-02-10 Thread TiborB
/337125 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/bug-1746481 into lp:widelands. ___ Mailing list: https://launchpad.net/~widelands-dev Post to : widelands-dev@lists.launchpad.net Unsubscribe

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

2018-02-10 Thread GunChleoc
clude " to main.cc in the commit that fixed the printf. -- https://code.launchpad.net/~widelands-dev/widelands/bug-1746481/+merge/337125 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/bug-1746481 into lp

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

2018-02-07 Thread TiborB
://code.launchpad.net/~widelands-dev/widelands/bug-1746481/+merge/337125 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/bug-1746481 into lp:widelands. ___ Mailing list: https://launchpad.net/~widelands-dev Post

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

2018-02-06 Thread TiborB
Please try once more.. -- https://code.launchpad.net/~widelands-dev/widelands/bug-1746481/+merge/337125 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/bug-1746481 into lp:widelands. ___ Mailing

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

2018-02-06 Thread TiborB
I omit to incorporate can_start_working() function to the check altogether, it simply cannot work properly -- https://code.launchpad.net/~widelands-dev/widelands/bug-1746481/+merge/337125 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands

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

2018-02-06 Thread R M
bug-1746481/+merge/337125 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/bug-1746481 into lp:widelands. ___ Mailing list: https://launchpad.net/~widelands-dev Post to : widelands-

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

2018-02-06 Thread TiborB
INDEED, I see a problem (in the code) now. I will fix and let you know... -- https://code.launchpad.net/~widelands-dev/widelands/bug-1746481/+merge/337125 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/bug-1746481 into lp:widelands

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

2018-02-06 Thread R M
://code.launchpad.net/~widelands-dev/widelands/bug-1746481/+merge/337125 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/bug-1746481 into lp:widelands. ___ Mailing list: https://launchpad.net/~widelands-dev Post

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

2018-02-06 Thread TiborB
>before basic economy. This is question of course, if this should be allowed... -- https://code.launchpad.net/~widelands-dev/widelands/bug-1746481/+merge/337125 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/bug-1746481 into

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

2018-02-06 Thread R M
Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/bug-1746481 into lp:widelands. ___ Mailing list: https://launchpad.net/~widelands-dev Post to : widelands-dev@lists.launchpad.net Unsubscribe : https

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

2018-02-04 Thread TiborB
Codecheck complains about printfs, but these are only for temporary DEBUG logs and will be removed before merge... -- https://code.launchpad.net/~widelands-dev/widelands/bug-1746481/+merge/337125 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev

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

2018-02-04 Thread bunnybot
/_widelands_dev_widelands_bug_1746481-2942. -- https://code.launchpad.net/~widelands-dev/widelands/bug-1746481/+merge/337125 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/bug-1746481 into lp:widelands. ___ Mailing list: https

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

2018-02-03 Thread TiborB
lding gets occupied and AI recognizes it. And this can cause inconsistency when a game is saved just during such period. But should not be very frequent After testing is done, I will remove NOCOM lines. -- Your team Widelands Developers is requested to review the proposed merge of lp:~wid