blob: c4d89c104f12559794ef0098f1093edbdc644afa [file] [log] [blame]
Wenzel Jakob17cdb062016-03-10 13:24:10 +01001/*
Dean Moldovana0c1ccf2016-08-12 13:50:00 +02002 tests/test_issues.cpp -- collection of testcases for miscellaneous issues
Wenzel Jakob17cdb062016-03-10 13:24:10 +01003
Wenzel Jakob8cb6cb32016-04-17 20:21:41 +02004 Copyright (c) 2016 Wenzel Jakob <wenzel.jakob@epfl.ch>
Wenzel Jakob17cdb062016-03-10 13:24:10 +01005
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 Moldovana0c1ccf2016-08-12 13:50:00 +020010#include "pybind11_tests.h"
11#include "constructor_stats.h"
Wenzel Jakobf54ded72016-04-20 17:00:57 +020012#include <pybind11/stl.h>
Jason Rhinelander1b05ce52016-08-09 17:57:59 -040013#include <pybind11/operators.h>
Wenzel Jakob17cdb062016-03-10 13:24:10 +010014
Wenzel Jakobd2b628b2016-04-30 23:02:39 +020015PYBIND11_DECLARE_HOLDER_TYPE(T, std::shared_ptr<T>);
16
Jason Rhinelander3f589372016-08-07 13:05:26 -040017#define TRACKERS(CLASS) CLASS() { print_default_created(this); } ~CLASS() { print_destroyed(this); }
18struct NestABase { int value = -2; TRACKERS(NestABase) };
19struct NestA : NestABase { int value = 3; NestA& operator+=(int i) { value += i; return *this; } TRACKERS(NestA) };
20struct NestB { NestA a; int value = 4; NestB& operator-=(int i) { value -= i; return *this; } TRACKERS(NestB) };
21struct NestC { NestB b; int value = 5; NestC& operator*=(int i) { value *= i; return *this; } TRACKERS(NestC) };
22
Wenzel Jakob17cdb062016-03-10 13:24:10 +010023void init_issues(py::module &m) {
24 py::module m2 = m.def_submodule("issues");
25
Wenzel Jakob9059bd82016-05-01 10:39:45 +020026#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 Jakobbd57eb42016-05-01 14:42:20 +020032 m2.def("getstmt", []() -> NonConstructible * { return nullptr; },
Wenzel Jakob9059bd82016-05-01 10:39:45 +020033 py::return_value_policy::reference);
34#endif
35
Wenzel Jakob17cdb062016-03-10 13:24:10 +010036 // #137: const char* isn't handled properly
Dean Moldovan665e8802016-08-12 22:28:31 +020037 m2.def("print_cchar", [](const char *s) { return std::string(s); });
Wenzel Jakobd3349af2016-03-26 23:04:10 +010038
39 // #150: char bindings broken
Dean Moldovan665e8802016-08-12 22:28:31 +020040 m2.def("print_char", [](char c) { return std::string(1, c); });
Wenzel Jakobf5c154a2016-04-11 18:13:08 +020041
42 // #159: virtual function dispatch has problems with similar-named functions
Dean Moldovan665e8802016-08-12 22:28:31 +020043 struct Base { virtual std::string dispatch() const {
Wenzel Jakobd2b628b2016-04-30 23:02:39 +020044 /* for some reason MSVC2015 can't compile this if the function is pure virtual */
Dean Moldovan665e8802016-08-12 22:28:31 +020045 return {};
Wenzel Jakobd2b628b2016-04-30 23:02:39 +020046 }; };
Wenzel Jakobe7074972016-04-30 22:44:00 +020047
48 struct DispatchIssue : Base {
Dean Moldovan665e8802016-08-12 22:28:31 +020049 virtual std::string dispatch() const {
50 PYBIND11_OVERLOAD_PURE(std::string, Base, dispatch, /* no arguments */);
Wenzel Jakobe7074972016-04-30 22:44:00 +020051 }
52 };
53
Jason Rhinelander5fffe202016-09-06 12:17:06 -040054 py::class_<Base, DispatchIssue>(m2, "DispatchIssue")
Wenzel Jakobf54ded72016-04-20 17:00:57 +020055 .def(py::init<>())
Wenzel Jakobf5c154a2016-04-11 18:13:08 +020056 .def("dispatch", &Base::dispatch);
57
Dean Moldovan665e8802016-08-12 22:28:31 +020058 m2.def("dispatch_issue_go", [](const Base * b) { return b->dispatch(); });
Wenzel Jakobe7074972016-04-30 22:44:00 +020059
60 struct Placeholder { int i; Placeholder(int i) : i(i) { } };
Wenzel Jakobf54ded72016-04-20 17:00:57 +020061
62 py::class_<Placeholder>(m2, "Placeholder")
Wenzel Jakobdbe43ff2016-04-21 12:21:14 +020063 .def(py::init<int>())
Wenzel Jakobf54ded72016-04-20 17:00:57 +020064 .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 Jakob85f07e12016-09-04 23:00:49 +090067 m2.def("return_vec_of_reference_wrapper", [](std::reference_wrapper<Placeholder> p4) {
Wenzel Jakobf54ded72016-04-20 17:00:57 +020068 Placeholder *p1 = new Placeholder{1};
69 Placeholder *p2 = new Placeholder{2};
Wenzel Jakobdbe43ff2016-04-21 12:21:14 +020070 Placeholder *p3 = new Placeholder{3};
Wenzel Jakobf54ded72016-04-20 17:00:57 +020071 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 Jakobdbe43ff2016-04-21 12:21:14 +020075 v.push_back(p4);
Wenzel Jakobf54ded72016-04-20 17:00:57 +020076 return v;
77 });
Wenzel Jakob6ca6e822016-04-27 14:33:52 +020078
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 Jakobd2b628b2016-04-30 23:02:39 +020083
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
Jason Rhinelander6b52c832016-09-06 12:27:00 -040099 py::class_<ElementA, ElementBase, std::shared_ptr<ElementA>>(m2, "ElementA")
Wenzel Jakobd2b628b2016-04-30 23:02:39 +0200100 .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)
Wenzel Jakob85f07e12016-09-04 23:00:49 +0900106 .def("get", [](ElementList &el) {
Wenzel Jakobd2b628b2016-04-30 23:02:39 +0200107 py::list list;
108 for (auto &e : el.l)
109 list.append(py::cast(e));
110 return list;
111 });
Wenzel Jakobbd57eb42016-05-01 14:42:20 +0200112
113 // (no id): should not be able to pass 'None' to a reference argument
Dean Moldovan99dbdc12016-08-19 13:45:36 +0200114 m2.def("get_element", [](ElementA &el) { return el.value(); });
Wenzel Jakob3f200fa2016-05-17 15:35:29 +0200115
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 Jakob86d825f2016-05-26 13:19:27 +0200119
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() {}
Dean Moldovan81511be2016-09-07 00:50:10 +0200124 virtual void f() { py::print("A.f()"); }
Wenzel Jakob86d825f2016-05-26 13:19:27 +0200125 };
126
127 struct PyA : A {
Dean Moldovan81511be2016-09-07 00:50:10 +0200128 PyA() { py::print("PyA.PyA()"); }
Wenzel Jakob86d825f2016-05-26 13:19:27 +0200129
130 void f() override {
Dean Moldovan81511be2016-09-07 00:50:10 +0200131 py::print("PyA.f()");
Wenzel Jakob86d825f2016-05-26 13:19:27 +0200132 PYBIND11_OVERLOAD(void, A, f);
133 }
134 };
135
136 auto call_f = [](A *a) { a->f(); };
137
Wenzel Jakobfb0e2e52016-07-01 14:54:24 +0200138 pybind11::class_<A, std::unique_ptr<A>, PyA>(m2, "A")
139 .def(py::init<>())
140 .def("f", &A::f);
Wenzel Jakob86d825f2016-05-26 13:19:27 +0200141
Wenzel Jakobfb0e2e52016-07-01 14:54:24 +0200142 m2.def("call_f", call_f);
Wenzel Jakob38d8b8c2016-05-31 09:53:28 +0200143
144 try {
145 py::class_<Placeholder>(m2, "Placeholder");
146 throw std::logic_error("Expected an exception!");
Dean Moldovanf2b36c22016-06-01 23:03:10 +0200147 } catch (std::runtime_error &) {
Wenzel Jakob38d8b8c2016-05-31 09:53:28 +0200148 /* All good */
149 }
Jason Rhinelander4e45e182016-07-17 17:43:00 -0400150
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 Moldovan99dbdc12016-08-19 13:45:36 +0200163 .def("__str__", [](const StrIssue &si) { return "StrIssue[" + std::to_string(si.value()) + "]"; })
Jason Rhinelander4e45e182016-07-17 17:43:00 -0400164 ;
165
Jason Rhinelander1b05ce52016-08-09 17:57:59 -0400166 // Issue #328: first member in a class can't be used in operators
Jason Rhinelanderf2ecd892016-08-10 12:08:04 -0400167 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 Rhinelander1b05ce52016-08-09 17:57:59 -0400170 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 Moldovan665e8802016-08-12 22:28:31 +0200172 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 Jakobc84b37b2016-09-07 00:47:17 +0900175
176 // Issue 389: r_v_p::move should fall-through to copy on non-movable objects
177 class MoveIssue1 {
178 public:
179 MoveIssue1(int v) : v{v} {}
Dean Moldovan81511be2016-09-07 00:50:10 +0200180 MoveIssue1(const MoveIssue1 &c) { v = c.v; }
Wenzel Jakobc84b37b2016-09-07 00:47:17 +0900181 MoveIssue1(MoveIssue1 &&) = delete;
182 int v;
183 };
184 class MoveIssue2 {
185 public:
186 MoveIssue2(int v) : v{v} {}
187 MoveIssue2(MoveIssue2 &&) = default;
188 int v;
189 };
190 py::class_<MoveIssue1>(m2, "MoveIssue1").def(py::init<int>()).def_readwrite("value", &MoveIssue1::v);
191 py::class_<MoveIssue2>(m2, "MoveIssue2").def(py::init<int>()).def_readwrite("value", &MoveIssue2::v);
192 m2.def("get_moveissue1", [](int i) -> MoveIssue1 * { return new MoveIssue1(i); }, py::return_value_policy::move);
193 m2.def("get_moveissue2", [](int i) { return MoveIssue2(i); }, py::return_value_policy::move);
Jason Rhinelander56f71772016-09-07 13:32:49 -0400194
195 // Issue 392: overridding reference-returning functions
196 class OverrideTest {
197 public:
198 struct A { int value = 99; };
199 int v;
200 A a;
201 explicit OverrideTest(int v) : v{v} {}
202 virtual int int_value() { return v; }
203 virtual int &int_ref() { return v; }
204 virtual A A_value() { return a; }
205 virtual A &A_ref() { return a; }
206 };
207 class PyOverrideTest : public OverrideTest {
208 public:
209 using OverrideTest::OverrideTest;
210 int int_value() override { PYBIND11_OVERLOAD(int, OverrideTest, int_value); }
Jason Rhinelanderc03db9b2016-09-07 13:38:32 -0400211 // Not allowed (uncommenting should hit a static_assert failure): we can't get a reference
212 // to a python numeric value, since we only copy values in the numeric type caster:
213// int &int_ref() override { PYBIND11_OVERLOAD(int &, OverrideTest, int_ref); }
Jason Rhinelander56f71772016-09-07 13:32:49 -0400214 A A_value() override { PYBIND11_OVERLOAD(A, OverrideTest, A_value); }
215 A &A_ref() override { PYBIND11_OVERLOAD(A &, OverrideTest, A_ref); }
216 };
217 py::class_<OverrideTest::A>(m2, "OverrideTest_A")
218 .def_readwrite("value", &OverrideTest::A::value);
219 py::class_<OverrideTest, PyOverrideTest>(m2, "OverrideTest")
220 .def(py::init<int>())
221 .def("int_value", &OverrideTest::int_value)
Jason Rhinelanderc03db9b2016-09-07 13:38:32 -0400222// .def("int_ref", &OverrideTest::int_ref)
Jason Rhinelander56f71772016-09-07 13:32:49 -0400223 .def("A_value", &OverrideTest::A_value)
224 .def("A_ref", &OverrideTest::A_ref);
Jason Rhinelanderc03db9b2016-09-07 13:38:32 -0400225
Wenzel Jakobbd57eb42016-05-01 14:42:20 +0200226}
Jason Rhinelander52f4be82016-09-03 14:54:22 -0400227
228// MSVC workaround: trying to use a lambda here crashes MSCV
229test_initializer issues(&init_issues);