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.

Upgrade to JCodeModel 2.7.11#1566

Merged
yDelouis merged 3 commits into
androidannotations:developandroidannotations/androidannotations:developfrom
WonderCsabo:update_jcodemodelWonderCsabo/androidannotations:update_jcodemodelCopy head branch name to clipboard
Oct 6, 2015
Merged

Upgrade to JCodeModel 2.7.11#1566
yDelouis merged 3 commits into
androidannotations:developandroidannotations/androidannotations:developfrom
WonderCsabo:update_jcodemodelWonderCsabo/androidannotations:update_jcodemodelCopy head branch name to clipboard

Conversation

@WonderCsabo

Copy link
Copy Markdown
Member

Taking advantage of some API changes, as well fixing #1323.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is more complicated. Why it is better ?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It will import just the R class instead of R.string, and fix #1323.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okay, thanks.

@yDelouis

Copy link
Copy Markdown
Contributor

Is it possible to remove the method in APTCodeModelHelper where we use reflection ?

@WonderCsabo

Copy link
Copy Markdown
Member Author

That method is a thorn in my flesh, but i think we can only achieve that through reflection.

@dodgex

dodgex commented Oct 6, 2015

Copy link
Copy Markdown
Member

Hey Guys, are there any News on this PR? :)

@WonderCsabo

Copy link
Copy Markdown
Member Author

From my side, it can be merged. @yDelouis ?

yDelouis added a commit that referenced this pull request Oct 6, 2015
@yDelouis yDelouis merged commit bcc4b5b into androidannotations:develop Oct 6, 2015
@yDelouis

yDelouis commented Oct 6, 2015

Copy link
Copy Markdown
Contributor

You're right. We didn't agree about throwing the exception but it's not a big deal.

@yDelouis yDelouis added this to the 4.0 milestone Oct 6, 2015
@WonderCsabo WonderCsabo deleted the update_jcodemodel branch October 7, 2015 09:28
@WonderCsabo

Copy link
Copy Markdown
Member Author

Yeah, thanks!

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.

3 participants

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