New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix failing CtExecutableReference#getActualMethod #1277

Merged
merged 1 commit into from May 4, 2017

Conversation

Projects
None yet
2 participants
@pvojtechovsky
Collaborator

pvojtechovsky commented May 2, 2017

should fix #1256, but I am not sure that it is correct solution. May be we should think about having reference to Formal Type declarer directly in CtTypePareameterReference (which would be added by compiler?) instead of searching for the declarer in all possible contexts where it might occur. How many place we need to fix next to this one?

@monperrus monperrus merged commit caafc7e into INRIA:master May 4, 2017

4 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage increased (+0.003%) to 81.479%
Details
pull_request-INRIA-spoon-master-docker /tmp/tmp68kg_ieu
Details
pull_request-INRIA-spoon-master-revapi Revapi: No change detected.
Details
@monperrus

This comment has been minimized.

Show comment
Hide comment
@monperrus

monperrus May 4, 2017

Collaborator

Thanks!

How many place we need to fix next to this one?

We'll see :-)

Collaborator

monperrus commented May 4, 2017

Thanks!

How many place we need to fix next to this one?

We'll see :-)

@pvojtechovsky pvojtechovsky deleted the pvojtechovsky:testGenericExecRefProblem branch May 5, 2017

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