On 26 juin 2013, at 12:54, Esteban Lorenzano wrote:
Hi
I finished the integration of the new class organizer, and I need people who can test it before integration. Please, merge the SLICE 6757
Tested and approved señor! Only one question why the AllProtocol is a subclass of Protocol? As I understand it, AllProtocol shouldn't respond to addMethod: removeMethod: ... and it shouldn't even have a methods instance variable. Why don't make an AbstractProtocol with two subclass Protocol and VirtualProtocol and make AllProtocol a subclass of VirtualProtocol. Like this, in the future we could for example add a new SmartProtocol (subclass of VirtualProtocol) that automagically categorize methods given some predicate.
https://pharo.fogbugz.com/f/cases/6757/
and after that execute this script:
Smalltalk allClassesAndTraitsDo: [:c || org proOrg | org := c theNonMetaClass organization. proOrg := ClassOrganization importFrom: org. c theNonMetaClass organization: proOrg.
org := c theMetaClass organization. proOrg := ClassOrganization importFrom: org. c theMetaClass organization: proOrg ].
This is an important step not just to have a better class organizer (current is pre-existance of dictionaries... so... :P)
Ah ok this is why it doesn't use Dictionary! I was wondering about the rationale behind the old organizer ;).
It is important also for the next step in RPackage stuff.
Esteban