Skip to content

Resolve NPE with smali breakpoint#2718

Merged
skylot merged 3 commits intoskylot:masterfrom
RedArms:fix-issue-2717
Dec 4, 2025
Merged

Resolve NPE with smali breakpoint#2718
skylot merged 3 commits intoskylot:masterfrom
RedArms:fix-issue-2717

Conversation

@RedArms
Copy link
Copy Markdown
Contributor

@RedArms RedArms commented Dec 4, 2025

Fixes #2717

Prevents a NPE when toggling breakpoints if the gutter is not initialized.
Refactored the check into a safe helper method safeRemoveTrackingIcon.

@skylot
Copy link
Copy Markdown
Owner

skylot commented Dec 4, 2025

@RedArms thanks for fix.
Although you somehow managed to include other commit into your PR, I applied a fix, but please review your commits before submitting PR 🙂

@RedArms
Copy link
Copy Markdown
Contributor Author

RedArms commented Dec 4, 2025

@skylot
Hello, I'm sorry, its my first contributing, thank you so much.
I will try to make other bug fixe (maybe all in one branch) and be more careful about my commits

@skylot skylot merged commit 27b15ae into skylot:master Dec 4, 2025
4 checks passed
@skylot skylot changed the title fix(gui): resolve exception in search dialog on project reload (#2714) Resolve NPE with smali breakpoint Dec 4, 2025
@skylot skylot added GUI Issues in jadx-gui module debugger labels Dec 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

debugger GUI Issues in jadx-gui module

Projects

None yet

2 participants