There is no reason why it should not be `noexcept`. `std::swap` cannot have any exceptions or handle any.
@@ -518,7 +518,7 @@ public:
*this = std::move(s);
}
- void swap(String& other)
+ void swap(String& other) noexcept
{
std::swap(this->string_, other.string_);
std::swap(this->view_, other.view_);