Hi Audrius,

Am Montag, den 15.05.2006, 21:35 +0200 schrieb Audrius Meskauskas:
> Roman Kennke wrote:
> 
> >Hi Audrius,
> >
> >+        // Where possible, do not repaint the component, extending the
> >+        // parent repaint region instead.
> >
> >You do understand that this was also happening in the previous
> >implementation? 
> >
> Something was not as expected, because (by putting 
> System.out.println(clip)  into BasicTableUI.paint) it was obvious that 
> the same region is repainted as much as nine times in a row during 
> resizing. Same output indicated for me that the blitting is gone somewhere.
> 
> Basically, the size of the repaintOrder in the replaced version is equal 
> to the number of the different components to repaint, even if some of 
> them are parents for other. Probably the code expected that some 
> rectangles are shared and removing one rectangle (after repainting) 
> would also result the null value to other. But this was not aways the case.

Aha. Anyway, good work! The speedup for internal frames is impressive.

/Roman

-- 
“Improvement makes straight roads, but the crooked roads, without
Improvement, are roads of Genius.” - William Blake

Attachment: signature.asc
Description: Dies ist ein digital signierter Nachrichtenteil

Reply via email to