Re: [cdt-patch] Solution to bug#43524 : Removing a define from a .c fil

[Hoda Amer <hamer@xxxxxxxxxx>]

Alain,
I have a solution to this problem. 
The previous handling ignored reconciling
in some cases, which was not correct. 
Actually, you get a reconcile for every
region once, but the redraw draws the whole thing.
The proper thing to do is to redraw
only if  we have a real change.
Give me until tomorrow and I will submit
the solution.

Thanks
Hoda Amer
Staff Software Engineer
Rational Software - IBM Software Group

"Alain Magloire" <alain@xxxxxxx> Sent by: cdt-patch-admin@xxxxxxxxxxx 10/20/2003 04:13 PM Please respond to cdt-patch

To:        cdt-patch@xxxxxxxxxxx         cc:                 Subject:        Re: [cdt-patch] Solution to bug#43524  : Removing a define from a .c file causes issues in

> Content-Type: text/plain; charset="US-ASCII"
> 
> UI: 
>         Fix for bug#43524 : Removing a define
from a .c file causes issues 
> in the outline
> 
> There was an unnecessary check before the call to reconcile() in 
> CReconcilingStrategy that did not always work.
> Multiple calls to reconcile() will have no effect after the first
update 
> of the file.
> 

I would like to revert this patch.
But It is causing major perfomance slow down on Eclipse/Motif or I guess
any other platform that is not Windows.

The outliner flickers and redraw like crazy.

The check is to limit the redrawing for the same region.

Comments?

_______________________________________________
cdt-patch mailing list
cdt-patch@xxxxxxxxxxx
http://dev.eclipse.org/mailman/listinfo/cdt-patch