add move construct for double link list

[wokron]

No description provided.

[Copilot]

Pull request overview

This PR updates the intrusive container utilities by adding move support for IntrusiveDoubleList, and extends the unit tests to validate basic move behavior.

Changes:

• Add move constructor and move-assignment operator to IntrusiveDoubleList.
• Remove debug-only ownership tracking from DoubleLinkEntry and related assertions.
• Add a new doctest covering move-assignment of a double intrusive list.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File	Description
include/condy/intrusive.hpp	Adds move operations for IntrusiveDoubleList and removes debug ownership checks from double-link entries.
tests/test_intrusive.cpp	Adds a new test case for IntrusiveDoubleList move behavior.

Comments suppressed due to low confidence (1)

include/condy/intrusive.hpp:110

• The PR removes the DoubleLinkEntry::owner debug field and the associated ownership assertions in push_back/pop_front/remove. That weakens debug-mode detection of cross-list misuse (e.g., calling remove() with an item belonging to a different list), which can silently corrupt lists. If move support is the motivation, consider keeping owner under #ifndef NDEBUG and updating it during move (O(n) but debug-only), or otherwise retaining an equivalent debug membership check.

    void push_back(T *item) noexcept {
        assert(item != nullptr);
        DoubleLinkEntry *entry = &(item->*Member);
        assert(entry->next == nullptr && entry->prev == nullptr);
        entry->next = nullptr;

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.