Zengtudor/pybind11
1
0
Fork 0
mirror of https://github.com/pybind/pybind11.git synced 2024-11-11 08:03:55 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #304 from trygvis/pybind-303

Browse Source 
Changes accessor::operator=() to throw error_already_set() instead of using pybind11_fail().
This commit is contained in:
Wenzel Jakob 2016-08-01 23:10:00 +02:00 committed by GitHub
commit f5d00105ce
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
include/pybind11/pytypes.h
View File

@ -117,10 +117,10 @@ public:
void operator=(const handle &value) { void operator=(const handle &value) {
if (attr) { if (attr) {
if (PyObject_SetAttr(obj.ptr(), key.ptr(), value.ptr()) == -1) if (PyObject_SetAttr(obj.ptr(), key.ptr(), value.ptr()) == -1)
pybind11_fail("Unable to set object attribute"); throw error_already_set();
} else { } else {
if (PyObject_SetItem(obj.ptr(), key.ptr(), value.ptr()) == -1) if (PyObject_SetItem(obj.ptr(), key.ptr(), value.ptr()) == -1)
pybind11_fail("Unable to set object item"); throw error_already_set();
} }
} }