I was wondering if you would consider switching from a deleted default constructor for NodeIterator to a defaulted one, to make the NodeView c++20-range compatible.
The concept std::ranges::range requires that NodeIterator (return value of NodeView::end()) is a sentinel for NodeIterator (return value of NodeView::begin()), which in turn requires that NodeIterator is default constructable.
If you wish, I can also directly provide a PR with this change.
I was wondering if you would consider switching from a
deleted default constructor forNodeIteratorto adefaulted one, to make theNodeViewc++20-range compatible.The concept
std::ranges::rangerequires thatNodeIterator(return value ofNodeView::end()) is a sentinel forNodeIterator(return value ofNodeView::begin()), which in turn requires thatNodeIteratoris default constructable.If you wish, I can also directly provide a PR with this change.