Factor out some anonymous IWeaveRequestor classes to inner classes

Drive-by cosmetics in the context of #279. Signed-off-by: Alexander Kriegisch <[email protected]>
eclipse-aspectj · Feb 8, 2024 · 6117924 · 6117924
1 parent faa36ca
commit 6117924
Show file tree

Hide file tree

Showing 2 changed files with 55 additions and 60 deletions.
diff --git a/weaver/src/main/java/org/aspectj/weaver/bcel/BcelWeaver.java b/weaver/src/main/java/org/aspectj/weaver/bcel/BcelWeaver.java
@@ -972,29 +972,7 @@ public Iterator<UnwovenClassFile> getClassFileIterator() {
 			}
 
 			public IWeaveRequestor getRequestor() {
-				return new IWeaveRequestor() {
-					public void acceptResult(IUnwovenClassFile result) {
-						try {
-							writeZipEntry(result.getFilename(), result.getBytes());
-						} catch (IOException ex) {
-						}
-					}
-
-					public void processingReweavableState() {
-					}
-
-					public void addingTypeMungers() {
-					}
-
-					public void weavingAspects() {
-					}
-
-					public void weavingClasses() {
-					}
-
-					public void weaveCompleted() {
-					}
-				};
+				return new WeaveRequestor();
 			}
 		});
 		// /* BUG 40943*/
@@ -1003,6 +981,25 @@ public void weaveCompleted() {
 		return c;
 	}
 
+	private class WeaveRequestor implements IWeaveRequestor {
+		public void acceptResult(IUnwovenClassFile result) {
+			try {
+				writeZipEntry(result.getFilename(), result.getBytes());
+			} catch (IOException ex) {
+			}
+		}
+
+		public void processingReweavableState() {}
+
+		public void addingTypeMungers() {}
+
+		public void weavingAspects() {}
+
+		public void weavingClasses() {}
+
+		public void weaveCompleted() {}
+	}
+
 	private Set<IProgramElement> candidatesForRemoval = null;
 
 	// variation of "weave" that sources class files from an external source.

diff --git a/weaver/src/main/java/org/aspectj/weaver/tools/WeavingAdaptor.java b/weaver/src/main/java/org/aspectj/weaver/tools/WeavingAdaptor.java
@@ -916,52 +916,50 @@ public Iterator<UnwovenClassFile> getClassFileIterator() {
 		}
 
 		public IWeaveRequestor getRequestor() {
-			return new IWeaveRequestor() {
-
-				public void acceptResult(IUnwovenClassFile result) {
-					if (wovenClass == null) {
-						wovenClass = result;
-						String name = result.getClassName();
-						if (shouldDump(name.replace('/', '.'), false)) {
-							dump(name, result.getBytes(), false);
-						}
-					} else {
-						// Classes generated by weaver e.g. around closure advice
-						String className = result.getClassName();
-						byte[]	resultBytes = result.getBytes();
-
-						if (SimpleCacheFactory.isEnabled()) {
-							SimpleCache lacache=SimpleCacheFactory.createSimpleCache();
-							lacache.put(result.getClassName(), wovenClass.getBytes(), result.getBytes());
-							lacache.addGeneratedClassesNames(wovenClass.getClassName(), wovenClass.getBytes(), result.getClassName());
-						}
+			return new WeaveRequestor();
+		}
 
-						generatedClasses.put(wovenClass.getClassName(), wovenClass);
-						generatedClassHandler.acceptClass(className, null, resultBytes);
+		private class WeaveRequestor implements IWeaveRequestor {
+			@Override
+			public void acceptResult(IUnwovenClassFile result) {
+				if (wovenClass == null) {
+					wovenClass = result;
+					String name = result.getClassName();
+					if (shouldDump(name.replace('/', '.'), false)) {
+						dump(name, result.getBytes(), false);
+					}
+				} else {
+					// Classes generated by weaver e.g. around closure advice
+					String className = result.getClassName();
+					byte[]	resultBytes = result.getBytes();
+
+					if (SimpleCacheFactory.isEnabled()) {
+						SimpleCache lacache=SimpleCacheFactory.createSimpleCache();
+						lacache.put(result.getClassName(), wovenClass.getBytes(), result.getBytes());
+						lacache.addGeneratedClassesNames(wovenClass.getClassName(), wovenClass.getBytes(), result.getClassName());
 					}
-				}
 
-				public void processingReweavableState() {
+					generatedClasses.put(wovenClass.getClassName(), wovenClass);
+					generatedClassHandler.acceptClass(className, null, resultBytes);
 				}
+			}
 
-				public void addingTypeMungers() {
-				}
+			public void processingReweavableState() {}
 
-				public void weavingAspects() {
-				}
+			public void addingTypeMungers() {}
 
-				public void weavingClasses() {
-				}
+			public void weavingAspects() {}
 
-				public void weaveCompleted() {
-					// ResolvedType.resetPrimitives();
-					if (delegateForCurrentClass != null) {
-						delegateForCurrentClass.weavingCompleted();
-					}
-					// ResolvedType.resetPrimitives();
-					// bcelWorld.discardType(typeBeingProcessed.getResolvedTypeX()); // work in progress
+			public void weavingClasses() {}
+
+			public void weaveCompleted() {
+				// ResolvedType.resetPrimitives();
+				if (delegateForCurrentClass != null) {
+					delegateForCurrentClass.weavingCompleted();
 				}
-			};
+				// ResolvedType.resetPrimitives();
+				// bcelWorld.discardType(typeBeingProcessed.getResolvedTypeX()); // work in progress
+			}
 		}
 	}