Issue 4679 in pharo: numberOfLinesOfCode wrongly defined

Status: Accepted
Owner: [hidden email]
CC: [hidden email]
Labels: Difficulty-Easy Milestone-1.4

New issue 4679 by [hidden email]: numberOfLinesOfCode wrongly defined
http://code.google.com/p/pharo/issues/detail?id=4679

Pharo image: 14097

It is currently:
-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=
ClassDescription>>numberOfLinesOfCode
        "Return the number of lines of code"
       
        | str |
        str := String new writeStream.
        Object fileOutOn: str.
        ^ str contents lineCount
-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=

It should do a "self fileOutOn: str." instead




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

Comment #1 on issue 4679 by [hidden email]: numberOfLinesOfCode  
wrongly defined
http://code.google.com/p/pharo/issues/detail?id=4679

in http://ss3.gemstone.com/ss/PharoInbox


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

Comment #2 on issue 4679 by [hidden email]: numberOfLinesOfCode  
wrongly defined
http://code.google.com/p/pharo/issues/detail?id=4679

Marcus just showed me linesOfCode, defined on ClassDescription. This method  
should be removed then.


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

Updates:
        Status: FixToInclude

Comment #3 on issue 4679 by [hidden email]: numberOfLinesOfCode  
wrongly defined
http://code.google.com/p/pharo/issues/detail?id=4679

attached is a removal of #numberOfLinesOfCode and the (amazingly) wrong test

Attachments:
        RemoveNumberOfLinesOfCode.1.cs  212 bytes


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

Updates:
        Status: Closed

Comment #4 on issue 4679 by [hidden email]: numberOfLinesOfCode  
wrongly defined
http://code.google.com/p/pharo/issues/detail?id=4679

in 14107


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