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

2018-02-07 Thread TiborB
I probably know what is going on. The problem is a gap when a building exists 
and is not occupied yet (scratched from the list)
AI when considering new buildings tests:
- it is on the list of (missing) basic buildings
- none is in construction

I have to rethink this
-- 
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://launchpad.net/~widelands-dev
Post to : widelands-dev@lists.launchpad.net
Unsubscribe : https://launchpad.net/~widelands-dev
More help   : https://help.launchpad.net/ListHelp


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

2018-02-07 Thread TiborB
I believed the name is the person who should improve the code in the future.

it is removed now :)
-- 
https://code.launchpad.net/~widelands-dev/widelands/ai_portspaces/+merge/335785
Your team Widelands Developers is subscribed to branch 
lp:~widelands-dev/widelands/ai_portspaces.

___
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/ListHelp


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

2018-02-07 Thread GunChleoc
Yep :)

No need for the extra ? though, it's supposed to be TODO(tiborb):

The reason for the username is that we will know who created the comment, in 
case somebody else will want to fix it later and has a question about the 
comment.
-- 
https://code.launchpad.net/~widelands-dev/widelands/ai_portspaces/+merge/335785
Your team Widelands Developers is subscribed to branch 
lp:~widelands-dev/widelands/ai_portspaces.

___
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/ListHelp