Description Dusko 2005-11-23 10:06:19 EST

There is memory leak in DeferredLayoutCommand. It has two fields that potentialy can keep lot of object in memory - containerEP and viewAdapters. Only the containerEP is cleaned after execution. Since the command goes on the undo stack after execution it holds onto objects in the viewAdapters list. In my particular scenario it accounted for a memory leak that was causing Out Of Memory exceptions.

Proposed fix:

1) Remove the "containerEP = null;" line from the "doExecute(...)" method.

2) Add the following method:

	protected void cleanup() {
		viewAdapters = null;
		containerEP = null;
		super.cleanup();
	}