Revert "Log parsed decorators."

This reverts commit 335d343.

com.ibm.wala.cast.python.jython3/source/com/ibm/wala/cast/python/parser/PythonParser.java

@@ -13,9 +13,7 @@
 import static com.ibm.wala.cast.python.util.Util.CLASS_METHOD_ANNOTATION_NAME;
 import static com.ibm.wala.cast.python.util.Util.DYNAMIC_ANNOTATION_KEY;
 import static com.ibm.wala.cast.python.util.Util.STATIC_METHOD_ANNOTATION_NAME;
-import static com.ibm.wala.cast.python.util.Util.getNames;
-import static com.ibm.wala.cast.python.util.Util.removeFileProtocolFromPath;
-import static java.util.logging.Logger.getLogger;
+import static com.ibm.wala.cast.python.util.Util.getNameStream;
 
 import com.ibm.wala.cast.ir.translator.AbstractClassEntity;
 import com.ibm.wala.cast.ir.translator.AbstractCodeEntity;
@@ -25,7 +23,6 @@
 import com.ibm.wala.cast.python.ir.PythonCAstToIRTranslator;
 import com.ibm.wala.cast.python.loader.DynamicAnnotatableEntity;
 import com.ibm.wala.cast.python.types.PythonTypes;
-import com.ibm.wala.cast.python.util.Util;
 import com.ibm.wala.cast.tree.CAst;
 import com.ibm.wala.cast.tree.CAstAnnotation;
 import com.ibm.wala.cast.tree.CAstEntity;
@@ -63,8 +60,6 @@
 import java.util.LinkedList;
 import java.util.Map;
 import java.util.function.Supplier;
-import java.util.logging.Level;
-import java.util.logging.Logger;
 import org.python.antlr.PythonTree;
 import org.python.antlr.ast.Assert;
 import org.python.antlr.ast.Assign;
@@ -146,8 +141,6 @@
 
 public abstract class PythonParser<T> extends AbstractParser<T> implements TranslatorToCAst {
 
-  private static final Logger LOGGER = getLogger(PythonParser.class.getName());
-
   private static boolean COMPREHENSION_IR = true;
 
   private CAstType codeBody =
@@ -1193,13 +1186,10 @@ public String toString() {
             };
 
         annotations.add(cAstAnnotation);
-
-        if (LOGGER.isLoggable(Level.INFO))
-          Util.getName(cAstAnnotation).ifPresent(n -> LOGGER.info("Found decorator: " + n));
       }
 
       boolean staticMethod =
-          getNames(annotations).stream().anyMatch(s -> s.equals(STATIC_METHOD_ANNOTATION_NAME));
+          getNameStream(annotations).anyMatch(s -> s.equals(STATIC_METHOD_ANNOTATION_NAME));
 
       CAstType functionType;
       boolean isMethod =
@@ -1276,8 +1266,7 @@ public CAstNode getAST() {
           if (function instanceof FunctionDef) {
 
             boolean classMethod =
-                getNames(annotations).stream()
-                    .anyMatch(s -> s.equals(CLASS_METHOD_ANNOTATION_NAME));
+                getNameStream(annotations).anyMatch(s -> s.equals(CLASS_METHOD_ANNOTATION_NAME));
 
             // Only add object metadata for non-static and non-class methods.
             if (isMethod && !staticMethod && !classMethod) {

com.ibm.wala.cast.python/source/com/ibm/wala/cast/python/loader/PythonLoader.java

@@ -1,7 +1,7 @@
 package com.ibm.wala.cast.python.loader;
 
 import static com.ibm.wala.cast.python.types.PythonTypes.pythonLoader;
-import static com.ibm.wala.cast.python.util.Util.getNames;
+import static com.ibm.wala.cast.python.util.Util.getNameStream;
 import static java.util.stream.Collectors.toList;
 
 import com.ibm.wala.cast.ir.translator.AstTranslator.AstLexicalInformation;
@@ -80,7 +80,7 @@ public DynamicMethodBody(
       // fill in the decorators.
       // FIXME: Process annotations with parameters.
       this.annotations =
-          getNames(entity.getAnnotations()).stream()
+          getNameStream(entity.getAnnotations())
               .map(s -> "L" + s)
               .map(TypeName::findOrCreate)
               .map(tn -> TypeReference.findOrCreate(pythonLoader, tn))

com.ibm.wala.cast.python/source/com/ibm/wala/cast/python/util/Util.java

@@ -15,13 +15,12 @@
 import com.ibm.wala.ipa.callgraph.Entrypoint;
 import com.ibm.wala.ipa.callgraph.propagation.PropagationCallGraphBuilder;
 import java.io.File;
-import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.Collection;
 import java.util.List;
-import java.util.Optional;
+import java.util.Objects;
 import java.util.logging.Logger;
-import java.util.stream.Collectors;
+import java.util.stream.Stream;
 
 public class Util {
 
@@ -71,48 +70,27 @@ public static List<File> getPathFiles(String pathSequence) {
   }
 
   /**
-   * Returns a {@link Collection} of annotation (decorator) names as {@link String}s from the given
-   * {@link Collection} of {@link CAstAnnotation}s. The decorator names may be dot (.)-separated.
+   * Returns a {@link Stream} of annotation (decorator) names as {@link String}s from the given
+   * {@link Collection} of {@link CAstAnnotation}s.
    *
    * @param annotations A {@link Collection} of {@link CAstAnnotation} for which to stream
    *     annotation (decorator) names.
-   * @return A {@link Collection} of names as {@link String}s corresponding to the given annotations
+   * @return A {@link Stream} of names as {@link String}s corresponding to the given annotations
    *     (decorators).
-   * @implNote The decorator names may not be fully-qualified. They are returned here as they are
-   *     presented in the CAst.
    */
-  public static Collection<String> getNames(Collection<CAstAnnotation> annotations) {
-    return annotations.stream().map(Util::getName).flatMap(Optional::stream).toList();
-  }
-
-  public static Optional<String> getName(CAstAnnotation annotation) {
-    if (annotation.getType().equals(CAST_DYNAMIC_ANNOTATION)) {
-      CAstNode node = (CAstNode) annotation.getArguments().get(DYNAMIC_ANNOTATION_KEY);
-      List<String> decoratorSegments = getDecoratorSegments(node.getChild(0));
-      String decoratorName = decoratorSegments.stream().collect(Collectors.joining("."));
-      return Optional.of(decoratorName);
-    }
-
-    return Optional.empty();
-  }
-
-  private static List<String> getDecoratorSegments(CAstNode node) {
-    List<String> ret = new ArrayList<>();
-
-    if (node != null) {
-      List<CAstNode> children = node.getChildren();
-
-      for (CAstNode child : children) {
-        List<String> childDecoratorSegments = getDecoratorSegments(child);
-        ret.addAll(childDecoratorSegments);
-      }
-
-      Object value = node.getValue();
-
-      if (value != null && value instanceof String) ret.add((String) value);
-    }
-
-    return ret;
+  public static Stream<String> getNameStream(Collection<CAstAnnotation> annotations) {
+    if (annotations == null) return Stream.empty();
+
+    return annotations.stream()
+        .filter(a -> a.getType().equals(CAST_DYNAMIC_ANNOTATION))
+        .map(a -> a.getArguments().get(DYNAMIC_ANNOTATION_KEY))
+        .filter(Objects::nonNull)
+        .map(CAstNode.class::cast)
+        .map(n -> n.getChild(0))
+        .map(n -> n.getChild(0))
+        .map(CAstNode::getValue)
+        .filter(v -> v instanceof String)
+        .map(String.class::cast);
   }
 
   /**