Re: Issue 1424 in pharo: Comparing a MethodProperties analogousCodeTo: an AdditionalMethodState is broken

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

Re: Issue 1424 in pharo: Comparing a MethodProperties analogousCodeTo: an AdditionalMethodState is broken

Marcus Denker-3

On Nov 20, 2009, at 9:57 PM, stephane ducasse wrote:

>
>
> Begin forwarded message:
>
>> From: [hidden email]
>> Date: November 20, 2009 8:08:46 PM GMT+01:00
>> To: [hidden email]
>> Subject: Re: Issue 1424 in pharo: Comparing a MethodProperties analogousCodeTo: an AdditionalMethodState is broken
>>
>>
>> Comment #10 on issue 1424 by nicolas.cellier.aka.nice: Comparing a MethodProperties analogousCodeTo: an AdditionalMethodState is broken
>> http://code.google.com/p/pharo/issues/detail?id=1424
>>
>> I mean you should
>> - correct ScriptLoader>>#cleanUpMethods
>> - then execute (ScriptLoader new cleanUpMethods) in an update stream to get rid of
>> these old MethodProperties

Ok, so the #cleanUpMethods is broken right now. And there is a problem with some Service thing holding on a method.
(we need reset Services in cleanUpMethods, too).

>>
>> Then remove or deprecate MethodProperties (classify in backward compatibility or
>> something...).
>> Don't know if you can simply remove it, or if some binary file outs of CompiledMethod
>> require it. Ask Marcus.

No, properties where used just internally for Pragmas. And those work. The other use of properties was
in Persephone/Reflectivity (http://scg.unibe.ch/research/reflectivity) and in Bytesurgeon (http://scg.unibe.ch/research/bytesurgeon).

Both of which will need some change (fort the better!)  to work on Pharo anyway. So no problem.

We should remove MethodProperty.

I will add report to the tracker for this.

        Marcus

--
Marcus Denker  -- http://www.marcusdenker.de


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