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

Use of non-array delete #16

Closed
wants to merge 3 commits into from
Closed

Use of non-array delete #16

wants to merge 3 commits into from

Commits on Nov 17, 2015

  1. Merge pull request #1 from Atlantis-PBEM/master 

    upstream changes
    @ennorehling
    ennorehling committed Nov 17, 2015
    Configuration menu
    Copy the full SHA
    0347b8c View commit details
    Browse the repository at this point in the history

  2. Use delete[] instead of non-array delete for scalars.

    @ennorehling
    ennorehling committed Nov 17, 2015
    Configuration menu
    Copy the full SHA
    5636796 View commit details
    Browse the repository at this point in the history

  3. GetNewUnit uses delete on this->ppUnits which is an array. It should 

    be deleted with operator delete[].
    @ennorehling
    ennorehling committed Nov 17, 2015
    Configuration menu
    Copy the full SHA
    a0d8ccc View commit details
    Browse the repository at this point in the history