Bug 444284

Description Jongwook Kim 2014-09-16 13:32:59 EDT

When A.m() is moved to class B via A.b below,
Reference b is updated to 'this' after refactoring.
That changes semantics.

--------------------
Before (Output: 2)

public class A {
	B b = new B(1);

	void n(B b1) {
		b = b1;
	}

	void m() {
		n(new B(2));
		B b2 = b;           // look how b changes after refactoring
		System.out.println(b2.i);  
	}
}

public class B {
	int i = 0;

	public B(int j) {
		i = j;
	}
}

public class C {
	public static void main(String[] args) {
		A a = new A();
		a.m();
	}
}

--------------------
After (Output: 1)

public class A {
	B b = new B(1);

	void n(B b1) {
		b = b1;
	}
}

public class B {
	int i = 0;

	public B(int j) {
		i = j;
	}

	void m(A a) {
		a.n(new B(2));
		B b2 = this;    // should be 'a.b', NOT 'this'
		System.out.println(b2.i);
	}
}

public class C {
	public static void main(String[] args) {
		A a = new A();
		a.b.m(a);
	}
}