[squeak-dev] some CharacterScanner methods weirdly send setStopConditions twice

Bob Arning arning315 at comcast.net
Sat Sep 28 20:17:37 UTC 2013


Change Set:        CenteredTextFix-BG
Date:            31 May 2003
Author:            Boris Gaertner

In an MVC project, centered text is not displayed centered when it is 
edited. This is an attempt to fix that glitch. Please have a look at it 
and give it some testing. After all, a careless change in 
CharacterScanner can easily ruin the image.
=======================

This was when a second #setStopConditions was added preceding the 
rightMargin setting. My guess is nobody thought to remove the original 
#setStopConditions further down. Then in 2006 the new #setStopConditions 
was moved after the rightMargin setting.

Cheers,
Bob

On 9/28/13 3:51 PM, Nicolas Cellier wrote:
> Sending twice some complex initialization message with inter-mixed 
> state messing smells...
> This is for example in CharacterBlockScanner>>buildCharacterBlockIn:
>
> I found http://bugs.squeak.org/view.php?id=3570 which were included in 
> Graphics-ar.39
> (in 3.9 repository see 
> http://source.squeak.org/39a/Graphics-ar.39.diff). 
> <http://source.squeak.org/39a/Graphics-ar.39.diff%29.>..
> ...but the doubled setStopConditions was there already, the change 
> just swapped some lines...
>
> This is not in the Multilingual branch, so it seems newer than 2002...
>
> Does anyone remember where to dig to understand why, which bug it 
> solves, which feature it supports, and whether it is really needed?
>
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20130928/6cef4471/attachment.htm


More information about the Squeak-dev mailing list