Skip to content
This repository was archived by the owner on Feb 12, 2022. It is now read-only.

Fixed some ClassCastException. #247

Closed
wants to merge 3 commits into from
Closed

Fixed some ClassCastException. #247

wants to merge 3 commits into from

Conversation

marcoaltieri
Copy link
Contributor

@marcoaltieri marcoaltieri commented Apr 30, 2017

They were caused by the casting to (Class) of a type that was a java.lang.reflect.ParameterizedType or java.lang.reflect.TypeVariable.
This seems to happen when the type to generate is for a class that contains generics

They were caused by the casting to (Class) of a type that was that was:
java.lang.reflect.ParameterizedType or java.lang.reflect.TypeVariable
…we have to cast it to Class.

Fixed minor typo in AnnotationInstanceEmitter.java
@jpbelang
Copy link
Contributor

jpbelang commented May 9, 2017

I'll look at this today. Thanks.

jpbelang added a commit that referenced this pull request May 29, 2017
@jpbelang
Copy link
Contributor

I've merged you pull request and added UTs.

To the question of TypeVariable and what to do with it, right now, I throw an exception, since there are no good raml responses I could write (or no way I could pull an annotation on a potientially unbounded variable). Opinions ?

@jpbelang
Copy link
Contributor

Oh, and thanks for fixing the variable names. And the typos :-)

@jstoiko
Copy link
Contributor

jstoiko commented Jun 14, 2017

Closing this since it has been merged already in another branch.

@jstoiko jstoiko closed this Jun 14, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants