Description Frank Fröhlich 2007-10-22 07:06:51 EDT

There is a coding error in method createCommand of class MacroCommandShell (pkg=org.eclipse.tptp.test.auto.gui.internal.commands) which prevents the recorder from insertion of focus commands. The first SWT.MouseUp event leads to setting the flag 'isMousePressed' to true. With a high degree of probalility this flag will never be reset because of the wrongly coded if-statement which follows the switch-statement for event.type:

if (event.type != SWT.KeyUp)
    isKeyPressed = false;
else if (event.type != SWT.MouseUp)
    isMousePressed = false;

Only in case of event.type is a SWT.KeyUp event (!!!), the isMousePressed flag is reset. The worse is that in case of the flag is set to true, SWT.MouseUp events will be ignored (see first if-statement of the method) and the handling for toBeAckEvent does not work (please have a look at case SWT.MouseUp in the switch-statement).

The 'else' need to be removed from the if-statement:

if (event.type != SWT.KeyUp)
    isKeyPressed = false;
if (event.type != SWT.MouseUp)
    isMousePressed = false;