Skip to content
This repository was archived by the owner on Feb 26, 2023. It is now read-only.

Remove try/catch block on @Background/@UIThread methods#682

Merged
DayS merged 2 commits intoandroidannotations:developfrom
tbruyelle:646_PropagateThreadExceptions
Jul 28, 2013
Merged

Remove try/catch block on @Background/@UIThread methods#682
DayS merged 2 commits intoandroidannotations:developfrom
tbruyelle:646_PropagateThreadExceptions

Conversation

@tbruyelle
Copy link
Contributor

Related to issue #646 and add fix for PR #677

Because the exception is now propagated, there is no need of catching
it.

Generated code :

    @Override
    public void backgroundThrowException() {
        BackgroundExecutor.execute(new BackgroundExecutor.Task("", 0, "") {


            @Override
            public void execute() {
                ThreadActivity_.super.backgroundThrowException();
            }

        }
        );
    }

Notes

  • The surroundWithTryCatch method is now unused, don't know if I have to remove it or not.

Loading
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants