From c26ae17ca7f49d08867647bcf42690e6e87bc0b7 Mon Sep 17 00:00:00 2001 From: Manu Sridharan Date: Mon, 13 May 2024 18:33:13 -0700 Subject: Delete OptionalEmptinessHandler method that is no longer needed (#954) Fixes #953 --- .../uber/nullaway/handlers/OptionalEmptinessHandler.java | 16 ---------------- 1 file changed, 16 deletions(-) diff --git a/nullaway/src/main/java/com/uber/nullaway/handlers/OptionalEmptinessHandler.java b/nullaway/src/main/java/com/uber/nullaway/handlers/OptionalEmptinessHandler.java index e8a70bc..4e1858b 100644 --- a/nullaway/src/main/java/com/uber/nullaway/handlers/OptionalEmptinessHandler.java +++ b/nullaway/src/main/java/com/uber/nullaway/handlers/OptionalEmptinessHandler.java @@ -50,7 +50,6 @@ import java.util.Objects; import java.util.Optional; import java.util.Set; import java.util.function.Consumer; -import java.util.function.Predicate; import javax.annotation.Nullable; import javax.lang.model.element.AnnotationMirror; import javax.lang.model.element.Element; @@ -164,21 +163,6 @@ public class OptionalEmptinessHandler extends BaseNoOpHandler { == Nullness.NULLABLE; } - @Override - public Predicate getAccessPathPredicateForNestedMethod( - TreePath path, VisitorState state) { - return ap -> { - if (ap.getElements().size() == 1) { - final Element e = ap.getRoot(); - if (e != null) { - return e.getKind().equals(ElementKind.LOCAL_VARIABLE) - && ap.getElements().get(0).getJavaElement() instanceof OptionalContentVariableElement; - } - } - return false; - }; - } - private void handleTestAssertions( VisitorState state, AccessPath.AccessPathContext apContext, -- cgit v1.2.3