Wenzel Jakob | 17cdb06 | 2016-03-10 13:24:10 +0100 | [diff] [blame] | 1 | /* |
Dean Moldovan | a0c1ccf | 2016-08-12 13:50:00 +0200 | [diff] [blame] | 2 | tests/test_issues.cpp -- collection of testcases for miscellaneous issues |
Wenzel Jakob | 17cdb06 | 2016-03-10 13:24:10 +0100 | [diff] [blame] | 3 | |
Wenzel Jakob | 8cb6cb3 | 2016-04-17 20:21:41 +0200 | [diff] [blame] | 4 | Copyright (c) 2016 Wenzel Jakob <wenzel.jakob@epfl.ch> |
Wenzel Jakob | 17cdb06 | 2016-03-10 13:24:10 +0100 | [diff] [blame] | 5 | |
| 6 | All rights reserved. Use of this source code is governed by a |
| 7 | BSD-style license that can be found in the LICENSE file. |
| 8 | */ |
| 9 | |
Dean Moldovan | a0c1ccf | 2016-08-12 13:50:00 +0200 | [diff] [blame] | 10 | #include "pybind11_tests.h" |
| 11 | #include "constructor_stats.h" |
Wenzel Jakob | f54ded7 | 2016-04-20 17:00:57 +0200 | [diff] [blame] | 12 | #include <pybind11/stl.h> |
Jason Rhinelander | 1b05ce5 | 2016-08-09 17:57:59 -0400 | [diff] [blame] | 13 | #include <pybind11/operators.h> |
Wenzel Jakob | 17cdb06 | 2016-03-10 13:24:10 +0100 | [diff] [blame] | 14 | |
Wenzel Jakob | d2b628b | 2016-04-30 23:02:39 +0200 | [diff] [blame] | 15 | PYBIND11_DECLARE_HOLDER_TYPE(T, std::shared_ptr<T>); |
| 16 | |
Jason Rhinelander | 3f58937 | 2016-08-07 13:05:26 -0400 | [diff] [blame] | 17 | #define TRACKERS(CLASS) CLASS() { print_default_created(this); } ~CLASS() { print_destroyed(this); } |
| 18 | struct NestABase { int value = -2; TRACKERS(NestABase) }; |
| 19 | struct NestA : NestABase { int value = 3; NestA& operator+=(int i) { value += i; return *this; } TRACKERS(NestA) }; |
| 20 | struct NestB { NestA a; int value = 4; NestB& operator-=(int i) { value -= i; return *this; } TRACKERS(NestB) }; |
| 21 | struct NestC { NestB b; int value = 5; NestC& operator*=(int i) { value *= i; return *this; } TRACKERS(NestC) }; |
| 22 | |
Wenzel Jakob | 17cdb06 | 2016-03-10 13:24:10 +0100 | [diff] [blame] | 23 | void init_issues(py::module &m) { |
| 24 | py::module m2 = m.def_submodule("issues"); |
| 25 | |
Wenzel Jakob | 9059bd8 | 2016-05-01 10:39:45 +0200 | [diff] [blame] | 26 | #if !defined(_MSC_VER) |
| 27 | // Visual Studio 2015 currently cannot compile this test |
| 28 | // (see the comment in type_caster_base::make_copy_constructor) |
| 29 | // #70 compilation issue if operator new is not public |
| 30 | class NonConstructible { private: void *operator new(size_t bytes) throw(); }; |
| 31 | py::class_<NonConstructible>(m, "Foo"); |
Wenzel Jakob | bd57eb4 | 2016-05-01 14:42:20 +0200 | [diff] [blame] | 32 | m2.def("getstmt", []() -> NonConstructible * { return nullptr; }, |
Wenzel Jakob | 9059bd8 | 2016-05-01 10:39:45 +0200 | [diff] [blame] | 33 | py::return_value_policy::reference); |
| 34 | #endif |
| 35 | |
Wenzel Jakob | 17cdb06 | 2016-03-10 13:24:10 +0100 | [diff] [blame] | 36 | // #137: const char* isn't handled properly |
Dean Moldovan | 665e880 | 2016-08-12 22:28:31 +0200 | [diff] [blame] | 37 | m2.def("print_cchar", [](const char *s) { return std::string(s); }); |
Wenzel Jakob | d3349af | 2016-03-26 23:04:10 +0100 | [diff] [blame] | 38 | |
| 39 | // #150: char bindings broken |
Dean Moldovan | 665e880 | 2016-08-12 22:28:31 +0200 | [diff] [blame] | 40 | m2.def("print_char", [](char c) { return std::string(1, c); }); |
Wenzel Jakob | f5c154a | 2016-04-11 18:13:08 +0200 | [diff] [blame] | 41 | |
| 42 | // #159: virtual function dispatch has problems with similar-named functions |
Dean Moldovan | 665e880 | 2016-08-12 22:28:31 +0200 | [diff] [blame] | 43 | struct Base { virtual std::string dispatch() const { |
Wenzel Jakob | d2b628b | 2016-04-30 23:02:39 +0200 | [diff] [blame] | 44 | /* for some reason MSVC2015 can't compile this if the function is pure virtual */ |
Dean Moldovan | 665e880 | 2016-08-12 22:28:31 +0200 | [diff] [blame] | 45 | return {}; |
Wenzel Jakob | d2b628b | 2016-04-30 23:02:39 +0200 | [diff] [blame] | 46 | }; }; |
Wenzel Jakob | e707497 | 2016-04-30 22:44:00 +0200 | [diff] [blame] | 47 | |
| 48 | struct DispatchIssue : Base { |
Dean Moldovan | 665e880 | 2016-08-12 22:28:31 +0200 | [diff] [blame] | 49 | virtual std::string dispatch() const { |
| 50 | PYBIND11_OVERLOAD_PURE(std::string, Base, dispatch, /* no arguments */); |
Wenzel Jakob | e707497 | 2016-04-30 22:44:00 +0200 | [diff] [blame] | 51 | } |
| 52 | }; |
| 53 | |
Wenzel Jakob | 86d825f | 2016-05-26 13:19:27 +0200 | [diff] [blame] | 54 | py::class_<Base, std::unique_ptr<Base>, DispatchIssue>(m2, "DispatchIssue") |
Wenzel Jakob | f54ded7 | 2016-04-20 17:00:57 +0200 | [diff] [blame] | 55 | .def(py::init<>()) |
Wenzel Jakob | f5c154a | 2016-04-11 18:13:08 +0200 | [diff] [blame] | 56 | .def("dispatch", &Base::dispatch); |
| 57 | |
Dean Moldovan | 665e880 | 2016-08-12 22:28:31 +0200 | [diff] [blame] | 58 | m2.def("dispatch_issue_go", [](const Base * b) { return b->dispatch(); }); |
Wenzel Jakob | e707497 | 2016-04-30 22:44:00 +0200 | [diff] [blame] | 59 | |
| 60 | struct Placeholder { int i; Placeholder(int i) : i(i) { } }; |
Wenzel Jakob | f54ded7 | 2016-04-20 17:00:57 +0200 | [diff] [blame] | 61 | |
| 62 | py::class_<Placeholder>(m2, "Placeholder") |
Wenzel Jakob | dbe43ff | 2016-04-21 12:21:14 +0200 | [diff] [blame] | 63 | .def(py::init<int>()) |
Wenzel Jakob | f54ded7 | 2016-04-20 17:00:57 +0200 | [diff] [blame] | 64 | .def("__repr__", [](const Placeholder &p) { return "Placeholder[" + std::to_string(p.i) + "]"; }); |
| 65 | |
| 66 | // #171: Can't return reference wrappers (or STL datastructures containing them) |
Wenzel Jakob | dbe43ff | 2016-04-21 12:21:14 +0200 | [diff] [blame] | 67 | m2.def("return_vec_of_reference_wrapper", [](std::reference_wrapper<Placeholder> p4){ |
Wenzel Jakob | f54ded7 | 2016-04-20 17:00:57 +0200 | [diff] [blame] | 68 | Placeholder *p1 = new Placeholder{1}; |
| 69 | Placeholder *p2 = new Placeholder{2}; |
Wenzel Jakob | dbe43ff | 2016-04-21 12:21:14 +0200 | [diff] [blame] | 70 | Placeholder *p3 = new Placeholder{3}; |
Wenzel Jakob | f54ded7 | 2016-04-20 17:00:57 +0200 | [diff] [blame] | 71 | std::vector<std::reference_wrapper<Placeholder>> v; |
| 72 | v.push_back(std::ref(*p1)); |
| 73 | v.push_back(std::ref(*p2)); |
| 74 | v.push_back(std::ref(*p3)); |
Wenzel Jakob | dbe43ff | 2016-04-21 12:21:14 +0200 | [diff] [blame] | 75 | v.push_back(p4); |
Wenzel Jakob | f54ded7 | 2016-04-20 17:00:57 +0200 | [diff] [blame] | 76 | return v; |
| 77 | }); |
Wenzel Jakob | 6ca6e82 | 2016-04-27 14:33:52 +0200 | [diff] [blame] | 78 | |
| 79 | // #181: iterator passthrough did not compile |
| 80 | m2.def("iterator_passthrough", [](py::iterator s) -> py::iterator { |
| 81 | return py::make_iterator(std::begin(s), std::end(s)); |
| 82 | }); |
Wenzel Jakob | d2b628b | 2016-04-30 23:02:39 +0200 | [diff] [blame] | 83 | |
| 84 | // #187: issue involving std::shared_ptr<> return value policy & garbage collection |
| 85 | struct ElementBase { virtual void foo() { } /* Force creation of virtual table */ }; |
| 86 | struct ElementA : ElementBase { |
| 87 | ElementA(int v) : v(v) { } |
| 88 | int value() { return v; } |
| 89 | int v; |
| 90 | }; |
| 91 | |
| 92 | struct ElementList { |
| 93 | void add(std::shared_ptr<ElementBase> e) { l.push_back(e); } |
| 94 | std::vector<std::shared_ptr<ElementBase>> l; |
| 95 | }; |
| 96 | |
| 97 | py::class_<ElementBase, std::shared_ptr<ElementBase>> (m2, "ElementBase"); |
| 98 | |
| 99 | py::class_<ElementA, std::shared_ptr<ElementA>>(m2, "ElementA", py::base<ElementBase>()) |
| 100 | .def(py::init<int>()) |
| 101 | .def("value", &ElementA::value); |
| 102 | |
| 103 | py::class_<ElementList, std::shared_ptr<ElementList>>(m2, "ElementList") |
| 104 | .def(py::init<>()) |
| 105 | .def("add", &ElementList::add) |
| 106 | .def("get", [](ElementList &el){ |
| 107 | py::list list; |
| 108 | for (auto &e : el.l) |
| 109 | list.append(py::cast(e)); |
| 110 | return list; |
| 111 | }); |
Wenzel Jakob | bd57eb4 | 2016-05-01 14:42:20 +0200 | [diff] [blame] | 112 | |
| 113 | // (no id): should not be able to pass 'None' to a reference argument |
Dean Moldovan | 99dbdc1 | 2016-08-19 13:45:36 +0200 | [diff] [blame^] | 114 | m2.def("get_element", [](ElementA &el) { return el.value(); }); |
Wenzel Jakob | 3f200fa | 2016-05-17 15:35:29 +0200 | [diff] [blame] | 115 | |
| 116 | // (no id): don't cast doubles to ints |
| 117 | m2.def("expect_float", [](float f) { return f; }); |
| 118 | m2.def("expect_int", [](int i) { return i; }); |
Wenzel Jakob | 86d825f | 2016-05-26 13:19:27 +0200 | [diff] [blame] | 119 | |
| 120 | // (no id): don't invoke Python dispatch code when instantiating C++ |
| 121 | // classes that were not extended on the Python side |
| 122 | struct A { |
| 123 | virtual ~A() {} |
| 124 | virtual void f() { std::cout << "A.f()" << std::endl; } |
| 125 | }; |
| 126 | |
| 127 | struct PyA : A { |
| 128 | PyA() { std::cout << "PyA.PyA()" << std::endl; } |
| 129 | |
| 130 | void f() override { |
| 131 | std::cout << "PyA.f()" << std::endl; |
| 132 | PYBIND11_OVERLOAD(void, A, f); |
| 133 | } |
| 134 | }; |
| 135 | |
| 136 | auto call_f = [](A *a) { a->f(); }; |
| 137 | |
Wenzel Jakob | fb0e2e5 | 2016-07-01 14:54:24 +0200 | [diff] [blame] | 138 | pybind11::class_<A, std::unique_ptr<A>, PyA>(m2, "A") |
| 139 | .def(py::init<>()) |
| 140 | .def("f", &A::f); |
Wenzel Jakob | 86d825f | 2016-05-26 13:19:27 +0200 | [diff] [blame] | 141 | |
Wenzel Jakob | fb0e2e5 | 2016-07-01 14:54:24 +0200 | [diff] [blame] | 142 | m2.def("call_f", call_f); |
Wenzel Jakob | 38d8b8c | 2016-05-31 09:53:28 +0200 | [diff] [blame] | 143 | |
| 144 | try { |
| 145 | py::class_<Placeholder>(m2, "Placeholder"); |
| 146 | throw std::logic_error("Expected an exception!"); |
Dean Moldovan | f2b36c2 | 2016-06-01 23:03:10 +0200 | [diff] [blame] | 147 | } catch (std::runtime_error &) { |
Wenzel Jakob | 38d8b8c | 2016-05-31 09:53:28 +0200 | [diff] [blame] | 148 | /* All good */ |
| 149 | } |
Jason Rhinelander | 4e45e18 | 2016-07-17 17:43:00 -0400 | [diff] [blame] | 150 | |
| 151 | // Issue #283: __str__ called on uninitialized instance when constructor arguments invalid |
| 152 | class StrIssue { |
| 153 | public: |
| 154 | StrIssue(int i) : val{i} {} |
| 155 | StrIssue() : StrIssue(-1) {} |
| 156 | int value() const { return val; } |
| 157 | private: |
| 158 | int val; |
| 159 | }; |
| 160 | py::class_<StrIssue> si(m2, "StrIssue"); |
| 161 | si .def(py::init<int>()) |
| 162 | .def(py::init<>()) |
Dean Moldovan | 99dbdc1 | 2016-08-19 13:45:36 +0200 | [diff] [blame^] | 163 | .def("__str__", [](const StrIssue &si) { return "StrIssue[" + std::to_string(si.value()) + "]"; }) |
Jason Rhinelander | 4e45e18 | 2016-07-17 17:43:00 -0400 | [diff] [blame] | 164 | ; |
| 165 | |
Jason Rhinelander | 1b05ce5 | 2016-08-09 17:57:59 -0400 | [diff] [blame] | 166 | // Issue #328: first member in a class can't be used in operators |
Jason Rhinelander | f2ecd89 | 2016-08-10 12:08:04 -0400 | [diff] [blame] | 167 | py::class_<NestABase>(m2, "NestABase").def(py::init<>()).def_readwrite("value", &NestABase::value); |
| 168 | py::class_<NestA>(m2, "NestA").def(py::init<>()).def(py::self += int()) |
| 169 | .def("as_base", [](NestA &a) -> NestABase& { return (NestABase&) a; }, py::return_value_policy::reference_internal); |
Jason Rhinelander | 1b05ce5 | 2016-08-09 17:57:59 -0400 | [diff] [blame] | 170 | py::class_<NestB>(m2, "NestB").def(py::init<>()).def(py::self -= int()).def_readwrite("a", &NestB::a); |
| 171 | py::class_<NestC>(m2, "NestC").def(py::init<>()).def(py::self *= int()).def_readwrite("b", &NestC::b); |
Dean Moldovan | 665e880 | 2016-08-12 22:28:31 +0200 | [diff] [blame] | 172 | m2.def("get_NestA", [](const NestA &a) { return a.value; }); |
| 173 | m2.def("get_NestB", [](const NestB &b) { return b.value; }); |
| 174 | m2.def("get_NestC", [](const NestC &c) { return c.value; }); |
Wenzel Jakob | bd57eb4 | 2016-05-01 14:42:20 +0200 | [diff] [blame] | 175 | } |