Skip to content
Projects
Groups
Snippets
Help
This project
Loading...
Sign in / Register
Toggle navigation
P
pybind11
Overview
Overview
Details
Activity
Cycle Analytics
Repository
Repository
Files
Commits
Branches
Tags
Contributors
Graph
Compare
Charts
Issues
0
Issues
0
List
Board
Labels
Milestones
Merge Requests
0
Merge Requests
0
CI / CD
CI / CD
Pipelines
Jobs
Schedules
Charts
Wiki
Wiki
Snippets
Snippets
Members
Collapse sidebar
Close sidebar
Activity
Graph
Charts
Create a new issue
Jobs
Commits
Issue Boards
Open sidebar
open
pybind11
Commits
56df3c46
Unverified
Commit
56df3c46
authored
Aug 21, 2020
by
Henry Schreiner
Committed by
GitHub
Aug 21, 2020
Browse files
Options
Browse Files
Download
Email Patches
Plain Diff
fix: a couple more places where pybind11 is missing 11 (#2421)
parent
d4d7ef5d
Hide whitespace changes
Inline
Side-by-side
Showing
2 changed files
with
2 additions
and
2 deletions
+2
-2
include/pybind11/cast.h
+1
-1
tools/pybind11NewTools.cmake
+1
-1
No files found.
include/pybind11/cast.h
View file @
56df3c46
...
...
@@ -1767,7 +1767,7 @@ detail::enable_if_t<!detail::move_never<T>::value, T> move(object &&obj) {
return
ret
;
}
// Calling cast() on an rvalue calls pybind::cast with the object rvalue, which does:
// Calling cast() on an rvalue calls pybind
11
::cast with the object rvalue, which does:
// - If we have to move (because T has no copy constructor), do it. This will fail if the moved
// object has multiple references, but trying to copy will fail to compile.
// - If both movable and copyable, check ref count: if 1, move; otherwise copy
...
...
tools/pybind11NewTools.cmake
View file @
56df3c46
...
...
@@ -79,7 +79,7 @@ execute_process(COMMAND ${_Python}::Python -c "import sys; print(hasattr(sys, 'g
# https://stackoverflow.com/questions/39161202/how-to-work-around-missing-pymodule-create2-in-amd64-win-python35-d-lib
if
(
PYTHON_IS_DEBUG
)
set_property
(
TARGET pybind::pybind11
TARGET pybind
11
::pybind11
APPEND
PROPERTY INTERFACE_COMPILE_DEFINITIONS Py_DEBUG
)
endif
()
...
...
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment