Skip to content

fixed: reply already sent and a possible ANR #410

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

Merged
merged 5 commits into from
Jan 16, 2025
Merged

fixed: reply already sent and a possible ANR #410

Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
3b80703
Merge pull request #375 from RodrigoSMarques/dev
RodrigoSMarques Aug 30, 2024
ec0fe56
Merge pull request #400 from RodrigoSMarques/dev
RodrigoSMarques Dec 20, 2024
e45821b
Merge pull request #402 from RodrigoSMarques/dev
RodrigoSMarques Dec 20, 2024
e015812
Merge pull request #404 from RodrigoSMarques/dev
RodrigoSMarques Dec 28, 2024
4ba8520
fixed: reply already sent and a possible ANR
Junglee-Faisal Jan 15, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
18 changes: 18 additions & 0 deletions android/src/main/java/br/com/rsmarques/flutter_branch_sdk/MethodResultWrapper.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,14 +9,26 @@
public class MethodResultWrapper implements MethodChannel.Result {
private final MethodChannel.Result methodResult;
private final Handler handler;
private boolean called;

MethodResultWrapper(MethodChannel.Result result) {
methodResult = result;
handler = new Handler(Looper.getMainLooper());
}

private synchronized boolean checkNotCalled() {
if (called) {
return false;
}
called = true;
return true;
}

@Override
public void success(final Object result) {
if (!checkNotCalled()) {
return;
}
handler.post(
new Runnable() {
@Override
Expand All @@ -33,6 +45,9 @@ public void run() {
@Override
public void error(
final String errorCode, final String errorMessage, final Object errorDetails) {
if (!checkNotCalled()) {
return;
}
handler.post(
new Runnable() {
@Override
Expand All @@ -48,6 +63,9 @@ public void run() {

@Override
public void notImplemented() {
if (!checkNotCalled()) {
return;
}
handler.post(
new Runnable() {
@Override
Expand Down
Loading