/* tests/test_issues.cpp -- collection of testcases for miscellaneous issues Copyright (c) 2016 Wenzel Jakob All rights reserved. Use of this source code is governed by a BSD-style license that can be found in the LICENSE file. */ #include "pybind11_tests.h" #include "constructor_stats.h" #include #include #include #define TRACKERS(CLASS) CLASS() { print_default_created(this); } ~CLASS() { print_destroyed(this); } struct NestABase { int value = -2; TRACKERS(NestABase) }; struct NestA : NestABase { int value = 3; NestA& operator+=(int i) { value += i; return *this; } TRACKERS(NestA) }; struct NestB { NestA a; int value = 4; NestB& operator-=(int i) { value -= i; return *this; } TRACKERS(NestB) }; struct NestC { NestB b; int value = 5; NestC& operator*=(int i) { value *= i; return *this; } TRACKERS(NestC) }; /// #393 class OpTest1 {}; class OpTest2 {}; OpTest1 operator+(const OpTest1 &, const OpTest1 &) { py::print("Add OpTest1 with OpTest1"); return OpTest1(); } OpTest2 operator+(const OpTest2 &, const OpTest2 &) { py::print("Add OpTest2 with OpTest2"); return OpTest2(); } OpTest2 operator+(const OpTest2 &, const OpTest1 &) { py::print("Add OpTest2 with OpTest1"); return OpTest2(); } // #461 class Dupe1 { public: Dupe1(int v) : v_{v} {} int get_value() const { return v_; } private: int v_; }; class Dupe2 {}; class Dupe3 {}; class DupeException : public std::runtime_error {}; // #478 template class custom_unique_ptr { public: custom_unique_ptr() { print_default_created(this); } custom_unique_ptr(T *ptr) : _ptr{ptr} { print_created(this, ptr); } custom_unique_ptr(custom_unique_ptr &&move) : _ptr{move._ptr} { move._ptr = nullptr; print_move_created(this); } custom_unique_ptr &operator=(custom_unique_ptr &&move) { print_move_assigned(this); if (_ptr) destruct_ptr(); _ptr = move._ptr; move._ptr = nullptr; return *this; } custom_unique_ptr(const custom_unique_ptr &) = delete; void operator=(const custom_unique_ptr ©) = delete; ~custom_unique_ptr() { print_destroyed(this); if (_ptr) destruct_ptr(); } private: T *_ptr = nullptr; void destruct_ptr() { delete _ptr; } }; PYBIND11_DECLARE_HOLDER_TYPE(T, custom_unique_ptr); /// Issue #528: templated constructor struct TplConstrClass { template TplConstrClass(const T &arg) : str{arg} {} std::string str; bool operator==(const TplConstrClass &t) const { return t.str == str; } }; namespace std { template <> struct hash { size_t operator()(const TplConstrClass &t) const { return std::hash()(t.str); } }; } void init_issues(py::module &m) { py::module m2 = m.def_submodule("issues"); #if !defined(_MSC_VER) // Visual Studio 2015 currently cannot compile this test // (see the comment in type_caster_base::make_copy_constructor) // #70 compilation issue if operator new is not public class NonConstructible { private: void *operator new(size_t bytes) throw(); }; py::class_(m, "Foo"); m2.def("getstmt", []() -> NonConstructible * { return nullptr; }, py::return_value_policy::reference); #endif // #137: const char* isn't handled properly m2.def("print_cchar", [](const char *s) { return std::string(s); }); // #150: char bindings broken m2.def("print_char", [](char c) { return std::string(1, c); }); // #159: virtual function dispatch has problems with similar-named functions struct Base { virtual std::string dispatch() const { /* for some reason MSVC2015 can't compile this if the function is pure virtual */ return {}; }; }; struct DispatchIssue : Base { virtual std::string dispatch() const { PYBIND11_OVERLOAD_PURE(std::string, Base, dispatch, /* no arguments */); } }; py::class_(m2, "DispatchIssue") .def(py::init<>()) .def("dispatch", &Base::dispatch); m2.def("dispatch_issue_go", [](const Base * b) { return b->dispatch(); }); struct Placeholder { int i; Placeholder(int i) : i(i) { } }; py::class_(m2, "Placeholder") .def(py::init()) .def("__repr__", [](const Placeholder &p) { return "Placeholder[" + std::to_string(p.i) + "]"; }); // #171: Can't return reference wrappers (or STL datastructures containing them) m2.def("return_vec_of_reference_wrapper", [](std::reference_wrapper p4) { Placeholder *p1 = new Placeholder{1}; Placeholder *p2 = new Placeholder{2}; Placeholder *p3 = new Placeholder{3}; std::vector> v; v.push_back(std::ref(*p1)); v.push_back(std::ref(*p2)); v.push_back(std::ref(*p3)); v.push_back(p4); return v; }); // #181: iterator passthrough did not compile m2.def("iterator_passthrough", [](py::iterator s) -> py::iterator { return py::make_iterator(std::begin(s), std::end(s)); }); // #187: issue involving std::shared_ptr<> return value policy & garbage collection struct ElementBase { virtual void foo() { } /* Force creation of virtual table */ }; struct ElementA : ElementBase { ElementA(int v) : v(v) { } int value() { return v; } int v; }; struct ElementList { void add(std::shared_ptr e) { l.push_back(e); } std::vector> l; }; py::class_> (m2, "ElementBase"); py::class_>(m2, "ElementA") .def(py::init()) .def("value", &ElementA::value); py::class_>(m2, "ElementList") .def(py::init<>()) .def("add", &ElementList::add) .def("get", [](ElementList &el) { py::list list; for (auto &e : el.l) list.append(py::cast(e)); return list; }); // (no id): should not be able to pass 'None' to a reference argument m2.def("get_element", [](ElementA &el) { return el.value(); }); // (no id): don't cast doubles to ints m2.def("expect_float", [](float f) { return f; }); m2.def("expect_int", [](int i) { return i; }); try { py::class_(m2, "Placeholder"); throw std::logic_error("Expected an exception!"); } catch (std::runtime_error &) { /* All good */ } // Issue #283: __str__ called on uninitialized instance when constructor arguments invalid class StrIssue { public: StrIssue(int i) : val{i} {} StrIssue() : StrIssue(-1) {} int value() const { return val; } private: int val; }; py::class_ si(m2, "StrIssue"); si .def(py::init()) .def(py::init<>()) .def("__str__", [](const StrIssue &si) { return "StrIssue[" + std::to_string(si.value()) + "]"; }) ; // Issue #328: first member in a class can't be used in operators py::class_(m2, "NestABase").def(py::init<>()).def_readwrite("value", &NestABase::value); py::class_(m2, "NestA").def(py::init<>()).def(py::self += int()) .def("as_base", [](NestA &a) -> NestABase& { return (NestABase&) a; }, py::return_value_policy::reference_internal); py::class_(m2, "NestB").def(py::init<>()).def(py::self -= int()).def_readwrite("a", &NestB::a); py::class_(m2, "NestC").def(py::init<>()).def(py::self *= int()).def_readwrite("b", &NestC::b); m2.def("get_NestA", [](const NestA &a) { return a.value; }); m2.def("get_NestB", [](const NestB &b) { return b.value; }); m2.def("get_NestC", [](const NestC &c) { return c.value; }); // Issue 389: r_v_p::move should fall-through to copy on non-movable objects class MoveIssue1 { public: MoveIssue1(int v) : v{v} {} MoveIssue1(const MoveIssue1 &c) { v = c.v; } MoveIssue1(MoveIssue1 &&) = delete; int v; }; class MoveIssue2 { public: MoveIssue2(int v) : v{v} {} MoveIssue2(MoveIssue2 &&) = default; int v; }; py::class_(m2, "MoveIssue1").def(py::init()).def_readwrite("value", &MoveIssue1::v); py::class_(m2, "MoveIssue2").def(py::init()).def_readwrite("value", &MoveIssue2::v); m2.def("get_moveissue1", [](int i) -> MoveIssue1 * { return new MoveIssue1(i); }, py::return_value_policy::move); m2.def("get_moveissue2", [](int i) { return MoveIssue2(i); }, py::return_value_policy::move); // Issues 392/397: overridding reference-returning functions class OverrideTest { public: struct A { std::string value = "hi"; }; std::string v; A a; explicit OverrideTest(const std::string &v) : v{v} {} virtual std::string str_value() { return v; } virtual std::string &str_ref() { return v; } virtual A A_value() { return a; } virtual A &A_ref() { return a; } }; class PyOverrideTest : public OverrideTest { public: using OverrideTest::OverrideTest; std::string str_value() override { PYBIND11_OVERLOAD(std::string, OverrideTest, str_value); } // Not allowed (uncommenting should hit a static_assert failure): we can't get a reference // to a python numeric value, since we only copy values in the numeric type caster: // std::string &str_ref() override { PYBIND11_OVERLOAD(std::string &, OverrideTest, str_ref); } // But we can work around it like this: private: std::string _tmp; std::string str_ref_helper() { PYBIND11_OVERLOAD(std::string, OverrideTest, str_ref); } public: std::string &str_ref() override { return _tmp = str_ref_helper(); } A A_value() override { PYBIND11_OVERLOAD(A, OverrideTest, A_value); } A &A_ref() override { PYBIND11_OVERLOAD(A &, OverrideTest, A_ref); } }; py::class_(m2, "OverrideTest_A") .def_readwrite("value", &OverrideTest::A::value); py::class_(m2, "OverrideTest") .def(py::init()) .def("str_value", &OverrideTest::str_value) // .def("str_ref", &OverrideTest::str_ref) .def("A_value", &OverrideTest::A_value) .def("A_ref", &OverrideTest::A_ref); /// Issue 393: need to return NotSupported to ensure correct arithmetic operator behavior py::class_(m2, "OpTest1") .def(py::init<>()) .def(py::self + py::self); py::class_(m2, "OpTest2") .def(py::init<>()) .def(py::self + py::self) .def("__add__", [](const OpTest2& c2, const OpTest1& c1) { return c2 + c1; }) .def("__radd__", [](const OpTest2& c2, const OpTest1& c1) { return c2 + c1; }); // Issue 388: Can't make iterators via make_iterator() with different r/v policies static std::vector list = { 1, 2, 3 }; m2.def("make_iterator_1", []() { return py::make_iterator(list); }); m2.def("make_iterator_2", []() { return py::make_iterator(list); }); static std::vector nothrows; // Issue 461: registering two things with the same name: py::class_(m2, "Dupe1") .def("get_value", &Dupe1::get_value) ; m2.def("dupe1_factory", [](int v) { return new Dupe1(v); }); py::class_(m2, "Dupe2"); py::exception(m2, "DupeException"); try { m2.def("Dupe1", [](int v) { return new Dupe1(v); }); nothrows.emplace_back("Dupe1"); } catch (std::runtime_error &) {} try { py::class_(m2, "dupe1_factory"); nothrows.emplace_back("dupe1_factory"); } catch (std::runtime_error &) {} try { py::exception(m2, "Dupe2"); nothrows.emplace_back("Dupe2"); } catch (std::runtime_error &) {} try { m2.def("DupeException", []() { return 30; }); nothrows.emplace_back("DupeException1"); } catch (std::runtime_error &) {} try { py::class_(m2, "DupeException"); nothrows.emplace_back("DupeException2"); } catch (std::runtime_error &) {} m2.def("dupe_exception_failures", []() { py::list l; for (auto &e : nothrows) l.append(py::cast(e)); return l; }); /// Issue #471: shared pointer instance not dellocated class SharedChild : public std::enable_shared_from_this { public: SharedChild() { print_created(this); } ~SharedChild() { print_destroyed(this); } }; class SharedParent { public: SharedParent() : child(std::make_shared()) { } const SharedChild &get_child() const { return *child; } private: std::shared_ptr child; }; py::class_>(m, "SharedChild"); py::class_>(m, "SharedParent") .def(py::init<>()) .def("get_child", &SharedParent::get_child, py::return_value_policy::reference); /// Issue/PR #478: unique ptrs constructed and freed without destruction class SpecialHolderObj { public: int val = 0; SpecialHolderObj *ch = nullptr; SpecialHolderObj(int v, bool make_child = true) : val{v}, ch{make_child ? new SpecialHolderObj(val+1, false) : nullptr} { print_created(this, val); } ~SpecialHolderObj() { delete ch; print_destroyed(this); } SpecialHolderObj *child() { return ch; } }; py::class_>(m, "SpecialHolderObj") .def(py::init()) .def("child", &SpecialHolderObj::child, pybind11::return_value_policy::reference_internal) .def_readwrite("val", &SpecialHolderObj::val) .def_static("holder_cstats", &ConstructorStats::get>, py::return_value_policy::reference); /// Issue #484: number conversion generates unhandled exceptions m2.def("test_complex", [](float x) { py::print("{}"_s.format(x)); }); m2.def("test_complex", [](std::complex x) { py::print("({}, {})"_s.format(x.real(), x.imag())); }); /// Issue #511: problem with inheritance + overwritten def_static struct MyBase { static std::unique_ptr make() { return std::unique_ptr(new MyBase()); } }; struct MyDerived : MyBase { static std::unique_ptr make() { return std::unique_ptr(new MyDerived()); } }; py::class_(m2, "MyBase") .def_static("make", &MyBase::make); py::class_(m2, "MyDerived") .def_static("make", &MyDerived::make) .def_static("make2", &MyDerived::make); py::dict d; std::string bar = "bar"; d["str"] = bar; d["num"] = 3.7; /// Issue #528: templated constructor m2.def("tpl_constr_vector", [](std::vector &) {}); m2.def("tpl_constr_map", [](std::unordered_map &) {}); m2.def("tpl_constr_set", [](std::unordered_set &) {}); #if defined(PYBIND11_HAS_OPTIONAL) m2.def("tpl_constr_optional", [](std::optional &) {}); #elif defined(PYBIND11_HAS_EXP_OPTIONAL) m2.def("tpl_constr_optional", [](std::experimental::optional &) {}); #endif } // MSVC workaround: trying to use a lambda here crashes MSVC test_initializer issues(&init_issues);