Refactoring: use Computable in JS std library search

F-22 · Apr 2, 2013 · d24b390 · d24b390
1 parent 957e97a
commit d24b390
Showing 1 changed file with 7 additions and 6 deletions.
diff --git a/idea/src/org/jetbrains/jet/plugin/framework/KotlinFrameworkDetector.java b/idea/src/org/jetbrains/jet/plugin/framework/KotlinFrameworkDetector.java
@@ -27,6 +27,7 @@
 import com.intellij.openapi.roots.OrderRootType;
 import com.intellij.openapi.roots.ProjectRootModificationTracker;
 import com.intellij.openapi.roots.libraries.Library;
+import com.intellij.openapi.util.Computable;
 import com.intellij.openapi.util.Key;
 import com.intellij.openapi.util.Pair;
 import com.intellij.openapi.util.Ref;
@@ -123,11 +124,11 @@ public boolean process(Library library) {
 
     @Nullable
     private static Library getJSStandardLibrary(final Module module) {
-        final Ref<Library> jsLibrary = Ref.create();
-
-        ApplicationManager.getApplication().runReadAction(new Runnable() {
+        return ApplicationManager.getApplication().runReadAction(new Computable<Library>() {
             @Override
-            public void run() {
+            public Library compute() {
+                final Ref<Library> jsLibrary = Ref.create();
+
                 ModuleRootManager.getInstance(module).orderEntries().librariesOnly().forEachLibrary(new Processor<Library>() {
                     @Override
                     public boolean process(Library library) {
@@ -139,9 +140,9 @@ public boolean process(Library library) {
                         return true;
                     }
                 });
+
+                return jsLibrary.get();
             }
         });
-
-        return jsLibrary.get();
     }
 }