Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings
This repository was archived by the owner on Feb 26, 2023. It is now read-only.

Remove restErrorHandler field in rest service if not necessary#894

Merged
DayS merged 1 commit into
androidannotations:developandroidannotations/androidannotations:developfrom
yDelouis:removeUselessErrorHandleryDelouis/androidannotations:removeUselessErrorHandlerCopy head branch name to clipboard
May 11, 2014
Merged

Remove restErrorHandler field in rest service if not necessary#894
DayS merged 1 commit into
androidannotations:developandroidannotations/androidannotations:developfrom
yDelouis:removeUselessErrorHandleryDelouis/androidannotations:removeUselessErrorHandlerCopy head branch name to clipboard

Conversation

@yDelouis

Copy link
Copy Markdown
Contributor

If the interface annotated with @Rest doesn't extends RestClientErrorHandling, the field restErrorHandler won't be assigned and will be always null. So, we shouldn't create it in the generated class and we should remove all the try-catch.
That's what does this PR.

@DayS

DayS commented Jan 20, 2014

Copy link
Copy Markdown
Contributor

Yep, I mentioned it in #888. Thanks for the fix

@DayS DayS merged commit bead889 into androidannotations:develop May 11, 2014
@DayS

DayS commented May 11, 2014

Copy link
Copy Markdown
Contributor

Merged. Thanks. I resolved the conflict during the merge

@DayS DayS added Defect and removed Defect labels May 11, 2014
@DayS DayS added this to the 3.1 milestone May 11, 2014
@yDelouis yDelouis deleted the removeUselessErrorHandler branch May 11, 2014 10:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.