Review

Calls JFrame.setVisible(true) before JFrame.pack(). This may lead to undesired effects, such as the window being moved to the default plattform location.

Call JFrame.pack() before JFrame.setVisible(true).

• misplaced/call

mubench/examples/survey/JFramePack.java

packAfterShowHasNoEffect(JFrame)

class JFramePack {
	public void packAfterShowHasNoEffect(JFrame f) {
		f.setVisible(true);
		f.pack();
	}

}

public class PackBeforeVisible {
	public void pattern(JFrame f) {
		f.pack();
		f.setVisible(true);
	}
}