billw2012

Text needs cleanup - spell and grammar check, lets automate it

9

Spelling, incorrect capitalization. 

This changes the group and units for no reason, causing logic loops. Instead implement functions for determining transport possibilities (and other stuff probably) for SM units, so we can split...

Perhaps the search algorithm is a problem, they tend to double back on themselves a lot. Check they are set to go far afield.

This might have been done and optimized already, just double check. Automated missionaries without valid targets are causing performance problems.

- Ensure they stay safe, run away from enemy, hide during war - Put cap on spending, make it configurable globally - Optionally calculate value for routes per plot so...

Look into replacing hashmaps with googles implementation.

4

https://github.com/sparsehash/sparsehash It isn't the absolute best performing, but it is C++03 compatible, and way better than any stl map implementations.

Look into replacing vector with a better implementation in some cases.

1

Old implementation is not good, it should have small vector optimization and better resize factor (need to find out what the optimal one is).

`AI_fulfillPropertyControlNeed` function, calling `AI_plotTargetMissionAIs`

Add the prerequisites to the queue as well. In fact always allow enqueuing of buildings that can't actually be built yet (marking them clearly as not currently buildable). Don't dequeue...

Update `CvGame::loadPirateShip` to consider size matters for transporting.

1

Ideally we should be entirely encapsulating all transport logic so that separate code paths are not required.