Skip to main content

Please review fix for 6822008 - context manager creation

1 reply [Last post]
Joined: 2004-12-13

Please review fix for:
6822008 ContextManager should not be created twice during ExecTool. execution

Added special case so that the default context manager is not confused with a
custom one. The custom one will be not be recreated, yet a custom one should
override a default one when the exec tool is being reinitialized (reused).

Reply viewing options

Select your preferred way to display the comments and click "Save settings" to activate your changes.
Joined: 2005-05-27

Fix is ok, but I would put the new code into a method to avoid duplication.

void unsetDefaultContextManager() {
if (context != null &&
"com.sun.javatest.exec.ContextManager".equals(context.getClass().getCanonicalName())) {
context = null;