widelands-dev team mailing list archive

[TiborB <tiborb95@xxxxxxxxx>]

I left a comment in the diff, one comment should be changed, if you will touch the code, please change it.
I think that only constructionsites can have such delayed deliveries...

Diff comments:

> === modified file 'src/economy/economy.cc'
> --- src/economy/economy.cc	2016-12-01 16:59:34 +0000
> +++ src/economy/economy.cc	2017-01-18 21:16:27 +0000
> @@ -701,6 +701,9 @@
>   * Walk all Requests and find potential transfer candidates.
>  */
>  void Economy::process_requests(Game& game, RSPairStruct& s) {
> +	// Algorithm can decide that wares are not to be delivered to productionsite

productionsite -> constructionsite

> +	// right now, therefore we need to shcedule next pairing
> +	bool postponed_pairing_needed = false;
>  	for (Request* temp_req : requests_) {
>  		Request& req = *temp_req;
>  


-- 
https://code.launchpad.net/~widelands-dev/widelands/bug-1656671/+merge/315075
Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/bug-1656671.