From c883747e4840e8dc2379482fe7efde74bada860f Mon Sep 17 00:00:00 2001 From: Michael Williamson Date: Mon, 31 Oct 2016 19:38:03 +0000 Subject: Remove unused code --- .../couscous/frontends/java/JavaReader.java | 25 ++++++++-------------- 1 file changed, 9 insertions(+), 16 deletions(-) diff --git a/src/main/java/org/zwobble/couscous/frontends/java/JavaReader.java b/src/main/java/org/zwobble/couscous/frontends/java/JavaReader.java index 6553b91..f7f6217 100644 --- a/src/main/java/org/zwobble/couscous/frontends/java/JavaReader.java +++ b/src/main/java/org/zwobble/couscous/frontends/java/JavaReader.java @@ -1,6 +1,5 @@ package org.zwobble.couscous.frontends.java; -import com.google.common.collect.ImmutableList; import org.eclipse.jdt.core.compiler.IProblem; import org.eclipse.jdt.core.dom.*; import org.zwobble.couscous.ast.*; @@ -14,6 +13,7 @@ import org.zwobble.couscous.util.ExtraLists; import java.io.IOException; import java.nio.file.Path; +import java.util.ArrayList; import java.util.List; import java.util.Optional; import java.util.Set; @@ -44,6 +44,9 @@ public class JavaReader { public static List readClassesFromFiles(List sourcePaths, List sourceFiles) throws IOException { JavaParser parser = new JavaParser(); JavaReader reader = new JavaReader(); + + List types = new ArrayList<>(); + for (Path sourceFile : sourceFiles) { CompilationUnit ast = parser.parseCompilationUnit(sourcePaths, sourceFile); @@ -52,12 +55,12 @@ public class JavaReader { throw new RuntimeException("Errors during parsing:\n\n" + describe(errors)); } try { - reader.readCompilationUnit(ast); + types.add(reader.readCompilationUnit(ast)); } catch (Exception exception) { throw new RuntimeException("Error reading " + sourceFile, exception); } } - return reader.types(); + return types; } private static String describe(List errors) { @@ -70,29 +73,19 @@ public class JavaReader { error.getMessage(); } - private final ImmutableList.Builder classes; - private int anonymousClassCount = 0; private final Scope topScope = Scope.create(); - private JavaReader() { - classes = ImmutableList.builder(); - } - - private void readCompilationUnit(CompilationUnit ast) { + private TypeNode readCompilationUnit(CompilationUnit ast) { AbstractTypeDeclaration type = (AbstractTypeDeclaration) ast.types().get(0); if (type instanceof TypeDeclaration) { - classes.add(readTypeDeclaration((TypeDeclaration) type)); + return readTypeDeclaration((TypeDeclaration) type); } else if (type instanceof EnumDeclaration) { - classes.add(readEnumDeclaration((EnumDeclaration)type)); + return readEnumDeclaration((EnumDeclaration)type); } else { throw new UnsupportedOperationException(); } } - private List types() { - return this.classes.build(); - } - private TypeNode readEnumDeclaration(EnumDeclaration type) { ScalarType name = (ScalarType) typeOf(type.resolveBinding()); @SuppressWarnings("unchecked") -- cgit v1.2.3