BcelObjectType: Add experimental code to set source file name for

reference

Experimental code leading to undesired ripple effects elsewhere,
requiring more rework which now I do not feel inclined to invest to
perfect source file string representation for post-compile binary
classes and aspects.

Relates to #218.

Signed-off-by: Alexander Kriegisch <[email protected]>

weaver/src/main/java/org/aspectj/weaver/bcel/BcelObjectType.java

@@ -159,10 +159,19 @@ public class BcelObjectType extends AbstractReferenceTypeDelegate {
 		// the only class in Java-1.4 with no superclasses
 		isObject = (javaClass.getSuperclassNameIndex() == 0);
 		ensureAspectJAttributesUnpacked();
-		// if (sourceContext instanceof SourceContextImpl) {
-		// ((SourceContextImpl)sourceContext).setSourceFileName(javaClass.
-		// getSourceFileName());
-		// }
+
+		// Experimental code leading to undesired ripple effects elsewhere, requiring more rework
+		/*
+		final String fileName = javaClass.getFileName();
+		final String sourceFileName = javaClass.getSourceFileName();
+		if (fileName == null || !fileName.endsWith(".class"))
+			setSourcefilename(sourceFileName);
+		else if (sourceFileName == null || sourceFileName.isEmpty() || sourceFileName.endsWith(".class"))
+			setSourcefilename(fileName);
+		else
+			setSourcefilename(fileName + " (from " + sourceFileName + ")");
+		*/
+
 		setSourcefilename(javaClass.getSourceFileName());
 	}