<div dir="auto">Sounds great!</div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, 24 Oct 2020 at 1:13 AM, Juan Vuletich via Cuis-dev <<a href="mailto:cuis-dev@lists.cuis.st">cuis-dev@lists.cuis.st</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;padding-left:1ex;border-left-color:rgb(204,204,204)">Hi Folks,<br>
<br>
Today, the morphic world is an instance of PasteUpMorph. But <br>
PasteUpMorph is a more general class, just an area where you can play <br>
with submorphs. For World functionality, there is an instance var called <br>
'worldState' that holds an instance of WorldState. This means that <br>
PasteUpMorph has two very different purposes, and that the World is <br>
actually split in two classes.<br>
<br>
I want to make the world an instance of a new class, WorldMorph, that is <br>
subclass of PasteUpMorph. This allows pushing down a lot of <br>
world-specific stuff from PasteUpMorph, making the world a self <br>
sufficient, meaningful object, and remove the ugly WorldState class. All <br>
this also means a reduction in the loc count for these objects by 10%.<br>
<br>
Still, before pushing these changes, I'd like to know if there's a <br>
downside, or any feedback.<br>
<br>
Thanks,<br>
<br>
-- <br>
Juan Vuletich<br>
<a href="http://www.cuis-smalltalk.org" rel="noreferrer" target="_blank">www.cuis-smalltalk.org</a><br>
<a href="https://github.com/Cuis-Smalltalk/Cuis-Smalltalk-Dev" rel="noreferrer" target="_blank">https://github.com/Cuis-Smalltalk/Cuis-Smalltalk-Dev</a><br>
<a href="https://github.com/jvuletich" rel="noreferrer" target="_blank">https://github.com/jvuletich</a><br>
<a href="https://www.linkedin.com/in/juan-vuletich-75611b3" rel="noreferrer" target="_blank">https://www.linkedin.com/in/juan-vuletich-75611b3</a>       <br>
@JuanVuletich<br>
<br>
-- <br>
Cuis-dev mailing list<br>
<a href="mailto:Cuis-dev@lists.cuis.st" target="_blank">Cuis-dev@lists.cuis.st</a><br>
<a href="https://lists.cuis.st/mailman/listinfo/cuis-dev" rel="noreferrer" target="_blank">https://lists.cuis.st/mailman/listinfo/cuis-dev</a><br>
</blockquote></div></div>