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

Conversation

@Twon
Copy link
Collaborator

@Twon Twon commented Oct 12, 2022

No description provided.

@Twon Twon requested a review from jbcoe October 12, 2022 10:44
public:
template <class T>
void operator()(T* t) const noexcept {
void operator()(T*& t) const noexcept {
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is this change needed?

void operator()(T*& t) const noexcept {
if (t != nullptr) {
t->destroy();
t = nullptr;
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this needed here?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants