fix: try disabling inside the namespace

Signed-off-by: Henry Schreiner <henryschreineriii@gmail.com>

Author: henryiii

include/pybind11/cast.h

@@ -868,12 +868,11 @@ struct always_construct_holder {
     static constexpr bool value = Value;
 };
 
-PYBIND11_WARNING_PUSH
-PYBIND11_WARNING_DISABLE_GCC("-Wpedantic")
-PYBIND11_WARNING_DISABLE_CLANG("-Wgnu-zero-variadic-macro-arguments")
 /// Create a specialization for custom holder types (silently ignores std::shared_ptr)
 #define PYBIND11_DECLARE_HOLDER_TYPE(type, holder_type, ...)                                      \
     PYBIND11_NAMESPACE_BEGIN(PYBIND11_NAMESPACE)                                                  \
+    PYBIND11_WARNING_DISABLE_GCC("-Wpedantic")                                                    \
+    PYBIND11_WARNING_DISABLE_CLANG("-Wgnu-zero-variadic-macro-arguments")                         \
     namespace detail {                                                                            \
     template <typename type>                                                                      \
     struct always_construct_holder<holder_type> : always_construct_holder<void, ##__VA_ARGS__> {  \
@@ -883,7 +882,6 @@ PYBIND11_WARNING_DISABLE_CLANG("-Wgnu-zero-variadic-macro-arguments")
         : public type_caster_holder<type, holder_type> {};                                        \
     }                                                                                             \
     PYBIND11_NAMESPACE_END(PYBIND11_NAMESPACE)
-PYBIND11_WARNING_POP
 
 // PYBIND11_DECLARE_HOLDER_TYPE holder types:
 template <typename base, typename holder>