Skip to content
This repository was archived by the owner on Nov 1, 2023. It is now read-only.

Do type_casters for *Ref need to use loader_life_support::add_patient? #7

Closed
virtuald opened this issue Jan 3, 2020 · 1 comment
Closed

Do type_casters for *Ref need to use loader_life_support::add_patient? #7

virtuald opened this issue Jan 3, 2020 · 1 comment

Comments

@virtuald
Copy link
Member

virtuald commented Jan 3, 2020

Would be cleaner.
@virtuald virtuald mentioned this issue Feb 14, 2020
Closed
@virtuald
Copy link
Member Author

Looking closer, the circumstances for which the life support is needed is when the type caster data directly references the incoming data somehow. In the case of Twine/StringRef, it references the internal buffer from the python string object.

I don't believe the other types we cast have a similar situation.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@virtuald