fix: extra parens matter?

Author: henryiii

include/pybind11/pybind11.h

@@ -1976,15 +1976,15 @@ struct iterator_access {
     }
 };
 
-template <typename Iterator, typename ResultType = decltype((*std::declval<Iterator>()).first) >
+template <typename Iterator, typename ResultType = decltype(((*std::declval<Iterator>()).first)) >
 struct iterator_key_access {
     using result_type = ResultType;
     result_type operator()(Iterator &it) const {
         return (*it).first;
     }
 };
 
-template <typename Iterator, typename ResultType = decltype((*std::declval<Iterator>()).second)>
+template <typename Iterator, typename ResultType = decltype(((*std::declval<Iterator>()).second))>
 struct iterator_value_access {
     using result_type = ResultType;
     result_type operator()(Iterator &it) const {

tests/test_sequences_and_iterators.py

@@ -23,7 +23,7 @@ def test_generalized_iterators():
 
     # keep_alive can't be attached
     with pytest.raises(RuntimeError):
-        m.IntPairs([(1, 2), (3, 4), (0, 5)]).make_iterator_keep_alive()
+        list(m.IntPairs([(1, 2), (3, 4), (0, 5)]).make_iterator_keep_alive())
 
     assert list(m.IntPairs([(1, 2), (3, 4), (0, 5)]).nonzero_keys()) == [1, 3]
     assert list(m.IntPairs([(1, 2), (2, 0), (0, 3), (4, 5)]).nonzero_keys()) == [1]