azalea/kotlin-fork
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Use Unit instead of Void

Browse Source
This commit is contained in:
Andrey Breslav
2013-10-04 20:59:50 +04:00
committed by Alexander Udalov
parent 2e585f88a3
commit 00c3156b59
4 changed files with 27 additions and 23 deletions
Download Patch File Download Diff File Expand all files Collapse all files
compiler/frontend/src/org/jetbrains/jet/lang/resolve/lazy/descriptors/LazyClassDescriptor.java
+4 -3
View File
@@ -23,6 +23,7 @@ import com.google.common.collect.Lists;
import com.intellij.psi.PsiElement;
import jet.Function0;
import jet.Function1;
import jet.Unit;
import org.jetbrains.annotations.NotNull;
import org.jetbrains.annotations.Nullable;
import org.jetbrains.jet.lang.descriptors.*;
@@ -401,11 +402,11 @@ public class LazyClassDescriptor extends ClassDescriptorBase implements LazyDesc
return Collections.emptyList();
}
},
new Function1<Collection<JetType>, Void>() {
new Function1<Collection<JetType>, Unit>() {
@Override
public Void invoke(@NotNull Collection<JetType> supertypes) {
public Unit invoke(@NotNull Collection<JetType> supertypes) {
findAndDisconnectLoopsInTypeHierarchy(supertypes);
return null;
return Unit.VALUE;
}
});
compiler/tests/org/jetbrains/jet/storage/StorageManagerTest.java
+16 -15
View File
@@ -2,6 +2,7 @@ package org.jetbrains.jet.storage;
import jet.Function0;
import jet.Function1;
import jet.Unit;
import junit.framework.TestCase;
import org.jetbrains.jet.util.ReenteringLazyValueComputationException;
@@ -238,10 +239,10 @@ public class StorageManagerTest extends TestCase {
}
},
null,
new Function1<String, Void>() {
new Function1<String, Unit>() {
@Override
public Void invoke(String s) {
return null;
public Unit invoke(String s) {
return Unit.VALUE;
}
}
);
@@ -272,12 +273,12 @@ public class StorageManagerTest extends TestCase {
return "tolerant";
}
},
new Function1<String, Void>() {
new Function1<String, Unit>() {
@Override
public Void invoke(String s) {
public Unit invoke(String s) {
counter.inc();
assertEquals("tolerant", s);
return null;
return Unit.VALUE;
}
}
);
@@ -301,12 +302,12 @@ public class StorageManagerTest extends TestCase {
}
},
null,
new Function1<Collection<String>, Void>() {
new Function1<Collection<String>, Unit>() {
@Override
public Void invoke(Collection<String> strings) {
public Unit invoke(Collection<String> strings) {
counter.inc();
strings.add("postComputed");
return null;
return Unit.VALUE;
}
}
);
@@ -327,12 +328,12 @@ public class StorageManagerTest extends TestCase {
return strings;
}
},
new Function1<Collection<String>, Void>() {
new Function1<Collection<String>, Unit>() {
@Override
public Void invoke(Collection<String> strings) {
public Unit invoke(Collection<String> strings) {
counter.inc();
strings.add("postComputed");
return null;
return Unit.VALUE;
}
}
);
@@ -361,11 +362,11 @@ public class StorageManagerTest extends TestCase {
return "second";
}
},
new Function1<String, Void>() {
new Function1<String, Unit>() {
@Override
public Void invoke(String s) {
public Unit invoke(String s) {
fail("Recursion-tolerating value should not be post computed");
return null;
return Unit.VALUE;
}
}
);
core/util.runtime/src/org/jetbrains/jet/storage/LockBasedStorageManager.java
+3 -2
View File
@@ -18,6 +18,7 @@ package org.jetbrains.jet.storage;
import jet.Function0;
import jet.Function1;
import jet.Unit;
import org.jetbrains.annotations.NotNull;
import org.jetbrains.annotations.Nullable;
import org.jetbrains.jet.utils.ExceptionUtils;
@@ -99,7 +100,7 @@ public class LockBasedStorageManager implements StorageManager {
public <T> NotNullLazyValue<T> createLazyValueWithPostCompute(
@NotNull Function0<T> computable,
final Function1<Boolean, T> onRecursiveCall,
@NotNull final Function1<T, Void> postCompute
@NotNull final Function1<T, Unit> postCompute
) {
return new LockBasedNotNullLazyValue<T>(lock, computable) {
@Nullable
@@ -138,7 +139,7 @@ public class LockBasedStorageManager implements StorageManager {
@NotNull
@Override
public <T> NullableLazyValue<T> createNullableLazyValueWithPostCompute(
@NotNull Function0<T> computable, @NotNull final Function1<T, Void> postCompute
@NotNull Function0<T> computable, @NotNull final Function1<T, Unit> postCompute
) {
return new LockBasedLazyValue<T>(lock, computable) {
@Override
core/util.runtime/src/org/jetbrains/jet/storage/StorageManager.java
+4 -3
View File
@@ -18,6 +18,7 @@ package org.jetbrains.jet.storage;
import jet.Function0;
import jet.Function1;
import jet.Unit;
import org.jetbrains.annotations.NotNull;
import org.jetbrains.annotations.Nullable;
@@ -28,6 +29,7 @@ public interface StorageManager {
* @param valuesReferenceKind how to store the memoized values
*
* NOTE: if compute() has side-effects the WEAK reference kind is dangerous: the side-effects will be repeated if
* the value gets collected and then re-computed
*/
@NotNull
<K, V> MemoizedFunctionToNotNull<K, V> createMemoizedFunction(@NotNull Function1<K, V> compute);
@@ -41,7 +43,6 @@ public interface StorageManager {
<T> NotNullLazyValue<T> createRecursionTolerantLazyValue(@NotNull Function0<T> computable, @NotNull T onRecursiveCall);
/**
* @param computable
* @param onRecursiveCall is called if the computation calls itself recursively.
* The parameter to it is {@code true} for the first call, {@code false} otherwise.
* If {@code onRecursiveCall} is {@code null}, an exception will be thrown on a recursive call,
@@ -52,7 +53,7 @@ public interface StorageManager {
<T> NotNullLazyValue<T> createLazyValueWithPostCompute(
@NotNull Function0<T> computable,
@Nullable Function1<Boolean, T> onRecursiveCall,
@NotNull Function1<T, Void> postCompute
@NotNull Function1<T, Unit> postCompute
);
@NotNull
@@ -66,7 +67,7 @@ public interface StorageManager {
* see it in between)
*/
@NotNull
<T> NullableLazyValue<T> createNullableLazyValueWithPostCompute(@NotNull Function0<T> computable, @NotNull Function1<T, Void> postCompute);
<T> NullableLazyValue<T> createNullableLazyValueWithPostCompute(@NotNull Function0<T> computable, @NotNull Function1<T, Unit> postCompute);
<T> T compute(@NotNull Function0<T> computable);
}