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

fix: improve line preservation mode #1335

Merged
merged 1 commit into from
May 30, 2017

Conversation

monperrus
Copy link
Collaborator

the line preservation mode can do a better job in preserving indentation of ending brackets.
here is a solution for this.

@monperrus monperrus force-pushed the improve-line-preseervation branch 2 times, most recently from 2f546ed to 5410d83 Compare May 30, 2017 07:45
@spoon-bot
Copy link
Collaborator

Revapi Analysis results

Old API: fr.inria.gforge.spoon:spoon-core:jar:5.7.0-20170529.224522-137

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

Detected changes: 1.

Change 1

Name Element
Old method void spoon.reflect.visitor.printer.PrinterHelper::adjustPosition(spoon.reflect.declaration.CtElement, spoon.reflect.cu.CompilationUnit)
New none
Code java.method.removed
Description Method was removed.
Breaking source: breaking

@monperrus monperrus force-pushed the improve-line-preseervation branch from 5410d83 to 97b4fdc Compare May 30, 2017 12:50
@spoon-bot
Copy link
Collaborator

Revapi Analysis results

Old API: fr.inria.gforge.spoon:spoon-core:jar:5.7.0-20170529.224522-137

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

Detected changes: 1.

Change 1

Name Element
Old method void spoon.reflect.visitor.printer.PrinterHelper::adjustPosition(spoon.reflect.declaration.CtElement, spoon.reflect.cu.CompilationUnit)
New none
Code java.method.removed
Description Method was removed.
Breaking source: breaking

@surli surli merged commit 67934ee into INRIA:master May 30, 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.

3 participants