← Back to team overview

widelands-dev team mailing list archive

Re: [Merge] lp:~widelands-dev/widelands/amazons-coding-changes into lp:widelands

 

Fixed the fieldaction bug and split off FindNodeTerraform.
New terraform syntax:
 "findspace=size:any radius:6 terraform",
 "walk=coords",
 "terraform",
 "return"

There´s no real reason why we need a World or EGBase in Tool parameters, is there? I removed it, as it can be fetched directly from the provided EIA.

I would prefer to keep the EGBase parameter in FindNode::accept. For now World and Map would suffice, but if we want to write some other new worker program for a new tribe someday (e.g. a forester that plants trees only in the territory of enemy players?) there´s no way around an EGBase param, no need to do the whole rework again then IMHO.
-- 
https://code.launchpad.net/~widelands-dev/widelands/amazons-coding-changes/+merge/371725
Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/amazons-coding-changes.


References