-
Notifications
You must be signed in to change notification settings - Fork 23
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.
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
Reduce dex method count impact #123
Comments
I agree, this could be super annoying during development. I hate having to use proguard during development. The guava dependency is not added to the apk btw., it only affects the library. |
we actually are considering to stop supporting the *.slim version as it 2016-06-14 14:44 GMT+02:00 Jasm Sison notifications@github.com:
|
@svenefftinge please continue supporting it - the dex method count is a major concern on Android, and a common compliant against Xtend is the bundled "bloated" Guava dependency. |
@svenefftinge although, I don't think I can use the .slim for Xtendroid anyway, since it has Active Annotations? |
the active annotations stuff is in a different library. the .slim stuff is 2016-06-14 20:10 GMT+02:00 Toby Kurien notifications@github.com:
|
See the method count impact of the dependencies here:
http://www.methodscount.com/?lib=com.github.tobykurien%3Axtendroid%3A0.13
Main culprits are Guava (we should use the slim version), Android Support Library, and the Gson dependency recently added.
The text was updated successfully, but these errors were encountered: