once you remove the m_is_done_iterating, this will be the standard default construction and you can remove the definition. just `SamWindowIterator() = default;` will suffice.