Re: [PATCH] functional-or returned #f for (functional-or #f #t #f). Fixed.

2009-11-22 Thread Patrick McCarty
On Tue, Nov 17, 2009 at 11:24 PM, David Kastrup d...@gnu.org wrote: Patrick McCarty pnor...@gmail.com writes: On 2009-11-16, David Kastrup wrote: It still suffers from not doing short-circuit evaluation.  For readability and efficiency, I'd really prefer replacing '(apply functional-or'

Re: [PATCH] functional-or returned #f for (functional-or #f #t #f). Fixed.

2009-11-22 Thread David Kastrup
Patrick McCarty pnor...@gmail.com writes: On Tue, Nov 17, 2009 at 11:24 PM, David Kastrup d...@gnu.org wrote: Patrick McCarty pnor...@gmail.com writes: On 2009-11-16, David Kastrup wrote: It still suffers from not doing short-circuit evaluation.  For readability and efficiency, I'd really

Re: [PATCH] functional-or returned #f for (functional-or #f #t #f). Fixed.

2009-11-17 Thread Patrick McCarty
On 2009-11-16, David Kastrup wrote: Patrick McCarty pnor...@gmail.com writes: Thanks, pushed. Thanks. The only caller (this is apparently in local namespace) as far as I can see is (define-public (fret-number-tablature-format string context event) (let* ((tuning

Re: [PATCH] functional-or returned #f for (functional-or #f #t #f). Fixed.

2009-11-17 Thread David Kastrup
Patrick McCarty pnor...@gmail.com writes: On 2009-11-16, David Kastrup wrote: It still suffers from not doing short-circuit evaluation. For readability and efficiency, I'd really prefer replacing '(apply functional-or' with '(any' It looks like Neil removed the is-harmonic binding

Re: [PATCH] functional-or returned #f for (functional-or #f #t #f). Fixed.

2009-11-16 Thread David Kastrup
Patrick McCarty pnor...@gmail.com writes: Thanks, pushed. Thanks. The only caller (this is apparently in local namespace) as far as I can see is (define-public (fret-number-tablature-format string context event) (let* ((tuning (ly:context-property context 'stringTunings)) (pitch

[PATCH] functional-or returned #f for (functional-or #f #t #f). Fixed.

2009-11-15 Thread David Kastrup
--- scm/lily-library.scm |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/scm/lily-library.scm b/scm/lily-library.scm index 4a7973a..5853744 100644 --- a/scm/lily-library.scm +++ b/scm/lily-library.scm @@ -273,7 +273,7 @@ (define (functional-or . rest) (if (pair?

Re: [PATCH] functional-or returned #f for (functional-or #f #t #f). Fixed.

2009-11-15 Thread Patrick McCarty
On 2009-11-15, David Kastrup wrote: --- scm/lily-library.scm |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/scm/lily-library.scm b/scm/lily-library.scm index 4a7973a..5853744 100644 --- a/scm/lily-library.scm +++ b/scm/lily-library.scm @@ -273,7 +273,7 @@