Skip to content

fix(ios): memory leak cause by style prop changes - #4021

Merged
mfazekas merged 1 commit into
rnmapbox:mainfrom
Gnative:fix-memory-leak-with-style-url
Oct 10, 2025
Merged

fix(ios): memory leak cause by style prop changes#4021
mfazekas merged 1 commit into
rnmapbox:mainfrom
Gnative:fix-memory-leak-with-style-url

Conversation

@Gnative

@Gnative Gnative commented Oct 8, 2025

Copy link
Copy Markdown
Contributor

Sorry @mfazekas I caused a memory leak with the previous update for styleURL when mounting/unmounting map instances. Hopefully this fix is ok.

  • I've read CONTRIBUTING.md
  • I updated the doc/other generated code with running yarn generate in the root folder
  • I have tested the new feature on /example app.
    • In V11 mode/ios
    • In New Architecture mode/ios
    • In V11 mode/android
    • In New Architecture mode/android
  • I added/updated a sample - if a new feature was implemented (/example)

@Gnative
Gnative temporarily deployed to CI with Mapbox Tokens October 8, 2025 18:54 — with GitHub Actions Inactive
@Gnative
Gnative temporarily deployed to CI with Mapbox Tokens October 8, 2025 18:54 — with GitHub Actions Inactive
@Gnative
Gnative temporarily deployed to CI with Mapbox Tokens October 8, 2025 18:54 — with GitHub Actions Inactive
@Gnative
Gnative temporarily deployed to CI with Mapbox Tokens October 8, 2025 18:54 — with GitHub Actions Inactive
@Gnative
Gnative temporarily deployed to CI with Mapbox Tokens October 8, 2025 18:54 — with GitHub Actions Inactive
@Gnative
Gnative temporarily deployed to CI with Mapbox Tokens October 8, 2025 18:54 — with GitHub Actions Inactive
@Gnative
Gnative temporarily deployed to CI with Mapbox Tokens October 8, 2025 18:54 — with GitHub Actions Inactive
@mfazekas
mfazekas merged commit 50ee8b8 into rnmapbox:main Oct 10, 2025
11 checks passed
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