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

#48 Release only initialized parsers #50

Merged

WojciechRynczuk merged 1 commit into v3_release from 048_Parser_memory_leak

Nov 25, 2018

Owner

WojciechRynczuk commented Nov 25, 2018

No description provided.


          #48 Release only initialized parsers

b9cac61

WojciechRynczuk self-assigned this

WojciechRynczuk added this to the v3.0.1 milestone

WojciechRynczuk merged commit 7aba320 into v3_release

endrjux reviewed

View reviewed changes

sources/parsers/inc/ExpressionContext.h

                           {}
                           /// Expression context destructor.
                           ~ExpressionContext()
                           {
-                              delete m_pExpression;
+                              if (m_pExpression != nullptr)

Collaborator

endrjux Nov 27, 2018

There is nothing bad with delete nullptr;. if is not needed.

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