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 > >
