Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Remove an unnecessary null pointer check #3

Open
elfring opened this issue Jun 13, 2016 · 2 comments
Open

Remove an unnecessary null pointer check #3

elfring opened this issue Jun 13, 2016 · 2 comments

Comments

@elfring
Copy link

elfring commented Jun 13, 2016

An extra null pointer check is not needed in the destructor for the class "TSmartMemory".
@RafaelCintron
Copy link
Member

Thank you for opening this issue @elfring . I agree the null pointer check is not needed. Feel free to send a pull request with the null pointer check removed.

@manish7294 manish7294 mentioned this issue Oct 5, 2018
Open
@Geraldjoshua
Copy link

Hi, can I request a pull with the null pointer check removed, for this issue, if it has not been fixed yet? Thank you!

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

No branches or pull requests
3 participants
@elfring @RafaelCintron @Geraldjoshua