New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Store shared_ptr for Attribute in ArraySchema #2887
Conversation
This pull request has been linked to Shortcut Story #14362: Store shared_ptr for Attribute in ArraySchema. |
118abe2
to
ecd048b
Compare
16e7dd7
to
1cff864
Compare
1cff864
to
5d3e022
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One memory leak remains. Smaller issues are a bit of excess code and redundant temporaries.
Close to ready, but not there yet.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Almost there.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This is the first step in smart pointer conversion for memory management. The ArraySchema's Attribute pointer has been changed to a
tdb_shared_ptr
. [ch14362]TYPE: IMPROVEMENT
DESC: ArraySchema's Attribute smart pointer conversion