Re: Issue 2149 in lilypond: Patch: Creates non-negative-integer? predicate.

[Ian Hulin]

On 29/12/11 11:13, address@hidden wrote:
> 
> Comment #6 on issue 2149 by address@hidden: Patch: Creates 
> non-negative-integer? predicate. 
> http://code.google.com/p/lilypond/issues/detail?id=2149
> 
> I'll change it to whatever people think would be easiest - I really
> have no preference, but I do think it's important to have a
> user-friendly predicate if LilyPond is to use the box-integral
> method to create vertical skylines for other objects.
Why not call it /positive-integer?/ and state in the docstring it defines
any integer i where i >= 0 as positive.
Or if you want to stick to Guiles /positive?/, maybe use
/positive-integer-or-zero?/.

Predicate names seem to work better if they describe what they're
testing for rather than what's being avoided.
<Bad pun alert>
It's better of they're positive . . .
</Bad pun alert>
Cheers,
Ian