Use existing constructor instead of adding a static method. Thanks @Skylion007 for pointing out.

Author: rwgk

include/pybind11/numpy.h

@@ -564,6 +564,8 @@ class dtype : public object {
         m_ptr = from_args(args).release().ptr();
     }
 
+    /// Return dtype for the given typenum (one of the NPY_TYPES).
+    /// https://numpy.org/devdocs/reference/c-api/array.html#c.PyArray_DescrFromType
     explicit dtype(int typenum)
         : object(detail::npy_api::get().PyArray_DescrFromType_(typenum), stolen_t{}) {
         if (m_ptr == nullptr) {
@@ -586,16 +588,6 @@ class dtype : public object {
         return detail::npy_format_descriptor<typename std::remove_cv<T>::type>::dtype();
     }
 
-    /// Return dtype for the given typenum (one of the NPY_TYPES).
-    /// https://numpy.org/devdocs/reference/c-api/array.html#c.PyArray_DescrFromType
-    static dtype from_typenum(int typenum) {
-        auto *ptr = detail::npy_api::get().PyArray_DescrFromType_(typenum);
-        if (!ptr) {
-            throw error_already_set();
-        }
-        return reinterpret_steal<dtype>(ptr);
-    }
-
     /// Size of the data type in bytes.
     ssize_t itemsize() const { return detail::array_descriptor_proxy(m_ptr)->elsize; }
 
@@ -1293,7 +1285,7 @@ struct npy_format_descriptor<
 public:
     static constexpr int value = values[detail::is_fmt_numeric<T>::index];
 
-    static pybind11::dtype dtype() { return pybind11::dtype::from_typenum(value); }
+    static pybind11::dtype dtype() { return pybind11::dtype(/*typenum*/ value); }
 };
 
 template <typename T>
@@ -1302,7 +1294,7 @@ struct npy_format_descriptor<T, enable_if_t<is_same_ignoring_cvref<T, PyObject *
 
     static constexpr int value = npy_api::NPY_OBJECT_;
 
-    static pybind11::dtype dtype() { return pybind11::dtype::from_typenum(value); }
+    static pybind11::dtype dtype() { return pybind11::dtype(/*typenum*/ value); }
 };
 
 #define PYBIND11_DECL_CHAR_FMT                                                                    \