Prunes stem::length down to the bare minimum.

Prunes stem::length down to the bare minimum.

[MikeSol, 2011-09-17 10:03:45]

A response to Neil's e-mail concerning stem::length - this is a cleaner
implementation.

Cheers,
MS

[Neil Puttock, 2011-09-17 11:38:31]

http://codereview.appspot.com/5057041/diff/1/scm/output-lib.scm
File scm/output-lib.scm (right):

http://codereview.appspot.com/5057041/diff/1/scm/output-lib.scm#newcode82
scm/output-lib.scm:82: "stem::length called but will not be used for beamed
stem."))))
remove full stop/period

Have you found a case where this will be triggered?  If so, it still needs to
return a sane value rather than unspecified.

[MikeSol, 2011-09-17 11:50:59]

On 2011/09/17 11:38:31, Neil Puttock wrote:
> http://codereview.appspot.com/5057041/diff/1/scm/output-lib.scm
> File scm/output-lib.scm (right):
> 
> http://codereview.appspot.com/5057041/diff/1/scm/output-lib.scm#newcode82
> scm/output-lib.scm:82: "stem::length called but will not be used for beamed
> stem."))))
> remove full stop/period
> 
> Have you found a case where this will be triggered?  If so, it still needs to
> return a sane value rather than unspecified.

No, but you're right that it should return something.  I'll have it return 0.

Cheers,
MS

[pkx166h, 2011-09-17 13:10:05]

Passes make and reg tests