Bug 209319 - [ast rewrite] @SuppressWarning is inserting an extra line break
Summary: [ast rewrite] @SuppressWarning is inserting an extra line break
Status: VERIFIED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Core (show other bugs)
Version: 3.4   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.4 M4   Edit
Assignee: Martin Aeschlimann CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-11-09 05:05 EST by Philipe Mulet CLA
Modified: 2007-12-11 10:06 EST (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Philipe Mulet CLA 2007-11-09 05:05:55 EST
Build 3.4M3

On following code,

class Y {
	public void foobar(String one, String two) {
		System.out.println("Y#foobar");
	}
}

Use the quickfix in left ruler (cool addition btw) to add a '@SuppressWarning("unused")' on parameter 'one'.
The resulting code is:

class Y {
	public void foobar(@SuppressWarnings("unused")
	String one, String two) {
		System.out.println("Y#foobar");
	}
}

I would have expected:

class Y {
	public void foobar(@SuppressWarnings("unused") String one, String two) {
		System.out.println("Y#foobar");
	}
}
Comment 1 Martin Aeschlimann CLA 2007-11-12 10:21:44 EST
fixed > 20071112
Comment 2 Philipe Mulet CLA 2007-12-11 10:06:15 EST
Verified for 3.4 M4 using build I20071211-0010