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/ _______________________________________________ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel