Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 370656

Summary: [assist] remove signatures from method binding chokes on inline comment
Product: [Tools] Objectteams Reporter: Stephan Herrmann <stephan.herrmann>
Component: OTDTAssignee: Stephan Herrmann <stephan.herrmann>
Status: VERIFIED FIXED QA Contact:
Severity: normal    
Priority: P3    
Version: 2.1   
Target Milestone: 2.1 M6   
Hardware: Other   
OS: Linux   
Whiteboard:

Description Stephan Herrmann CLA 2012-02-05 08:29:06 EST 
Given this callin binding

	void checkAgainstInheritedMethods(MethodNode currentMethod,
			MethodNode[] methods /* remaining args are ignored*/)
	<- after
	void checkAgainstInheritedMethods(MethodBinding currentMethod,
			MethodBinding[] methods, int length,
			MethodBinding[] allInheritedMethods);

invoking the quick assist "Remove signatures..." creates bogus output:
	checkAgainstInheritedMethods/* remaining args are ignored */)
	<- after
	checkAgainstInheritedMethods;

Someone forgot to remove the closing ')' :)
Comment 1 Stephan Herrmann CLA 2012-02-05 13:15:18 EST 
AstRewriteAnalyzer needs to properly scan over comments and trailing ')' instead of just adding offset 1 after the last parameter.

Resolved for 2.1 M6 via commit 72d84d535b4ae88b8ed728eea05fd9c732187e01
Comment 2 Stephan Herrmann CLA 2012-09-22 16:31:02 EDT 
Verified using build 2.2.0.201209221349