Switching autotrade to use StockpileIterator.

This should speed it up a bit, and prevents it from marking a stream of empty bins.

plugins/autotrade.cpp

@@ -8,6 +8,7 @@
 #include "df/viewscreen_dwarfmodest.h"
 #include "df/viewscreen_tradegoodsst.h"
 #include "df/building_stockpilest.h"
+#include "modules/Buildings.h"
 #include "modules/Items.h"
 #include "df/building_tradedepotst.h"
 #include "df/general_ref_building_holderst.h"
@@ -202,8 +203,6 @@ static void mark_all_in_stockpiles(vector<PersistentStockpileInfo> &stockpiles)
     if (!depot_info.findDepot())
         return;
 
-    std::vector<df::item*> &items = world->items.other[items_other_id::IN_PLAY];
-
 
     // Precompute a bitmask with the bad flags
     df::item_flags bad_flags;
@@ -218,18 +217,19 @@ static void mark_all_in_stockpiles(vector<PersistentStockpileInfo> &stockpiles)
 
     size_t marked_count = 0;
     size_t error_count = 0;
-    for (size_t i = 0; i < items.size(); i++)
+    for (auto it = stockpiles.begin(); it != stockpiles.end(); it++)
     {
-        df::item *item = items[i];
-        if (item->flags.whole & bad_flags.whole)
+        if (!it->isValid())
             continue;
 
-        if (!is_valid_item(item))
+        Buildings::StockpileIterator stored;
+        for (stored.begin(it->getStockpile()); !stored.done(); ++stored)
+        {
+            df::item *item = *stored;
+            if (item->flags.whole & bad_flags.whole)
                 continue;
 
-        for (auto it = stockpiles.begin(); it != stockpiles.end(); it++)
-        {
-            if (!it->inStockpile(item))
+            if (!is_valid_item(item))
                 continue;
 
             // In case of container, check contained items for mandates

plugins/uicommon.h

@@ -346,6 +346,11 @@ public:
         return this->sp == sp;
     }
 
+    df::building_stockpilest* getStockpile()
+    {
+        return sp;
+    }
+
 protected:
     int32_t id;
     df::building_stockpilest* sp;