lilypond-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Add lilypond version predicates/operators (issue 317270043 by addres


From: git
Subject: Re: Add lilypond version predicates/operators (issue 317270043 by address@hidden)
Date: Tue, 14 Feb 2017 10:20:56 -0800


https://codereview.appspot.com/317270043/diff/20001/scm/lily-library.scm
File scm/lily-library.scm (right):

https://codereview.appspot.com/317270043/diff/20001/scm/lily-library.scm#newcode899
scm/lily-library.scm:899: (let* ((a (car v1)) (b (car v2))
On 2017/02/14 17:29:33, dak wrote:
Requires v1/v2 to have at least one element.  Admittedly quite a
reasonable
precondition.

OIf we have that, we don't need (op 0 0) but can use (op a a) instead.
 The
advantage is that we are no longer bound to _numeric_ comparisons
then.  This
lexicographic compare will then also work with string= and similar.

Acknowledged.

https://codereview.appspot.com/317270043/diff/20001/scm/lily-library.scm#newcode903
scm/lily-library.scm:903: ((not axb) #f)
On 2017/02/14 17:29:33, dak wrote:
This will let (version-compare? '(2 18) < '(2 19)) deliver #f after
comparing
only 2 with 2, right?

Ah yes. I had removed the
  (and (eq? op =)
part, which I shouldn't have done

https://codereview.appspot.com/317270043/



reply via email to

[Prev in Thread] Current Thread [Next in Thread]