[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/
- Re: Add lilypond version predicates/operators (issue 317270043 by address@hidden), (continued)
- Re: Add lilypond version predicates/operators (issue 317270043 by address@hidden), git, 2017/02/14
- Re: Add lilypond version predicates/operators (issue 317270043 by address@hidden), paulwmorris, 2017/02/14
- Re: Add lilypond version predicates/operators (issue 317270043 by address@hidden), git, 2017/02/14
- Re: Add lilypond version predicates/operators (issue 317270043 by address@hidden), dak, 2017/02/14
- Re: Add lilypond version predicates/operators (issue 317270043 by address@hidden), dak, 2017/02/14
- Re: Add lilypond version predicates/operators (issue 317270043 by address@hidden), dak, 2017/02/14
- Re: Add lilypond version predicates/operators (issue 317270043 by address@hidden), git, 2017/02/14
- Re: Add lilypond version predicates/operators (issue 317270043 by address@hidden), dak, 2017/02/14
- Re: Add lilypond version predicates/operators (issue 317270043 by address@hidden), dak, 2017/02/14
- Re: Add lilypond version predicates/operators (issue 317270043 by address@hidden),
git <=
- Re: Add lilypond version predicates/operators (issue 317270043 by address@hidden), dak, 2017/02/14
- Re: Add lilypond version predicates/operators (issue 317270043 by address@hidden), thomasmorley65, 2017/02/14
- Re: Add lilypond version predicates/operators (issue 317270043 by address@hidden), thomasmorley65, 2017/02/14
- Re: Add lilypond version predicates/operators (issue 317270043 by address@hidden), thomasmorley65, 2017/02/14
- Re: Add lilypond version predicates/operators (issue 317270043 by address@hidden), paulwmorris, 2017/02/18
- Re: Add lilypond version predicates/operators (issue 317270043 by address@hidden), git, 2017/02/18
- Re: Add lilypond version predicates/operators (issue 317270043 by address@hidden), dak, 2017/02/18