From 7bd20737d2b41779ab3b650d18197cd609b2e7c8 Mon Sep 17 00:00:00 2001 From: "Denis.Zhdanov" Date: Mon, 21 May 2012 08:48:43 +0400 Subject: [PATCH] Correct wrap processing at the core formatter (PhpFormatterTest.testReformatWI10456) --- .../intellij/formatting/FormatProcessor.java | 20 +------------------ 1 file changed, 1 insertion(+), 19 deletions(-) diff --git a/platform/lang-impl/src/com/intellij/formatting/FormatProcessor.java b/platform/lang-impl/src/com/intellij/formatting/FormatProcessor.java index 76cb6b3232a1..baf5f788838c 100644 --- a/platform/lang-impl/src/com/intellij/formatting/FormatProcessor.java +++ b/platform/lang-impl/src/com/intellij/formatting/FormatProcessor.java @@ -16,8 +16,6 @@ package com.intellij.formatting; -import com.intellij.lang.ASTNode; -import com.intellij.openapi.application.ApplicationManager; import com.intellij.openapi.diagnostic.Logger; import com.intellij.openapi.editor.Document; import com.intellij.openapi.editor.TextChange; @@ -25,13 +23,10 @@ import com.intellij.openapi.editor.ex.DocumentEx; import com.intellij.openapi.editor.impl.BulkChangesMerger; import com.intellij.openapi.editor.impl.TextChangeImpl; import com.intellij.openapi.fileTypes.StdFileTypes; -import com.intellij.openapi.project.ProjectManager; import com.intellij.openapi.util.TextRange; -import com.intellij.psi.PsiElement; import com.intellij.psi.codeStyle.CodeStyleSettings; import com.intellij.psi.codeStyle.CommonCodeStyleSettings; import com.intellij.psi.formatter.DocumentBasedFormattingModel; -import com.intellij.psi.formatter.PsiBasedFormattingModel; import com.intellij.util.ui.UIUtil; import gnu.trove.TIntObjectHashMap; import org.jetbrains.annotations.NotNull; @@ -538,8 +533,7 @@ class FormatProcessor { return true; } if (wrap != null && wrap.getFirstEntry() != null) { - //myCurrentBlock = wrap.getFirstEntry(); - myCurrentBlock = getFirstBlockOnNewLine(); + myCurrentBlock = wrap.getFirstEntry(); wrap.markAsUsed(); return true; } @@ -582,18 +576,6 @@ class FormatProcessor { return false; } - @Nullable - private LeafBlockWrapper getFirstBlockOnNewLine() { - LeafBlockWrapper current = myCurrentBlock; - while (current != null) { - WhiteSpace whiteSpace = current.getWhiteSpace(); - if (whiteSpace.containsLineFeeds() && whiteSpace.containsLineFeedsInitially()) return current; - if (current.getPreviousBlock() == null) return current; - current = current.getPreviousBlock(); - } - return null; - } - /** * Allows to answer if wrap of the {@link #myWrapCandidate} object (if any) may be replaced by the given wrap. *