I have a problem with 11126

hi igor


up to 11125 it works well and after when I was trying to do the last two cs using MC

in 11125 I get

new
        ^ self trailerClass basicNew initialize
trailerClass
        ^ TrailerClass ifNil: [ OldMethodTrailer ]

in 11126 when I try to load these packages.

I get  CompiledMethodTrailer new
                         does understand trailerClass






Stef (now I have to run downtown with my boys and I will come back in 2hours or so).

CompiledMethodTrailer class(Object)>>doesNotUnderstand: #trailerClass
CompiledMethodTrailer class>>new
CompiledMethod>>trailer
CompiledMethod>>getSourceFor:in:
Metaclass(Behavior)>>sourceCodeAt:ifAbsent:
Metaclass(Behavior)>>ultimateSourceCodeAt:ifAbsent:
Browser(CodeHolder)>>sourceStringPrettifiedAndDiffed
Browser>>selectedMessage
Browser>>contents
PluggableTextMorph>>getText
PluggableTextMorph>>update:
[] in Browser(Object)>>changed:
DependentsArray>>do:
Browser(Object)>>changed:
Browser(Object)>>contentsChanged
Browser(CodeHolder)>>contentsChanged
Browser>>messageListIndex:
PluggableListMorph>>changeModelSelection:
PluggableListMorph>>mouseUp:
PluggableListMorph(Morph)>>handleMouseUp:
MouseButtonEvent>>sentTo:
PluggableListMorph(Morph)>>handleEvent:
MorphicEventDispatcher>>dispatchDefault:with:
MorphicEventDispatcher>>dispatchEvent:with:
PluggableListMorph(Morph)>>processEvent:using:
PluggableListMorph(Morph)>>processEvent:
PluggableListMorph>>handleFocusEvent:
[] in HandMorph>>sendFocusEvent:to:clear:
[] in PasteUpMorph>>becomeActiveDuring:
BlockClosure>>on:do:
PasteUpMorph>>becomeActiveDuring:
HandMorph>>sendFocusEvent:to:clear:
HandMorph>>sendEvent:focus:clear:
HandMorph>>sendMouseEvent:
HandMorph>>handleEvent:
HandMorph>>processEvents
[] in WorldState>>doOneCycleNowFor:
Array(SequenceableCollection)>>do:
WorldState>>handsDo:
_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project

2009/12/30 Stéphane Ducasse <[hidden email]>:
You seem stepped on same trap as Andreas did.  :)
MC minds own method removal order,
and removes #trailerClass before #new,
which then leads to DNU.

Split cleanup into two stages:
- in first update, remove just #new
- in second , remove #trailerClass and the rest of stuff

so, MC will have no chances to interfere with removal order.

--
Best regards,
Igor Stasenko AKA sig.

_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project

I'm back from christmas shpping mess.
ok I will do it.


On Dec 30, 2009, at 5:20 PM, Igor Stasenko wrote:


_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project

Igor I push the latest bit out.
I did not remove trailerClass because the classVariable was still there.
Let me know if it should be there or not (not time to check - wife shouting for diner :)

Stef
On Dec 30, 2009, at 6:10 PM, Stéphane Ducasse wrote:


_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project

2009/12/30 Stéphane Ducasse <[hidden email]>:
> Igor I push the latest bit out.
> I did not remove trailerClass because the classVariable was still there.
> Let me know if it should be there or not (not time to check - wife shouting for diner :)
>
Once you remove #new, you can remove everything else in any order you
prefer, including trailerClass & classvar.



--
Best regards,
Igor Stasenko AKA sig.

_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project

Ok this is strange that the classVar was not removed in the cleanup.
Anyway I will do it. this is cool to have such part of the system cleaned that way.

Stef




_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project