[squeak-dev] duplicated rectangles in DamageRecorder

K. K. Subramaniam subbukk at gmail.com
Fri Oct 2 15:44:54 UTC 2009


On Friday 02 Oct 2009 6:07:31 pm Juan Vuletich wrote:
> Hi Subbu,
>
> K. K. Subramaniam wrote:
> > On Friday 02 Oct 2009 5:23:26 pm Juan Vuletich wrote:
> >> In any case, unneeded calls to self changed are usual, as it says in the
> >> last comment of TextMorph>>#fit. So, I think that the check and early
> >> exit you propose in #recordInvalidRect: is a good idea and will include
> >> it in Cuis.
> >
> > Why not fix recordInvalidRect: to drop duplicates? Duplicates are getting
> > in because of the test for large intersections before a merge. This will
> > also avoid triggering Merge threshold unnecessarily.
> >
> > Subbu
>
> That's exactly what Yoshiki suggested and I said I'm doing in Cuis,
> isn't it?
Yes, indeed. I deserve a whack on my head for rushing a response before 
reading his mail completely.

Subbu




More information about the Squeak-dev mailing list