On Sat, 10 Aug 2013 11:39:40 +0200 Michael Vehrs <michael.bursc...@gmx.de> wrote: > Excellent. The change role message is, of course, an essential part of > switching to a role model.
No worries. What is your preferred sequence here? I am busy chipping away at the PlayerExploredTile problems again ATM, but I can probably get a first cut of ChangeRoleMessage (using enum Role) done early next week, including having the AI use it. Or do you want to get the class Role routines in shape first? Either way, recently I extended UnitLocation to implement canBuildEquipment, so as to have a uniform test across Europe and Settlements. Perhaps this should now become canBuildRoleEquipment, and then add buildRoleEquipment(Unit, Role) to do the real work, which would be much the same as the existing Unit.equipForRole for Settlements (Europe is messier, and probably has to be done in ServerEurope because of the sales involved). Does that make sense as part of the API? (The names are a bit odd I admit, I was trying to avoid bumping into the similar Settlement.canProvideEquipment that the AI uses for planning) Cheers, Mike Pope
signature.asc
Description: PGP signature
------------------------------------------------------------------------------ Get 100% visibility into Java/.NET code with AppDynamics Lite! It's a free troubleshooting tool designed for production. Get down to code-level detail for bottlenecks, with <2% overhead. Download for free and get started troubleshooting in minutes. http://pubads.g.doubleclick.net/gampad/clk?id=48897031&iu=/4140/ostg.clktrk
_______________________________________________ Freecol-developers mailing list Freecol-developers@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/freecol-developers
