Skip to content

Conversation

@j-jorge
Copy link
Contributor

@j-jorge j-jorge commented Oct 15, 2025

_animatedScrollAction is retained but was not released on destruction of the ScrollView.

Describe your changes

Issue ticket number and link

Checklist before requesting a review

For each PR

  • Add Copyright if it missed:
    - "Copyright (c) 2019-present Axmol Engine contributors (see AUTHORS.md)."

  • I have performed a self-review of my code.

    Optional:

    • I have checked readme and add important infos to this PR.
    • I have added/adapted some tests too.

For core/new feature PR

  • I have checked readme and add important infos to this PR.
  • I have added thorough tests.

Axmol 3.x ------------------------------------------------------------

For each 3.x PR

  • Check the '#include "axmol.h"' and replace it with the needed headers.

@halx99 halx99 changed the title Fix memory leak in ScrollView. Fix memory leak in extension::GUI ScrollView. Oct 16, 2025
@halx99
Copy link
Collaborator

halx99 commented Oct 16, 2025

Maybe shoud target branch release/2.x

@j-jorge j-jorge changed the base branch from dev to release/2.x October 16, 2025 04:33
_animatedScrollAction is retained but was not released on destruction
of the ScrollView.
@halx99 halx99 added this to the 2.9.1 milestone Oct 16, 2025
@halx99 halx99 merged commit 6ae3e2a into axmolengine:release/2.x Oct 16, 2025
27 of 29 checks passed
tkzcfc pushed a commit to tkzcfc/axmol that referenced this pull request Oct 17, 2025
* release/2.x:
  Fix memory leak in extension::GUI ScrollView. (axmolengine#2821)
  Add new test scroll view v2 (axmolengine#2820)
  AutoPolygon::findFirstNoneTransparentPixel() detected Y pos is 1 pixe… (axmolengine#2810)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants