Bug 357285

Description Tiberiu Rogojan 2011-09-09 16:20:06 EDT

Build Identifier: 20100917-0705

Let's take the following class for instance:

.....
public MessagePacket(SocketAddress from, SocketAddress to, Calendar timestamp, String message) {
		this.from = from;
		this.to = to;
		this.timestamp = timestamp;
		this.message = message;
	}
	
	// HERE YOU HAVE 5 NEW LINES!
	
	
	
	@Override
	public int hashCode() {
          // hashCode method....
        }
.....

If you chose to generate setters and getters and place them before the hashcode method, and your cursor position is let's say 4 LINES above hashcode method, then all the generated setters and getters will end up having 4 LINES between them.
The code generation code, shouldn't use the lines between cursor position and the next statement in order to compute the lines that it should use between getter and setter methods that are about to be generated. It should solely be based on a template that let's say uses one new line.

Reproducible: Always

Steps to Reproduce:
Leave N LINES between cursor position and a statement (method or whatever).
Alt + Shift + S -> generate getters and setters.
Select either getters, either setters, or both and hit OK
The generated setXXX getXXX methods have N Lines of new line between them!