Merge pull request #1152 from Liyw979/improve/buildClassFrom2

remove Optional for JimpleView
soot-oss · Jan 9, 2025 · 31e67ba · 31e67ba
2 parents 7e85b26 + d7e9ede
commit 31e67ba
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/sootup.jimple.frontend/src/main/java/sootup/jimple/frontend/JimpleView.java b/sootup.jimple.frontend/src/main/java/sootup/jimple/frontend/JimpleView.java
@@ -129,11 +129,11 @@ Optional<SootClass> getAbstractClass(@Nonnull ClassType type) {
                   .collect(Collectors.joining(",")),
           foundClassSources.get(0).getSourcePath());
     }
-    return buildClassFrom(foundClassSources.get(0));
+    return Optional.of(buildClassFrom(foundClassSources.get(0)));
   }
 
   @Nonnull
-  private synchronized Optional<SootClass> buildClassFrom(AbstractClassSource classSource) {
+  private synchronized SootClass buildClassFrom(AbstractClassSource classSource) {
 
     ClassType classType = classSource.getClassType();
     SootClass theClass;
@@ -144,7 +144,7 @@ private synchronized Optional<SootClass> buildClassFrom(AbstractClassSource clas
       theClass = cache.getClass(classType);
     }
 
-    return Optional.of(theClass);
+    return theClass;
   }
 
   private synchronized void resolveAll() {