On 15 Nov 2013, at 14:07, Roberto Minelli <[email protected]> wrote:

> Hi,
> 
> I looked at the implementation of #linesOfCode of ClassDescription. It is 
> actually wrong, a correct version should be:
> 
> linesOfCode 
>       "An approximate measure of lines of code. Includes comments, but 
> excludes blank lines."
>       | lines |
>       
>       lines := self localMethods inject: 0 into: [:sum :each | sum + each 
> linesOfCode]. 
>       ^ self isMeta not
>               ifTrue: [lines]
>               ifFalse: [lines + self class linesOfCode]
> 

indeed!

and after fixing, Pharo3 shrinks from 444kLoc to 368KLoc :-)

        Marcus

Reply via email to