[squeak-dev] duplicated rectangles in DamageRecorder

Juan Vuletich juan at jvuletich.org
Fri Oct 2 12:37:31 UTC 2009


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?

Cheers,
Juan Vuletich



More information about the Squeak-dev mailing list