Aldo and I seat down during the Pharo Sprint on Saturday. We had fun.  
Below what we did.


ISSUE #262 (fixed)
Doits are now executed by primtive. This means, a doit method is  
*never* installed in a class.
We removed all now unnecessary checks of isDoit

- Behavior>>allLocalCallsOn:
- ChangeSet>>atSelector: selector class: class put: changeType
- MethodReference>>isValid
- PCCByCompilation>>methodsWithDisabledCall
- PackageInfo>>methods
- SystemNavigation>>browseAllCallsOn: aLiteral localTo: aClass

Pair programmed with Aldo Mettini

Slice_262_isDoIt-Alexandre_Bergel.1 in Pharo Inbox

-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=
ISSUE #1486 (fixed)

Added BehaviorTest>>testAllLocalCallsOn

Pair programmed with Aldo Mettini during the Pharo Sprint @ Smalltalks  
2009

-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=

ISSUE #1488 (new)

Dictionary is not a superclass of SmallDictionary, and   
SmallDictionary is not a subclass of Dictionary. The only common  
superclass of these two classes is Collection. Putting  
unreferencedKeys in Collection does not look too good as well.  
Therefore, unreferencedKeys looks like to be a good candidate for  
being shared by a trait.

Pair programmed with Aldo Mettini during the Pharo Sprint @ Smalltalks  
2009

-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=

ISSUE #261 (fix)

Even though allCallsOn: implies something about "calls" which seems to  
be related to sends, it actually even is used for references to classes.

We added a SystemNavigation>>allReferencesTo: aLiteral that returns  
the list of methods referencing aLiteral.
We simplified allCallsOn: since now it acts on symbols only. It raises  
a deprecation warning if a binding is provided to allCallsOn:. We  
refactored all callers of allCallsOn: and wrote some tests (for the  
one that was easy to write): BehaviorTest>>testAllCallsOn

Pair programmed with Aldo Mettini during the Pharo Sprint @ Smalltalks  
2009

-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=

Cheers,
Alexandre
-- 
_,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:
Alexandre Bergel  http://www.bergel.eu
^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.






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

Reply via email to