You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@geode.apache.org by "ASF subversion and git services (JIRA)" <ji...@apache.org> on 2018/02/06 19:40:00 UTC
[jira] [Commented] (GEODE-4605) Replace void* with
std::shared_ptr in PdxSerializer/PdxWrapper
[ https://issues.apache.org/jira/browse/GEODE-4605?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16354444#comment-16354444 ]
ASF subversion and git services commented on GEODE-4605:
--------------------------------------------------------
Commit f4f4c7b34d194d5d4d68aad34cc2d6c95b3721c1 in geode-native's branch refs/heads/develop from Jacob Barrett
[ https://gitbox.apache.org/repos/asf?p=geode-native.git;h=f4f4c7b ]
GEODE-4605: Keep domain object via shared_ptr<void>. (#205)
- Removes UserDeallocator function.
> Replace void* with std::shared_ptr<void> in PdxSerializer/PdxWrapper
> --------------------------------------------------------------------
>
> Key: GEODE-4605
> URL: https://issues.apache.org/jira/browse/GEODE-4605
> Project: Geode
> Issue Type: Improvement
> Components: native client
> Reporter: Jacob S. Barrett
> Assignee: Jacob S. Barrett
> Priority: Major
> Labels: pull-request-available
> Time Spent: 50m
> Remaining Estimate: 0h
>
> With {{void*}} it is necessary to keep a deleter function around to free memory allocated to the type erased pointer. Switching to \{{std::shared_ptr<void>}} keeps the type erased pointer but preserves the correct delete behavior with the added bonus of non-ambiguous pointer ownership.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)