widelands-dev team mailing list archive

Thread
Date

Re: [Merge] lp:~widelands-dev/widelands/seafaring-ai into lp:widelands

To: mp+242271@xxxxxxxxxxxxxxxxxx
From: TiborB <tiborb95@xxxxxxxxx>
Date: Mon, 02 Feb 2015 11:52:05 -0000
In-reply-to: <20141119212032.12050.36664.launchpad@ackee.canonical.com>
Reply-to: mp+242271@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

@SirVer

So I did some simple tests and it is BAD. List of ships is empty after load. So AI will not manage them at all.

But his is analogous to productionsites, militarysites vectors when loading game - so I tested militarysites and these are properly initialized = AI knows about all of them after load. (I tested both single player and mulitiplayer game).

Though, when I removed 'if (player_ == nullptr)' test for NoteImmovable it crashed for netwok game and survived for single player game.

But is seems that there is some mechanism that take care of NoteImmovable messages and is not in effect for NoteShipMessages.

What do you think?
-- 
https://code.launchpad.net/~widelands-dev/widelands/seafaring-ai/+merge/242271
Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/seafaring-ai.

References

[Merge] lp:~widelands-dev/widelands/seafaring-ai into lp:widelands
From: TiborB, 2014-11-19