xo-alloc2: working operator* for AAllocIterator

include/xo/alloc2/alloc/RAllocIterator.hpp

@@ -27,6 +27,8 @@ namespace xo {
                 return O::iface()->compare(O::data(), other); }
             void next() noexcept { O::iface()->next(O::data()); }
 
+            AllocInfo operator*() const noexcept { return deref(); }
+
             /** triggers operator++ in obj<RAllocIterator<Object>> **/
             void _preincrement() noexcept { this->next(); }
 

src/alloc2/IAllocIterator_DArenaIterator.cpp

@@ -32,7 +32,7 @@ namespace xo {
             if (!other)
                 return cmpresult::incomparable();
 
-            DArenaIterator & other_ix = *other;
+            DArenaIterator & other_ix = *other.data();
 
             log && log(xtag("&other_ix", &other_ix),
                        xtag("other_ix.arena", other_ix.arena_),

src/alloc2/IAllocIterator_DX1CollectorIterator.cpp

@@ -25,7 +25,7 @@ namespace xo {
             if (!other)
                 return cmpresult::incomparable();
 
-            DX1CollectorIterator & other_ix = *other;
+            DX1CollectorIterator & other_ix = *other.data();
 
             return ix.compare(other_ix);
         }

utest/DArenaIterator.test.cpp

@@ -269,6 +269,22 @@ namespace xo {
                     REQUIRE(ix == end_ix);
                 }
             }
+
+            // repeat, this time using range iteration
+            {
+                DArena scratch_mm
+                    = DArena::map(
+                        ArenaConfig{
+                            .size_ = 4*1024,
+                            .hugepage_z_ = 4*1024});
+
+                for (const auto & info : a1o.alloc_range(scratch_mm)) {
+                    REQUIRE(info.is_valid());
+                    REQUIRE(info.size() == padding::with_padding(req_z));
+                    REQUIRE(info.payload().first == mem);
+                    REQUIRE(info.payload().second == mem + info.size());
+                }
+            }
         }
 
     } /*namespace ut*/

utest/DX1CollectorIterator.test.cpp

@@ -198,6 +198,22 @@ namespace xo {
                     REQUIRE(ix == end_ix);
                 }
             }
+
+            // repeat, this time using range iteration
+            {
+                DArena scratch_mm
+                    = DArena::map(
+                        ArenaConfig{
+                            .size_ = 4*1024,
+                            .hugepage_z_ = 4*1024});
+
+                for (const auto & info : a1o.alloc_range(scratch_mm)) {
+                    REQUIRE(info.is_valid());
+                    REQUIRE(info.size() == padding::with_padding(req_z));
+                    REQUIRE(info.payload().first == mem);
+                    REQUIRE(info.payload().second == mem + info.size());
+                }
+            }
         }
     } /*namespace ut*/
 } /*namespace xo*/