Skip to content
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

General fixes #1

Merged
merged 5 commits into from Jun 18, 2014
Merged

General fixes #1

merged 5 commits into from Jun 18, 2014

Conversation

yelinaung
Copy link
Contributor

I've bumped the gradle plugin versions, build tool version.
Also I had to add okhttp 1.6.0 and okhttp-urlconnection 1.6.0 because Picasso was raising an Exception. ( Detail here )

At least, I am able to build it and working for me now. Thanks a lot for open sourcing it.

chrisbanes added a commit that referenced this pull request Jun 18, 2014
@chrisbanes chrisbanes merged commit 33560c0 into chrisbanes:master Jun 18, 2014
@@ -115,7 +115,7 @@ public void run() {
} catch (RetrofitError re) {
retrofitError = re;
if (Constants.DEBUG) {
Log.d(getClass().getSimpleName(), "Error while completing network call", re);
Log.d(((Object) this).getClass().getSimpleName(), "Error while completing network call", re);

Choose a reason for hiding this comment

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

I'm curious to know the reason for this change. It was necesary? I don't think that this change improve readability.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Because Android Studion was raising an error while I'm looking around the codes. And I found a similar problem on stackoverflow and fixed it according to the accepted answer there.

I think it is a bug in Android Studio. As we know, Android Studio is based on the IntelliJ Platform and the existing functionality of IntelliJ IDEA Community Edition.

Google has developed it in cooperation with JetBrains. And same bug is reported to happen in IntelliJ as well. Have a look at the Error report

However a workaround to this issue is:

Casting the getClass() call to Object as follows:

((Object) this).getClass()

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants