restoring original behavior, only leaving the new lambda function in test_stl.cpp

include/pybind11/stl.h

@@ -144,11 +144,15 @@ template <typename Type, typename Value> struct list_caster {
     using value_conv = make_caster<Value>;
 
     bool load(handle src, bool convert) {
+#if 1
+        if (!isinstance<sequence>(src) || isinstance<str>(src))
+#else
         if (!isinstance<sequence>(src) || (
 #if PY_MAJOR_VERSION >= 3
             !isinstance<bytes>(src) &&
 #endif
             isinstance<str>(src)))
+#endif
             return false;
         auto s = reinterpret_borrow<sequence>(src);
         value.clear();

tests/test_stl.py

@@ -245,11 +245,11 @@ def test_function_with_string_and_vector_string_arg():
     assert m.func_with_string_or_vector_string_arg_overload("A") == 3
 
 
-def test_bytes_to_vector_uint8_t():
-    """Check if a bytes is implicitly converted to std::vector<uint8_t>, issue #1807"""
-    assert m.func_with_vector_uint8_t_arg([ord(c) for c in b'abc']) == 3
-    with pytest.raises(TypeError):
-        m.func_with_vector_uint8_t_arg('stringval')
+#def test_bytes_to_vector_uint8_t():
+#    """Check if a bytes is implicitly converted to std::vector<uint8_t>, issue #1807"""
+#    assert m.func_with_vector_uint8_t_arg([ord(c) for c in b'abc']) == 3
+#    with pytest.raises(TypeError):
+#        m.func_with_vector_uint8_t_arg('stringval')
 
 
 def test_stl_ownership():