Issue 7145 in pharo: simplify MethodContext>>#tempNamed:

Status: Accepted
Owner: [hidden email]
Labels: Type-Cleanup Milestone-2.0

New issue 7145 by [hidden email]: simplify MethodContext>>#tempNamed:
http://code.google.com/p/pharo/issues/detail?id=7145

... do not call

self tempNames indexOf: aName

two times  but use the variable instead. (it does not change in between the  
calls)


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

Updates:
        Status: FixReviewNeeded

Comment #1 on issue 7145 by [hidden email]: simplify  
MethodContext>>#tempNamed:
http://code.google.com/p/pharo/issues/detail?id=7145

Name: SLICE-Issue-7145-simplify-MethodContextgtgttempNamed-MarcusDenker.1
Author: MarcusDenker
Time: 19 December 2012, 11:21:57.192 am
UUID: dff560aa-fed8-4c38-b6be-73db40087325
Ancestors:
Dependencies: Kernel-MarcusDenker.1268

Issue 7145: simplify MethodContext>>#tempNamed:


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

Updates:
        Status: FixToInclude

Comment #2 on issue 7145 by [hidden email]: simplify  
MethodContext>>#tempNamed:
http://code.google.com/p/pharo/issues/detail?id=7145

The slice contains too many changes: in Behavior, BlockClosure, Class,  
ClassDescription, False, Integer, Job, Metaclass and True.
There is a new slice.




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

Comment #3 on issue 7145 by [hidden email]: simplify  
MethodContext>>#tempNamed:
http://code.google.com/p/pharo/issues/detail?id=7145

I did the slice just like I always do them.

So there is something broken.


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

Updates:
        Status: Integrated

Comment #4 on issue 7145 by [hidden email]: simplify  
MethodContext>>#tempNamed:
http://code.google.com/p/pharo/issues/detail?id=7145

in 2.0 444


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