Issue 3266 in pharo: Clean TextEditor

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
30 messages Options
12
Reply | Threaded
Open this post in threaded view
|

Re: Issue 3266 in pharo: Clean TextEditor

Juan Vuletich-4
Hi Stef,

Stéphane Ducasse wrote:
>>> - clean again all the MVC dependencies....
>>>  
>>>      
>
> FillInTheBlank? PopUpMenu?
>  

Those might be not pretty, and I agree they can be replaced by something
better, but they are not MVC dependencies.

>  
>> Well, that doesn't stop Mariano from fighting the deepest internals of the VM... Developing any needed know-how in the Pharo community will ensure that Pharo will survive any technical difficulties that might arise in any part of the system.
>>    
>
> yes this is the idea.
> And we should do it but again my plate is full

That's ok. But you are the leader of the community. When you say:

"I do not think that we can build something on simpleMorphic, what we
can do is look at the kernel and see how the changes can be applied to
Morphic",

 it really means that it should better not be done. It means "I think it
can't be done, don't waste your time".

If, instead of that, you say "I can't help, but feel free to go ahead",
the reaction of the community will be completely different.

Cheers,
Juan Vuletich

Reply | Threaded
Open this post in threaded view
|

Re: Issue 3266 in pharo: Clean TextEditor

Stéphane Ducasse
In reply to this post by Stéphane Ducasse
Indeed. I mean
        I can't help, but feel free to go ahead

Stef

>>>>
> "I do not think that we can build something on simpleMorphic, what we can do is look at the kernel and see how the changes can be applied to Morphic",
>
> it really means that it should better not be done. It means "I think it can't be done, don't waste your time".
>
> If, instead of that, you say "I can't help, but feel free to go ahead", the reaction of the community will be completely different.
>
> Cheers,
> Juan Vuletich
>


Reply | Threaded
Open this post in threaded view
|

Re: Issue 3266 in pharo: Clean TextEditor

Igor Stasenko
IMO SimpleMorphic, minus 'color' ivar in Morph class is a way to go.
(i mean get rid that 'color' thing in base class)  :)


On 24 November 2010 16:49, Stéphane Ducasse <[hidden email]> wrote:

> Indeed. I mean
>        I can't help, but feel free to go ahead
>
> Stef
>
>>>>>
>> "I do not think that we can build something on simpleMorphic, what we can do is look at the kernel and see how the changes can be applied to Morphic",
>>
>> it really means that it should better not be done. It means "I think it can't be done, don't waste your time".
>>
>> If, instead of that, you say "I can't help, but feel free to go ahead", the reaction of the community will be completely different.
>>
>> Cheers,
>> Juan Vuletich
>>
>
>
>



--
Best regards,
Igor Stasenko AKA sig.

Reply | Threaded
Open this post in threaded view
|

Re: Issue 3266 in pharo: Clean TextEditor

Juan Vuletich-4
In reply to this post by Stéphane Ducasse
Stéphane Ducasse wrote:
> Indeed. I mean
> I can't help, but feel free to go ahead
>
> Stef
>
>  
Great! Thanks for the clarification.

Cheers,
Juan Vuletich

Reply | Threaded
Open this post in threaded view
|

Re: Issue 3266 in pharo: Clean TextEditor

Juan Vuletich-4
In reply to this post by Igor Stasenko
Igor Stasenko wrote:
> IMO SimpleMorphic, minus 'color' ivar in Morph class is a way to go.
> (i mean get rid that 'color' thing in base class)  :)
>
>  

Yep. I fully agree.

Cheers,
Juan Vuletich

Reply | Threaded
Open this post in threaded view
|

Re: Issue 3266 in pharo: Clean TextEditor

Igor Stasenko
On 25 November 2010 03:45, Juan Vuletich <[hidden email]> wrote:
> Igor Stasenko wrote:
>>
>> IMO SimpleMorphic, minus 'color' ivar in Morph class is a way to go.
>> (i mean get rid that 'color' thing in base class)  :)
>>
>>
>
> Yep. I fully agree.
>
I think, for things like Morphic, where there are dozens of various
properties expected
to be in basic morph class, we should use dictionaries, backed up by
clever VM semantics.
I cannot stop thinking about extending object format to allow a
dynamic number of slots per object,
like in self or javascript.


> Cheers,
> Juan Vuletich
>
>



--
Best regards,
Igor Stasenko AKA sig.

Reply | Threaded
Open this post in threaded view
|

Re: Issue 3266 in pharo: Clean TextEditor

Juan Vuletich-4
Igor Stasenko wrote:
> I think, for things like Morphic, where there are dozens of various
> properties expected
> to be in basic morph class, we should use dictionaries, backed up by
> clever VM semantics.
> I cannot stop thinking about extending object format to allow a
> dynamic number of slots per object,
> like in self or javascript.
>
>  

That sounds interesting, and I'm sure there are many applications that
could benefit from it.

Cheers,
Juan Vuletich

Reply | Threaded
Open this post in threaded view
|

Re: Issue 3266 in pharo: Clean TextEditor

Stéphane Ducasse
In reply to this post by Juan Vuletich-4
this is just for state. You have traits for methods.
For me the way to go is first clean the classbuilder, have first class instance variables,
what you want is just
                hashSubclass:
                instead of variableSubclass:
Stef

>
> IMO SimpleMorphic, minus 'color' ivar in Morph class is a way to go.
>>> (i mean get rid that 'color' thing in base class)  :)
>>>
>>>
>>
>> Yep. I fully agree.
>>
> I think, for things like Morphic, where there are dozens of various
> properties expected
> to be in basic morph class, we should use dictionaries, backed up by
> clever VM semantics.
> I cannot stop thinking about extending object format to allow a
> dynamic number of slots per object,
> like in self or javascript.
>
>
>> Cheers,
>> Juan Vuletich
>>
>>
>
>
>
> --
> Best regards,
> Igor Stasenko AKA sig.
>


Reply | Threaded
Open this post in threaded view
|

Re: Issue 3266 in pharo: Clean TextEditor

Stéphane Ducasse
In reply to this post by Stéphane Ducasse
Now I will see who will port/rewrite what we have on top of a cleaned SimpleMorphic.


> Stéphane Ducasse wrote:
>> Indeed. I mean
>> I can't help, but feel free to go ahead
>>
>> Stef
>>
>>  
> Great! Thanks for the clarification.
>
> Cheers,
> Juan Vuletich
>


Reply | Threaded
Open this post in threaded view
|

Re: Issue 3266 in pharo: Clean TextEditor

Eliot Miranda-2
In reply to this post by Igor Stasenko


On Wed, Nov 24, 2010 at 6:10 PM, Igor Stasenko <[hidden email]> wrote:
On 25 November 2010 03:45, Juan Vuletich <[hidden email]> wrote:
> Igor Stasenko wrote:
>>
>> IMO SimpleMorphic, minus 'color' ivar in Morph class is a way to go.
>> (i mean get rid that 'color' thing in base class)  :)
>>
>>
>
> Yep. I fully agree.
>
I think, for things like Morphic, where there are dozens of various
properties expected
to be in basic morph class, we should use dictionaries, backed up by
clever VM semantics.
I cannot stop thinking about extending object format to allow a
dynamic number of slots per object,
like in self or javascript.

But you don't need to change the object model for this.  The VM still supports the mixed format of named and indexed inst vars (e.g. MethodContext) and so you can make an object indexed and store properties in indexed inst vars (e.g. as flattened key, value pairs).  The down side is the cost of become, but one can over-allocate, e.g. room for 8 properties so that one becomes only every 8th add.  Alternatively have an uninitialized properties inst var which when needed holds a dictionary.  Or a global weak dictionary. Or...  And of course all these variants can be mixed, e.g. by providing a settable policy in the class so different classes ca do different things.  The global properties dictionary can be used as a fallback when one wants to add a property to an instance of a class that doesn't implement one of the other policies.

best
Eliot


> Cheers,
> Juan Vuletich
>
>



--
Best regards,
Igor Stasenko AKA sig.


12