From fd2b9895f01fe3ba786bb49b583f0cf02c559e52 Mon Sep 17 00:00:00 2001
From: Dmitry Jemerov <yole@jetbrains.com>
Date: Tue, 21 Sep 2010 14:37:42 +0400
Subject: [PATCH] failing test for PY-1065

---
 python/src/META-INF/python-plugin-common.xml  |  1 +
 .../jetbrains/python/formatter/PyBlock.java   |  2 ++
 .../PyWhiteSpaceFormattingStrategy.java       | 20 +++++++++++++++++++
 .../formatter/spaceAfterCommaWrappedLine.py   |  2 ++
 .../spaceAfterCommaWrappedLine_after.py       |  2 ++
 .../com/jetbrains/python/PyFormatterTest.java |  4 ++++
 6 files changed, 31 insertions(+)
 create mode 100644 python/src/com/jetbrains/python/formatter/PyWhiteSpaceFormattingStrategy.java
 create mode 100644 python/testData/formatter/spaceAfterCommaWrappedLine.py
 create mode 100644 python/testData/formatter/spaceAfterCommaWrappedLine_after.py

diff --git a/python/src/META-INF/python-plugin-common.xml b/python/src/META-INF/python-plugin-common.xml
index 322c139b6009..74a122710edd 100644
--- a/python/src/META-INF/python-plugin-common.xml
+++ b/python/src/META-INF/python-plugin-common.xml
@@ -176,6 +176,7 @@
                    serviceImplementation="com.jetbrains.python.psi.resolve.PythonPathCache"/>
 
     <preserveIndentOnPaste fileType="Python"/>
+    <lang.whiteSpaceFormattingStrategy language="Python" implementationClass="com.jetbrains.python.formatter.PyWhiteSpaceFormattingStrategy"/>
 
     <!-- Django -->
     <codeInsight.lineMarkerProvider language="Python" implementationClass="com.jetbrains.django.annotator.ViewMethodLineMarkerProvider"/>
diff --git a/python/src/com/jetbrains/python/formatter/PyBlock.java b/python/src/com/jetbrains/python/formatter/PyBlock.java
index 5b850e222f10..a0e82a143d27 100644
--- a/python/src/com/jetbrains/python/formatter/PyBlock.java
+++ b/python/src/com/jetbrains/python/formatter/PyBlock.java
@@ -98,9 +98,11 @@ public class PyBlock implements ASTBlock {
         // whitespace between sub-blocks would include \ characters,
         // because the IDEA 5.0/5.1 formatter core requires that whitespace
         // between blocks must include whitespace-only characters
+        /*
         if (child.getText().indexOf('\\') >= 0) {
           return Collections.emptyList();
         }
+        */
 
         continue;
       }
diff --git a/python/src/com/jetbrains/python/formatter/PyWhiteSpaceFormattingStrategy.java b/python/src/com/jetbrains/python/formatter/PyWhiteSpaceFormattingStrategy.java
new file mode 100644
index 000000000000..ca1913c6f849
--- /dev/null
+++ b/python/src/com/jetbrains/python/formatter/PyWhiteSpaceFormattingStrategy.java
@@ -0,0 +1,20 @@
+package com.jetbrains.python.formatter;
+
+import com.intellij.formatting.WhiteSpaceFormattingStrategy;
+import org.jetbrains.annotations.NotNull;
+
+/**
+ * @author yole
+ */
+public class PyWhiteSpaceFormattingStrategy implements WhiteSpaceFormattingStrategy {
+  @Override
+  public int check(@NotNull CharSequence text, int start, int end) {
+    for (int i = start; i < end; i++) {
+      char c = text.charAt(i);
+      if (c != ' ' && c != '\t' && c != '\n' && c != '\\') {
+        return i;
+      }
+    }
+    return end;
+  }
+}
diff --git a/python/testData/formatter/spaceAfterCommaWrappedLine.py b/python/testData/formatter/spaceAfterCommaWrappedLine.py
new file mode 100644
index 000000000000..fa5eee9b5f39
--- /dev/null
+++ b/python/testData/formatter/spaceAfterCommaWrappedLine.py
@@ -0,0 +1,2 @@
+import contextlib,\
+    math,decimal
diff --git a/python/testData/formatter/spaceAfterCommaWrappedLine_after.py b/python/testData/formatter/spaceAfterCommaWrappedLine_after.py
new file mode 100644
index 000000000000..5b1c4b0e5866
--- /dev/null
+++ b/python/testData/formatter/spaceAfterCommaWrappedLine_after.py
@@ -0,0 +1,2 @@
+import contextlib,\
+    math, decimal
diff --git a/python/testSrc/com/jetbrains/python/PyFormatterTest.java b/python/testSrc/com/jetbrains/python/PyFormatterTest.java
index ae77184d39c4..29d1986ffe81 100644
--- a/python/testSrc/com/jetbrains/python/PyFormatterTest.java
+++ b/python/testSrc/com/jetbrains/python/PyFormatterTest.java
@@ -72,6 +72,10 @@ public class PyFormatterTest extends PyLightFixtureTestCase {
     doTest();
   }
 
+  public void _testSpaceAfterCommaWrappedLine() {  // PY-1065
+    doTest();
+  }
+
   private void doTest() {
     myFixture.configureByFile("formatter/" + getTestName(true) + ".py");
     ApplicationManager.getApplication().runWriteAction(new Runnable() {