From 24fde7110bdca770f18f49f42a9be9f1a47d8784 Mon Sep 17 00:00:00 2001
From: Zalim Bashorov <Zalim.Bashorov@jetbrains.com>
Date: Fri, 20 Sep 2013 21:48:42 +0400
Subject: [PATCH] J2K: reverted temporary fix after update to IDEA 132.325 EAP
 -- removed JavaToKotlinTranslator#NEVER_CHANGE.

---
 .../jet/j2k/JavaToKotlinTranslator.java         | 12 ++----------
 .../StandaloneJavaToKotlinConverterTest.java    | 17 +++++++----------
 2 files changed, 9 insertions(+), 20 deletions(-)

diff --git a/j2k/src/org/jetbrains/jet/j2k/JavaToKotlinTranslator.java b/j2k/src/org/jetbrains/jet/j2k/JavaToKotlinTranslator.java
index eaefa3dfdfc..3bb67ff6be2 100644
--- a/j2k/src/org/jetbrains/jet/j2k/JavaToKotlinTranslator.java
+++ b/j2k/src/org/jetbrains/jet/j2k/JavaToKotlinTranslator.java
@@ -20,7 +20,6 @@ import com.intellij.core.JavaCoreProjectEnvironment;
 import com.intellij.lang.java.JavaLanguage;
 import com.intellij.openapi.Disposable;
 import com.intellij.openapi.project.Project;
-import com.intellij.openapi.roots.ProjectRootModificationTracker;
 import com.intellij.psi.PsiElement;
 import com.intellij.psi.PsiFile;
 import com.intellij.psi.PsiFileFactory;
@@ -44,13 +43,6 @@ public class JavaToKotlinTranslator {
         }
     };
 
-    /*package*/ static final ProjectRootModificationTracker NEVER_CHANGED = new ProjectRootModificationTracker() {
-        @Override
-        public long getModificationCount() {
-            return 0;
-        }
-    };
-
     private JavaToKotlinTranslator() {
     }
 
@@ -74,8 +66,6 @@ public class JavaToKotlinTranslator {
         JavaCoreApplicationEnvironment applicationEnvironment = new JavaCoreApplicationEnvironment(DISPOSABLE);
         JavaCoreProjectEnvironment javaCoreEnvironment = new JavaCoreProjectEnvironment(DISPOSABLE, applicationEnvironment);
 
-        javaCoreEnvironment.getProject().registerService(ProjectRootModificationTracker.class, NEVER_CHANGED);
-
         javaCoreEnvironment.addJarToClassPath(PathUtil.findRtJar());
         File annotations = findAnnotations();
         if (annotations != null && annotations.exists()) {
@@ -167,6 +157,8 @@ public class JavaToKotlinTranslator {
         }
     }
 
+    // Used in the Kotlin Web Demo.
+    @SuppressWarnings("UnusedDeclaration")
     public static String translateToKotlin(String code) {
         return generateKotlinCode(code);
     }
diff --git a/j2k/tests/test/org/jetbrains/jet/j2k/StandaloneJavaToKotlinConverterTest.java b/j2k/tests/test/org/jetbrains/jet/j2k/StandaloneJavaToKotlinConverterTest.java
index 564eac543e0..fa18e12eec2 100644
--- a/j2k/tests/test/org/jetbrains/jet/j2k/StandaloneJavaToKotlinConverterTest.java
+++ b/j2k/tests/test/org/jetbrains/jet/j2k/StandaloneJavaToKotlinConverterTest.java
@@ -16,8 +16,6 @@
 
 package org.jetbrains.jet.j2k;
 
-import com.intellij.mock.MockProject;
-import com.intellij.openapi.roots.ProjectRootModificationTracker;
 import com.intellij.openapi.util.io.FileUtil;
 import com.intellij.psi.PsiFile;
 import com.intellij.psi.PsiJavaFile;
@@ -35,10 +33,12 @@ import java.io.IOException;
 
 import static org.jetbrains.jet.JetTestUtils.createEnvironmentWithMockJdkAndIdeaAnnotations;
 
+@SuppressWarnings("JUnitTestCaseWithNoTests")
 public class StandaloneJavaToKotlinConverterTest extends UsefulTestCase {
     private final String myDataPath;
     private final String myName;
 
+    @SuppressWarnings("JUnitTestCaseWithNonTrivialConstructors")
     public StandaloneJavaToKotlinConverterTest(String dataPath, String name) {
         myDataPath = dataPath;
         myName = name;
@@ -48,10 +48,7 @@ public class StandaloneJavaToKotlinConverterTest extends UsefulTestCase {
     protected void runTest() throws Throwable {
         JetCoreEnvironment jetCoreEnvironment = createEnvironmentWithMockJdkAndIdeaAnnotations(getTestRootDisposable(), ConfigurationKind.JDK_ONLY);
 
-        MockProject project = (MockProject) jetCoreEnvironment.getProject();
-        project.registerService(ProjectRootModificationTracker.class, JavaToKotlinTranslator.NEVER_CHANGED);
-
-        Converter converter = new Converter(project);
+        Converter converter = new Converter(jetCoreEnvironment.getProject());
 
         String javaPath = "j2k/tests/testData/" + getTestFilePath();
         String kotlinPath = javaPath.replace(".jav", ".kt");
@@ -127,7 +124,7 @@ public class StandaloneJavaToKotlinConverterTest extends UsefulTestCase {
     }
 
     @NotNull
-    private String fileToKotlin(Converter converter, @NotNull String text) {
+    private static String fileToKotlin(Converter converter, @NotNull String text) {
         return generateKotlinCode(converter, JavaToKotlinTranslator.createFile(converter.getProject(), text));
     }
 
@@ -141,7 +138,7 @@ public class StandaloneJavaToKotlinConverterTest extends UsefulTestCase {
     }
 
     @NotNull
-    private String methodToKotlin(Converter converter, String text) throws IOException {
+    private static String methodToKotlin(Converter converter, String text) throws IOException {
         String result = fileToKotlin(converter, "final class C {" + text + "}")
                 .replaceAll("class C\\(\\) \\{", "");
         result = result.substring(0, result.lastIndexOf("}"));
@@ -149,7 +146,7 @@ public class StandaloneJavaToKotlinConverterTest extends UsefulTestCase {
     }
 
     @NotNull
-    private String statementToKotlin(Converter converter, String text) throws Exception {
+    private static String statementToKotlin(Converter converter, String text) throws Exception {
         String result = methodToKotlin(converter, "void main() {" + text + "}");
         int pos = result.lastIndexOf("}");
         result = result.substring(0, pos).replaceFirst("fun main\\(\\) : Unit \\{", "");
@@ -157,7 +154,7 @@ public class StandaloneJavaToKotlinConverterTest extends UsefulTestCase {
     }
 
     @NotNull
-    private String expressionToKotlin(Converter converter, String code) throws Exception {
+    private static String expressionToKotlin(Converter converter, String code) throws Exception {
         String result = statementToKotlin(converter, "Object o =" + code + "}");
         result = result.replaceFirst("var o : Any\\? =", "");
         return prettify(result);