Skip to content
This repository has been archived by the owner on Aug 8, 2023. It is now read-only.

deprecation cleanup for 5.0.0 #7767

Merged
merged 1 commit into from
Jan 18, 2017
Merged

deprecation cleanup for 5.0.0 #7767

merged 1 commit into from
Jan 18, 2017

Conversation

tobrun
Copy link
Member

@tobrun tobrun commented Jan 18, 2017

Closes #7575, finalising public api for 5.0.0, cleanup deprecation

  • remove deprecated emerald
  • remove deprecation annotation for setStyle
  • remove deprecated dismiss tracking on gesture

This PR doesn't include the removal of TextureView.
As proposed here, I'm going to pick this up with #5766.

Review @zugaldia

[android] - remove deprecation annoation for setStyle

[android] - dismiss tracking on gesture deprecation cleanup
@tobrun tobrun added the Android Mapbox Maps SDK for Android label Jan 18, 2017
@tobrun tobrun added this to the android-v5.0.0 milestone Jan 18, 2017
@tobrun tobrun self-assigned this Jan 18, 2017
@tobrun tobrun requested a review from zugaldia January 18, 2017 12:24
@mention-bot
Copy link

@tobrun, thanks for your PR! By analyzing this pull request, we identified @tmcw to be potential reviewers.

@@ -824,7 +824,6 @@ public void setStyleUrl(@NonNull String url) {
* @deprecated use {@link #setStyleUrl(String)} instead with versioned url methods from {@link Style}
*/
@UiThread
@Deprecated
Copy link
Member

Choose a reason for hiding this comment

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

This isn't removing the method, only the annotation.

Copy link
Member Author

Choose a reason for hiding this comment

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

That is correct. There is actually some back story behind this deprecation.. At some point in time we introduced versioned style methods. This introduced the @Deprecated annotation to setStyleUrl. Not much after that, this was reverted in #5193 but the deprecated annotation was not removed. I'm definitely open to revisit the decisions made in these issues, I actually don't have a strong opinion on this, but I think removing the annotation in this PR reflects the current Style.java implementation.

Copy link
Member

Choose a reason for hiding this comment

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

I think removing the annotation in this PR reflects the current Style.java implementation

In that case, let's leave like you put it.

@tobrun tobrun merged commit c637a65 into master Jan 18, 2017
@tobrun tobrun deleted the tvn-deprecation-cleanup branch January 18, 2017 17:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Android Mapbox Maps SDK for Android
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants