Skip to content

#1751 Fix handling of possible builder creation methods with generic signature#1807

Merged
filiphr merged 1 commit into
mapstruct:masterfrom
filiphr:1751
May 24, 2019
Merged

#1751 Fix handling of possible builder creation methods with generic signature#1807
filiphr merged 1 commit into
mapstruct:masterfrom
filiphr:1751

Conversation

@filiphr

@filiphr filiphr commented May 4, 2019

Copy link
Copy Markdown
Member

When a method has a generic signature and the builder type is generic then the method return type does not match the builder type.
Therefore check only for raw types.
Add extra check for void method since a void method can't be a builder creation method

Fixes #1751
Fixes #1716

… generic signature

When a method has a generic signature and the builder type is generic then the method return type does not match the builder type.
Therefore check only for raw types.
Add extra check for void method since a void method can't be a builder creation method

@sjaakd sjaakd left a comment

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.

LGTM .. please merge :)

@filiphr

filiphr commented May 24, 2019

Copy link
Copy Markdown
Member Author

Thanks for the review @sjaakd

@filiphr filiphr merged commit 60c159a into mapstruct:master May 24, 2019
@filiphr filiphr deleted the 1751 branch May 24, 2019 21:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants