Revert operator abuse

Author: Skylion007

include/pybind11/pytypes.h

@@ -13,7 +13,6 @@
 #include "buffer_info.h"
 
 #include <exception>
-#include <stdexcept>
 #include <string>
 #include <type_traits>
 #include <utility>
@@ -388,16 +387,13 @@ class PYBIND11_EXPORT_EXCEPTION error_already_set : public std::runtime_error {
         }
     }
 
-    error_already_set(const error_already_set &) noexcept = default;
-    error_already_set(error_already_set &&) noexcept = default;
+    error_already_set(const error_already_set &) = default;
+    error_already_set(error_already_set &&) = default;
 
     inline ~error_already_set() override;
 
     const char *what() const noexcept override {
-        auto *sup_what = std::runtime_error::what();
-        if (sup_what[0] != '\0') {
-            return sup_what;
-        } else if (m_lazy_what.empty()) {
+        if (m_lazy_what.empty()) {
             try {
                 m_lazy_what = detail::error_string(m_type.ptr(), m_value.ptr(), m_trace.ptr());
             } catch (const std::exception &e) {
@@ -411,7 +407,6 @@ class PYBIND11_EXPORT_EXCEPTION error_already_set : public std::runtime_error {
                 return m_lazy_what.c_str();
             }
         }
-
         assert(!m_lazy_what.empty());
         return m_lazy_what.c_str();
     }
@@ -420,8 +415,7 @@ class PYBIND11_EXPORT_EXCEPTION error_already_set : public std::runtime_error {
     /// already set it is cleared first.  After this call, the current object no longer stores the
     /// error variables (but the `.what()` string is still available).
     void restore() {
-        // Abuse assignment operator to cache what()
-        std::runtime_error::operator=(std::runtime_error(what())); // Force-build `.what()`.
+        what(); // Force-build `.what()`.
         if (m_lazy_what.empty()) {
             pybind11_fail("Critical error building lazy error_string().");
         }