[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[news.eclipse.tools] Re: TreeViewer doUpdateItem() optimization

Dave,

Thanks for the suggestion.  We actually had similar checks in TableViewer at
one point, but took them out because getting the text and image were a
noticeable performance hit.  In most of our cases, the flicker wasn't too
bad.  It may be worse for trees though.
It may be possible to push this comparison down into SWT, and minimize the
performance hit.  I'll discuss it with the SWT team.

The refresh updates all items because that's how refresh is spec'ed.  The
labels may have changed, even if the same elements are there.
It would be nice to have a flag indicating whether labels should be updated
for existing elements.
We are considering adding this for 2.0.

Nick