We check if the Node's children array is empty here. But because we pre-allocated 256 slots in the std::array here, the call to empty() will always return false. We should fix this to return nullptr appropriately and add a unit test when there are no children.