Issue 3875 in pharo: Merge minor NumberParser refactorings from Squeak

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

Issue 3875 in pharo: Merge minor NumberParser refactorings from Squeak

pharo
Status: Accepted
Owner: [hidden email]
Labels: Type-Squeak

New issue 3875 by [hidden email]: Merge minor NumberParser  
refactorings from Squeak
http://code.google.com/p/pharo/issues/detail?id=3875

There are three changed methods:
1) using peekSignIsMinus gives a chance to subclasses to parse a leading +
2) using ifNil: is faster than isNil ifTrue: (with current Compiler)
3) skip: -1 in an error case so as to insert the error message at the right  
place



Reply | Threaded
Open this post in threaded view
|

Re: Issue 3875 in pharo: Merge minor NumberParser refactorings from Squeak

pharo
Updates:
        Status: FixProposed

Comment #1 on issue 3875 by [hidden email]: Merge minor NumberParser  
refactorings from Squeak
http://code.google.com/p/pharo/issues/detail?id=3875

SLICE-Issue-3875-MinorNumberParserRefactoringsFromSqueak-nice.1
Dependencies: Kernel-nice.848




Reply | Threaded
Open this post in threaded view
|

Re: Issue 3875 in pharo: Merge minor NumberParser refactorings from Squeak

pharo
Updates:
        Status: Closed

Comment #2 on issue 3875 by [hidden email]: Merge minor NumberParser  
refactorings from Squeak
http://code.google.com/p/pharo/issues/detail?id=3875

in 13111