Skip to content
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

Review fix(role): fix wrong roles #1397

Merged
merged 2 commits into from
Jun 23, 2017
Merged

Review fix(role): fix wrong roles #1397

merged 2 commits into from
Jun 23, 2017

Conversation

tdurieux
Copy link
Collaborator

@tdurieux tdurieux commented Jun 12, 2017

fix #1386

@tdurieux tdurieux force-pushed the fix_role branch 2 times, most recently from d46dacd to fa95062 Compare June 23, 2017 07:40
@tdurieux
Copy link
Collaborator Author

@monperrus @pvojtechovsky What is the best role for CtNewClassImpl#anonymousClass?

@spoon-bot
Copy link
Collaborator

Revapi Analysis results

Old API: fr.inria.gforge.spoon:spoon-core:jar:5.8.0-20170622.224606-57

New API: fr.inria.gforge.spoon:spoon-core:jar:5.8.0-SNAPSHOT

Detected changes: 1.

Change 1

Name Element
Old method spoon.reflect.path.CtRole spoon.reflect.annotations.MetamodelPropertyField::role()
New method spoon.reflect.path.CtRole[] spoon.reflect.annotations.MetamodelPropertyField::role()
Code java.method.returnTypeChanged
Description The return type changed from 'spoon.reflect.path.CtRole' to 'spoon.reflect.path.CtRole[]'.
Breaking binary: breaking

@monperrus
Copy link
Collaborator

What is the best role for CtNewClassImpl#anonymousClass?

What about CtRole.NESTED_TYPE?

@tdurieux
Copy link
Collaborator Author

Done for me

@tdurieux tdurieux changed the title WIP fix(role): fix wrong roles Review fix(role): fix wrong roles Jun 23, 2017
@INRIA INRIA deleted a comment from spoon-bot Jun 23, 2017
@spoon-bot
Copy link
Collaborator

Revapi Analysis results

Old API: fr.inria.gforge.spoon:spoon-core:jar:5.8.0-20170623.121440-58

New API: fr.inria.gforge.spoon:spoon-core:jar:5.8.0-SNAPSHOT

Detected changes: 1.

Change 1

Name Element
Old method spoon.reflect.path.CtRole spoon.reflect.annotations.MetamodelPropertyField::role()
New method spoon.reflect.path.CtRole[] spoon.reflect.annotations.MetamodelPropertyField::role()
Code java.method.returnTypeChanged
Description The return type changed from 'spoon.reflect.path.CtRole' to 'spoon.reflect.path.CtRole[]'.
Breaking binary: breaking

@monperrus monperrus merged commit 99a84a7 into INRIA:master Jun 23, 2017
@INRIA INRIA deleted a comment from spoon-bot Jun 23, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Wrong CtRole?
3 participants