Skip to content

added setters to location engine and javadoc#402

Merged
cammace merged 3 commits into
masterfrom
cam-395
Mar 24, 2017
Merged

added setters to location engine and javadoc#402
cammace merged 3 commits into
masterfrom
cam-395

Conversation

@cammace

@cammace cammace commented Mar 21, 2017

Copy link
Copy Markdown

Closes #395

@mention-bot

Copy link
Copy Markdown

@cammace, thanks for your PR! By analyzing the history of the files in this pull request, we identified @zugaldia, @Guardiola31337 and @tobrun to be potential reviewers.

.setInterval(1000)
.setFastestInterval(1000)
.setSmallestDisplacement(3.0f);
.setInterval(interval)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@cammace I like where this is going, particularly the great javadoc added. I'd like to make this fields optional to set, like GMS does (example. How about we make them objects instead with a default value of null?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants