Bug 53999 - "can't bind type" message has $ for . in type name for declare soft
Summary: "can't bind type" message has $ for . in type name for declare soft
Status: RESOLVED FIXED
Alias: None
Product: AspectJ
Classification: Tools
Component: Compiler (show other bugs)
Version: 1.1.1   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 1.2.1   Edit
Assignee: Adrian Colyer CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-03-07 22:07 EST by Wes Isberg CLA
Modified: 2004-10-21 04:31 EDT (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Wes Isberg CLA 2004-03-07 22:07:22 EST
cosmetic mistake in error message...

expected: can't bind type name 'unknown.Error'
  actual: can't bind type name 'unknown$Error'

(not sure why it would ever be munged with $)

public class Test {
	public static void main(String[] args) {
		test();
	}
	static void test() {
		throw new Error("hello");
	}
	static aspect A {
		declare soft : unknown.Error : call(void test()); // CE
	}
}
Comment 1 Jim Hugunin CLA 2004-03-18 14:15:17 EST
This error message is caused by first looking for a type Error in the unknown 
package and then looking for the inner type Error in the class unknown.  This 
second check uses the binary format for inner type names of unknown$Error.

I'm leaving as a bug until the cosmetic error is fixed.
Comment 2 Andrew Clement CLA 2004-07-27 09:28:38 EDT
fix integrated - waiting for build.
Comment 3 Andrew Clement CLA 2004-07-28 11:00:35 EDT
Fixed as of:

BUILD COMPLETE -  build.317
Date of build: 07/28/2004 12:16:33
Time to build: 121 minutes 44 seconds
Last changed: 07/27/2004 18:38:25
Latest good AspectJ jar available at:
download.eclipse.org/technology/ajdt/dev/aspectj-DEVELOPMENT.jar
Comment 4 Adrian Colyer CLA 2004-10-21 04:31:17 EDT
Fix released as part of AspectJ 1.2.1