Skip to content

Fix handler cleanup for memory leaks#105

Closed
callebtc wants to merge 1 commit intomainfrom
fix-handler-memory-leaks
Closed

Fix handler cleanup for memory leaks#105
callebtc wants to merge 1 commit intomainfrom
fix-handler-memory-leaks

Conversation

@callebtc
Copy link
Copy Markdown
Collaborator

Summary

  • ensure `PosUiCoordinator` clears pending callbacks when stopping so the activity reference is not leaked through its `Handler`
  • make `BalanceCheckActivity` remove pending messages in `onDestroy()` so foreground-dispatch callbacks do not leak the activity
  • documented lint/test attempts (blocked locally because Android SDK is not configured on this runner)

Testing

  • ./gradlew lint (fails: SDK location not found on runner)
  • ./gradlew test (fails: SDK location not found on runner)

@callebtc
Copy link
Copy Markdown
Collaborator Author

utACK

@ye0man ye0man added this to Numo Jan 15, 2026
@github-project-automation github-project-automation Bot moved this to Backlog in Numo Jan 15, 2026
@a1denvalu3 a1denvalu3 closed this Apr 1, 2026
@github-project-automation github-project-automation Bot moved this from Backlog to Done in Numo Apr 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants