Re: [cdt-dev] Debugger tabs activate/deactivate methods

[Vladimir Prus <vladimir@xxxxxxxxxxxxxxxx>]

On Monday 22 February 2010 16:35:15 Mikhail Khodjaiants wrote:

Hi Mikhail,

> deactivated() was added to AbstractLaunchConfigurationTab for Eclipse
> 3.0. It probably went unnoticed at that time. Your suggestion to
> overwrite it for CLaunchConfigurationTab makes sense, my only concern
> is that it would change the behaviour of public classes.

Well, yes, it will. OTOH, as I've said a launch tab that fails to call
updateLaunchConfigurationDialog on change (and therefore implicitly call
performApply) is already broken.

I've made such a change locally, and it seems OK. I'll let it soak for
a while and unless any issues surfaces will created a patch in Bugzilla.

Thanks,

-- 
Vladimir Prus
CodeSourcery
vladimir@xxxxxxxxxxxxxxxx
(650) 331-3385 x722