From: aclement <aclement>
Date: Sat, 7 Nov 2009 02:04:54 +0000 (+0000)
Subject: more optimizations
X-Git-Tag: V1_6_7~137
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=7ab4ecf0d42728083c60d0dd86fb04e0fcd6bc7e;p=aspectj.git

more optimizations
---

diff --git a/weaver/src/org/aspectj/weaver/bcel/BcelClassWeaver.java b/weaver/src/org/aspectj/weaver/bcel/BcelClassWeaver.java
index 1743be01d..0e04fe37b 100644
--- a/weaver/src/org/aspectj/weaver/bcel/BcelClassWeaver.java
+++ b/weaver/src/org/aspectj/weaver/bcel/BcelClassWeaver.java
@@ -2521,7 +2521,7 @@ class BcelClassWeaver implements IClassWeaver {
 		// we want to match ajsynthetic constructors...
 		if (startsAngly && mg.getName().equals("<init>")) {
 			return matchInit(mg, shadowAccumulator);
-		} else if (!shouldWeaveBody(mg)) { // .isAjSynthetic()) {
+		} else if (!shouldWeaveBody(mg)) {
 			return false;
 		} else {
 			if (startsAngly && mg.getName().equals("<clinit>")) {
@@ -3067,11 +3067,8 @@ class BcelClassWeaver implements IClassWeaver {
 	private static boolean captureLowLevelContext = false;
 
 	private boolean match(BcelShadow shadow, List<BcelShadow> shadowAccumulator) {
-		// System.err.println("match: " + shadow);
-		if (captureLowLevelContext) { // duplicate blocks - one with context
-			// capture, one without, seems faster
-			// than multiple
-			// 'ifs()'
+		// Duplicate blocks - one with context one without, seems faster than multiple 'ifs'
+		if (captureLowLevelContext) {
 			ContextToken shadowMatchToken = CompilationAndWeavingContext.enteringPhase(
 					CompilationAndWeavingContext.MATCHING_SHADOW, shadow);
 			boolean isMatched = false;