Merge "JDWP: update ReferenceType.InstancesTest"

1 files changed, 18 insertions, 11 deletions

diff --git a/jdwp/src/test/java/org/apache/harmony/jpda/tests/jdwp/ReferenceType/InstancesDebuggee.java b/jdwp/src/test/java/org/apache/harmony/jpda/tests/jdwp/ReferenceType/InstancesDebuggee.java
index b165f26..f59adf7 100644
--- a/jdwp/src/test/java/org/apache/harmony/jpda/tests/jdwp/ReferenceType/InstancesDebuggee.java
+++ b/jdwp/src/test/java/org/apache/harmony/jpda/tests/jdwp/ReferenceType/InstancesDebuggee.java
@@ -39,23 +39,30 @@ public class InstancesDebuggee extends SyncDebuggee {
 
 	@Override
 	public void run() {
-		//Objects reachable for garbage collection purpose
-		
+		// Objects reachable for garbage collection purpose.
 		ArrayList<MockClass> reachableObjs = new ArrayList<MockClass>();
-        
-		for(int i = 0; i < reachableObjNum; i++) {
-			reachableObjs.add(new MockClass(true));
-		}
-       
-		//Objects unreachable
-		for(int i = 0; i < unreachableObjNum; i++) {
-			new MockClass(false);
-		}
+		allocateReachableObjects(reachableObjs);
+
+		// Objects unreachable.
+		allocateUnreachableObjects();
+
 		synchronizer.sendMessage(JPDADebuggeeSynchronizer.SGNL_READY);
         logWriter.println("--> Debuggee: InstancesDebuggee...");
         synchronizer.receiveMessage(JPDADebuggeeSynchronizer.SGNL_CONTINUE);
 	}
 
+	private static void allocateReachableObjects(ArrayList<MockClass> reachableObjs) {
+	    for (int i = 0; i < reachableObjNum; i++) {
+            reachableObjs.add(new MockClass(true));
+        }
+	}
+
+	private static void allocateUnreachableObjects() {
+        for (int i = 0; i < unreachableObjNum; i++) {
+            new MockClass(false);
+        }
+	}
+
 	public static void main(String[] args) {
 		runDebuggee(InstancesDebuggee.class);
 	}