[Cuis-dev] LayoutMorph subclass instance #beColumn has not effect

[Juan Vuletich]

On 3/13/2023 8:41 PM, Alexandre Rousseau wrote:
> I looked at the commit ...
>
>      aMorph == self ifTrue: [^ self error: 'cannot be own submorph'].
>
>
> but this will not save me from a more insidious mistake such as ...
>
>     m1 = Morph new.
>     m2 = Morph new.
>     m3 = Morph new.
>
>     m1 addSubmorph: m2.
>     m2 addSubmorph: m3.
>     m3 addSubmorph: m1.
>
>
> Anyway, I didn't want to complicate your life/lives.   Thanks for 
> this. It will help reduce some mistakes.
>
> A.
>

Right. But a single test for identity is quick, while a validation of 
the whole tree structure could get pretty expensive. That's why I didn't 
add it.

Thanks,

-- 
Juan Vuletich
cuis.st
github.com/jvuletich
researchgate.net/profile/Juan-Vuletich
independent.academia.edu/JuanVuletich
patents.justia.com/inventor/juan-manuel-vuletich
linkedin.com/in/juan-vuletich-75611b3
twitter.com/JuanVuletich

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.cuis.st/mailman/archives/cuis-dev/attachments/20230314/fe4f2722/attachment.htm>