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

refactor: Move ElementPrinterHelper #1551

Merged
merged 2 commits into from
Sep 21, 2017

Conversation

pvojtechovsky
Copy link
Collaborator

  • ElementPrinterHelper moved next to DJPP - it belongs there some like others.
  • ListPrinter made package protected too

@pvojtechovsky
Copy link
Collaborator Author

@monperrus WDYT?

@monperrus
Copy link
Collaborator

indeed, thanks.

the fact that it is not tested directly clearly shows that it was never part of the public API.

@spoon-bot
Copy link
Collaborator

Revapi Analysis results

Old API: fr.inria.gforge.spoon:spoon-core:jar:6.0.0-20170921.175952-35

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

Detected changes: 3.

Change 1

Name Element
Old class spoon.reflect.visitor.ListPrinter
New class spoon.reflect.visitor.ListPrinter
Code java.class.nonPublicPartOfAPI
Description Class 'spoon.reflect.visitor.ListPrinter' is indirectly included in the API (by the means of method return type for example) but the class is not accessible (neither public nor protected).
Breaking binary: non_breaking,

Change 1

Name Element
Old class spoon.reflect.visitor.ListPrinter
New class spoon.reflect.visitor.ListPrinter
Code java.class.visibilityReduced
Description Visibility was reduced from 'public' to 'package'.
Breaking binary: breaking

Change 2

Name Element
Old class spoon.reflect.visitor.printer.ElementPrinterHelper
New none
Code java.class.removed
Description Class was removed.
Breaking binary: breaking

Change 3

Name Element
Old method spoon.reflect.visitor.printer.ElementPrinterHelper spoon.reflect.visitor.DefaultJavaPrettyPrinter::getElementPrinterHelper()
New method spoon.reflect.visitor.ElementPrinterHelper spoon.reflect.visitor.DefaultJavaPrettyPrinter::getElementPrinterHelper()
Code java.method.returnTypeChanged
Description The return type changed from 'spoon.reflect.visitor.printer.ElementPrinterHelper' to 'spoon.reflect.visitor.ElementPrinterHelper'.
Breaking binary: breaking

@monperrus monperrus merged commit ec1cb94 into INRIA:master Sep 21, 2017
@pvojtechovsky pvojtechovsky deleted the moveElementPrinterHelper branch September 21, 2017 18:36
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.

3 participants