widelands-dev team mailing list archive

Thread
Date

Re: [Merge] lp:~widelands-dev/widelands/frisian_balancing_with_ai_hints into lp:widelands

To: mp+347166@xxxxxxxxxxxxxxxxxx
From: TiborB <tiborb95@xxxxxxxxx>
Date: Wed, 06 Jun 2018 13:31:57 -0000
In-reply-to: <152770868066.28124.257132151679656963.launchpad@ackee.canonical.com>
Reply-to: mp+347166@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

But for performance purposes it cannot stay this way, would you agree with creating two similar functions:

get_stocklevel_lowest(BuildingObserver& bo, const uint32_t gametime, const WareWorker what)
get_stocklevel_highest(BuildingObserver& bo, const uint32_t gametime, const WareWorker what)

?
-- 
https://code.launchpad.net/~widelands-dev/widelands/frisian_balancing_with_ai_hints/+merge/347166
Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/frisian_balancing_with_ai_hints into lp:widelands.

References

[Merge] lp:~widelands-dev/widelands/frisian_balancing_with_ai_hints into lp:widelands
From: TiborB, 2018-05-30