Skip to content
Snippets Groups Projects
Commit 05ea2eb2 authored by Owen Arnold's avatar Owen Arnold
Browse files

refs #24322. Comments no longer apply

parent 1247ce7c
No related merge requests found
......@@ -25,13 +25,4 @@ void export_ComponentInfoPythonIterator() {
.def("next", &ComponentInfoPythonIterator::next)
#endif
;
/*
Return value policy for next is to copy the const reference. Copy by value is
essential for python 2.0 compatibility because items (DetectorInfoItem) will
outlive their iterators if declared as part of for loops. There is no good
way to deal with this other than to force a copy so that internals of the
item are not also corrupted. Future developers may wish to choose a separte
policy for python 3.0 where this is not a concern, and const ref returns
would be faster.
*/
}
......@@ -25,13 +25,4 @@ void export_DetectorInfoPythonIterator() {
.def("next", &DetectorInfoPythonIterator::next)
#endif
;
/*
Return value policy for next is to copy the const reference. Copy by value is
essential for python 2.0 compatibility because items (DetectorInfoItem) will
outlive their iterators if declared as part of for loops. There is no good
way to deal with this other than to force a copy so that internals of the
item are not also corrupted. Future developers may wish to choose a separte
policy for python 3.0 where this is not a concern, and const ref returns
would be faster.
*/
}
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment