On 03.10.2012 15:10, Michael T. Pope wrote:
> On Wed, 3 Oct 2012 02:08:08 PM Michael Vehrs wrote:
>    
>> I'm just wondering about ServerColony.csNewTurn(). If a new building is
>> built, we consider the tile to be dirty, since a building upgrade might
>> have ejected units (to the tile). If a new unit is created, we do not
>> consider the tile to be dirty,
>>      
> Actually I think we do, the code no model of clarity, alas.  If we build a
> unit (or building), the build queue is added to the built list.  A bit further
> on, we always set the tile to be dirty if the built list is non-empty.
>
> I think it would be less misleading to remove the "tileDirty = true" at 249.
> The csNewTurn routines show evidence of evolution:-S.
>
> Cheers,
> Mike Pope
>    

Alright. I see I should have read the whole shebang before jumping to 
conclusions. Still, isn't it true that the tile is only dirty if a unit 
is added?

Regards

Michael



------------------------------------------------------------------------------
Don't let slow site performance ruin your business. Deploy New Relic APM
Deploy New Relic app performance management and know exactly
what is happening inside your Ruby, Python, PHP, Java, and .NET app
Try New Relic at no cost today and get our sweet Data Nerd shirt too!
http://p.sf.net/sfu/newrelic-dev2dev
_______________________________________________
Freecol-developers mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/freecol-developers

Reply via email to