Yes, it should. Could you also write a small test for this?

Cheers, Doru

On Wed, Aug 26, 2015 at 11:29 PM, stephan stephan@stack.nl wrote:

Shouldn't

FAMIXBehaviouralEntity>>computeNumberOfLinesOfCode self hasSourceAnchor ifTrue: [ ^ self sourceAnchor lineCount ].

^ (self sourceText lineCount = '')
        ifTrue: [ self notExistentMetricValue ]
        ifFalse: [ self sourceText lineCount ]

be

computeNumberOfLinesOfCode self hasSourceAnchor ifTrue: [ ^ self sourceAnchor lineCount ]. ^ (self sourceText = '') ifTrue: [ self notExistentMetricValue ] ifFalse: [ self sourceText lineCount ] _______________________________________________ Moose-dev mailing list Moose-dev@iam.unibe.ch https://www.iam.unibe.ch/mailman/listinfo/moose-dev