diff --git a/js/js.tests/test/org/jetbrains/k2js/test/config/TestConfig.java b/js/js.tests/test/org/jetbrains/k2js/test/config/TestConfig.java index cb830ee2d56..66760d4ebdd 100644 --- a/js/js.tests/test/org/jetbrains/k2js/test/config/TestConfig.java +++ b/js/js.tests/test/org/jetbrains/k2js/test/config/TestConfig.java @@ -49,7 +49,6 @@ public final class TestConfig extends Config { List libFiles = new ArrayList(); for (String libFileName : LIB_FILE_NAMES) { JetFile file = null; - //TODO: close stream? try { @SuppressWarnings("IOResourceOpenedButNotSafelyClosed") InputStream stream = new FileInputStream(LIBRARIES_LOCATION + libFileName); @@ -70,6 +69,7 @@ public final class TestConfig extends Config { return libFiles; } + @Override @NotNull public List generateLibFiles() { if (jsLibFiles == null) { diff --git a/js/js.tests/test/org/jetbrains/k2js/test/semantics/FunctionTest.java b/js/js.tests/test/org/jetbrains/k2js/test/semantics/FunctionTest.java index 4015543b0a4..c2f4f0fbce1 100644 --- a/js/js.tests/test/org/jetbrains/k2js/test/semantics/FunctionTest.java +++ b/js/js.tests/test/org/jetbrains/k2js/test/semantics/FunctionTest.java @@ -94,13 +94,13 @@ public class FunctionTest extends AbstractExpressionTest { } //TODO: disabled. Probable cause : vars in closures. -// -// public void testKT921() throws Exception { -// -// checkOutput("KT-921.kt", "1,\n" + -// "1, 2,\n" + -// "2, "); -// } + + //public void testKT921() throws Exception { + // + // checkOutput("KT-921.kt", "1,\n" + + // "1, 2,\n" + + // "2, "); + //} public void testFunctionInsideFunction() throws Exception { fooBoxTest(); diff --git a/js/js.translator/src/org/jetbrains/k2js/analyze/AnalyzerFacadeForJS.java b/js/js.translator/src/org/jetbrains/k2js/analyze/AnalyzerFacadeForJS.java index 922995b6095..7886c6818a0 100644 --- a/js/js.translator/src/org/jetbrains/k2js/analyze/AnalyzerFacadeForJS.java +++ b/js/js.translator/src/org/jetbrains/k2js/analyze/AnalyzerFacadeForJS.java @@ -39,7 +39,6 @@ import org.jetbrains.jet.lang.types.lang.JetStandardClasses; import org.jetbrains.k2js.config.Config; import java.util.Collection; -import java.util.Collections; import java.util.List; import java.util.Set; @@ -67,7 +66,7 @@ public final class AnalyzerFacadeForJS { final ModuleDescriptor owner = new ModuleDescriptor(""); - Predicate completely = Predicates.and(notLibFiles(config.getLibFiles()), filesToAnalyzeCompletely); + Predicate completely = Predicates.and(notLibFiles(config.getLibFiles()), filesToAnalyzeCompletely); TopDownAnalysisParameters topDownAnalysisParameters = new TopDownAnalysisParameters(completely, false, false); @@ -115,25 +114,6 @@ public final class AnalyzerFacadeForJS { return result; } - //TODO: exclude? - @NotNull - public static BindingContext analyzeNamespace(@NotNull JetFile file) { - BindingTraceContext bindingTraceContext = new BindingTraceContext(); - Project project = file.getProject(); - - final ModuleDescriptor owner = new ModuleDescriptor(""); - - TopDownAnalysisParameters topDownAnalysisParameters = new TopDownAnalysisParameters( - Predicates.alwaysTrue(), false, false); - - InjectorForTopDownAnalyzerForJs injector = new InjectorForTopDownAnalyzerForJs( - project, topDownAnalysisParameters, new ObservableBindingTrace(bindingTraceContext), owner, - JetControlFlowDataTraceFactory.EMPTY, JsConfiguration.jsLibConfiguration(project)); - - injector.getTopDownAnalyzer().analyzeFiles(Collections.singletonList(file)); - return bindingTraceContext.getBindingContext(); - } - private static final class JsConfiguration implements ModuleConfiguration { @NotNull diff --git a/js/js.translator/src/org/jetbrains/k2js/facade/K2JSTranslator.java b/js/js.translator/src/org/jetbrains/k2js/facade/K2JSTranslator.java index d7cf7ac9c31..99a0c17ee32 100644 --- a/js/js.translator/src/org/jetbrains/k2js/facade/K2JSTranslator.java +++ b/js/js.translator/src/org/jetbrains/k2js/facade/K2JSTranslator.java @@ -36,8 +36,6 @@ import java.io.FileWriter; import java.io.IOException; import java.util.*; -//TODO: clean up the code - /** * @author Pavel Talanov *