Skip to content

Porting code to be compatible with Java 1.6#42

Merged
broady merged 2 commits intogooglemaps:masterfrom
nutsiepully:android_port
Oct 4, 2014
Merged

Porting code to be compatible with Java 1.6#42
broady merged 2 commits intogooglemaps:masterfrom
nutsiepully:android_port

Conversation

@nutsiepully
Copy link
Copy Markdown
Contributor

Hi,

I just wanted to let you know that I was able to get the library running on Android by making the library compatible with Java 6 instead of 7.

This is not helpful directly for the codebase, but I was wondering whether this is helpful as a side branch in case someone else runs into this issue or wants to use the library for Java 6/Android.

Best,
Pulkit

@nutsiepully
Copy link
Copy Markdown
Contributor Author

This is not meant to be merged into the current codebase, but as something that might be helpful on a side branch.

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.

Could you flip these equals comparators?

"OK".equals(status)
"INVALID_REQUEST".equals(status)

etc.

@broady
Copy link
Copy Markdown
Contributor

broady commented Oct 4, 2014

Thanks. Have you signed the Google CLA?

@nutsiepully
Copy link
Copy Markdown
Contributor Author

@broady - Incorporated the feedback. Just signed it. Thanks. Please let me know if any other changes are required.

@broady
Copy link
Copy Markdown
Contributor

broady commented Oct 4, 2014

Thanks!

broady added a commit that referenced this pull request Oct 4, 2014
Porting code to be compatible with Java 1.6
@broady broady merged commit d8d8b25 into googlemaps:master Oct 4, 2014
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.

2 participants