Andre Poenitz wrote:
On Wed, Jun 13, 2007 at 11:25:34PM +0200, Abdelrazak Younes wrote:
I think this is too complicated. A better solution IMHO is to remove the
data member altogether and put that in the BufferView (either in the
CoordCache or in in a new DimensionCache):
[...]
You are certainly right when it comes to Model/View separation. The
dimension cache is a 'view thing' (depends e.g. on font size), so it
does not really belong _inside_ the insets. Also, insets that are way
off the visible area do not need it at all.
What kept me away from an 'external' solution so far was the
perceived cost as we would have a O(log n) vs O(1) lookup for
dimension data. With n being roughly the number of visible insets this
might not even be noticable, though.
Yes, the idea is to keep only the dimensions of Insets that are already
in the CoordCache. The performance impact should be very minimal if any.
Abdel.
