Skip to content

Conversation

@tobrun
Copy link
Member

@tobrun tobrun commented Nov 21, 2018

This PR uses generics to apply to correct geometry type for an annotation.

@tobrun tobrun added this to the annotation-0.4.0 milestone Nov 21, 2018
@tobrun tobrun self-assigned this Nov 21, 2018
@tobrun tobrun requested a review from LukasPaczos November 21, 2018 17:26
@tobrun tobrun force-pushed the tvn-generic-geometry branch from 9513907 to 2d76836 Compare November 21, 2018 18:11
Copy link
Contributor

@LukasPaczos LukasPaczos left a comment

Choose a reason for hiding this comment

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

One note, otherwise 🚀

@tobrun tobrun force-pushed the tvn-generic-geometry branch from 2d76836 to 5ff54bf Compare November 22, 2018 09:41
@tobrun tobrun force-pushed the tvn-generic-geometry branch from 5ff54bf to b75a175 Compare November 22, 2018 10:38
Copy link
Contributor

@LukasPaczos LukasPaczos left a comment

Choose a reason for hiding this comment

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

🚀

}

void setGeometry(Geometry geometry) {
public void setGeometry(T geometry) {
Copy link
Contributor

Choose a reason for hiding this comment

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

Since we are making set/get public, javadoc entry would be great.

@tobrun tobrun force-pushed the tvn-generic-geometry branch from b75a175 to 966f874 Compare December 2, 2018 09:53
@tobrun tobrun merged commit 990124b into master Dec 2, 2018
@tobrun tobrun deleted the tvn-generic-geometry branch December 2, 2018 20:57
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.

2 participants