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

JS backend: descriptor parameter of DescriptorPredicate#apply marked as @NotNull.

Browse Source
This commit is contained in:
develar
2013-08-06 17:24:48 +04:00
committed by Zalim Bashorov
parent 9b09d5f6df
commit 7b67c47594
4 changed files with 10 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files
js/js.translator/src/org/jetbrains/k2js/translate/intrinsic/functions/factories/CompositeFIF.java
+3 -4
View File
@@ -16,7 +16,6 @@
package org.jetbrains.k2js.translate.intrinsic.functions.factories;
import com.google.common.base.Predicate;
import com.intellij.openapi.util.Pair;
import org.jetbrains.annotations.NotNull;
import org.jetbrains.annotations.Nullable;
@@ -36,11 +35,11 @@ public abstract class CompositeFIF implements FunctionIntrinsicFactory {
@NotNull
@Override
public Predicate<FunctionDescriptor> getPredicate() {
public DescriptorPredicate getPredicate() {
return new DescriptorPredicate() {
@Override
public boolean apply(@Nullable FunctionDescriptor descriptor) {
return descriptor != null && findIntrinsic(descriptor) != null;
public boolean apply(@NotNull FunctionDescriptor descriptor) {
return findIntrinsic(descriptor) != null;
}
};
}
js/js.translator/src/org/jetbrains/k2js/translate/intrinsic/functions/factories/PrimitiveUnaryOperationFIF.java
+2 -3
View File
@@ -53,8 +53,7 @@ public enum PrimitiveUnaryOperationFIF implements FunctionIntrinsicFactory {
@NotNull
private static final DescriptorPredicate NO_PARAMETERS = new DescriptorPredicate() {
@Override
public boolean apply(@Nullable FunctionDescriptor descriptor) {
assert descriptor != null;
public boolean apply(@NotNull FunctionDescriptor descriptor) {
return !JsDescriptorUtils.hasParameters(descriptor);
}
};
@@ -72,7 +71,7 @@ public enum PrimitiveUnaryOperationFIF implements FunctionIntrinsicFactory {
public FunctionIntrinsic getIntrinsic(@NotNull FunctionDescriptor descriptor) {
Name name = descriptor.getName();
JsUnaryOperator jsOperator = null;
JsUnaryOperator jsOperator;
if ("inv".equals(name.asString())) {
jsOperator = JsUnaryOperator.BIT_NOT;
}
js/js.translator/src/org/jetbrains/k2js/translate/intrinsic/functions/patterns/DescriptorPredicate.java
+2 -2
View File
@@ -17,10 +17,10 @@
package org.jetbrains.k2js.translate.intrinsic.functions.patterns;
import com.google.common.base.Predicate;
import org.jetbrains.annotations.Nullable;
import org.jetbrains.annotations.NotNull;
import org.jetbrains.jet.lang.descriptors.FunctionDescriptor;
public interface DescriptorPredicate extends Predicate<FunctionDescriptor> {
@Override
boolean apply(@Nullable FunctionDescriptor descriptor);
boolean apply(@SuppressWarnings("NullableProblems") @NotNull FunctionDescriptor descriptor);
}
js/js.translator/src/org/jetbrains/k2js/translate/intrinsic/functions/patterns/PatternBuilder.java
+3 -5
View File
@@ -18,7 +18,6 @@ package org.jetbrains.k2js.translate.intrinsic.functions.patterns;
import com.google.common.collect.Lists;
import org.jetbrains.annotations.NotNull;
import org.jetbrains.annotations.Nullable;
import org.jetbrains.jet.lang.descriptors.DeclarationDescriptor;
import org.jetbrains.jet.lang.descriptors.FunctionDescriptor;
import org.jetbrains.jet.lang.descriptors.NamespaceDescriptor;
@@ -85,8 +84,7 @@ public final class PatternBuilder {
return new DescriptorPredicate() {
@Override
public boolean apply(@Nullable FunctionDescriptor descriptor) {
assert descriptor != null;
public boolean apply(@NotNull FunctionDescriptor descriptor) {
//TODO: no need to wrap if we check beforehand
try {
return doApply(descriptor);
@@ -146,8 +144,8 @@ public final class PatternBuilder {
}
@Override
public boolean apply(@Nullable FunctionDescriptor functionDescriptor) {
if (functionDescriptor == null || (functionDescriptor.getReceiverParameter() == null) == receiverParameterExists) {
public boolean apply(@NotNull FunctionDescriptor functionDescriptor) {
if ((functionDescriptor.getReceiverParameter() == null) == receiverParameterExists) {
return false;
}