On 2015/08/26 08:55:14, dak wrote:
https://codereview.appspot.com/257580043/diff/1/scm/lily-library.scm
File scm/lily-library.scm (right):


https://codereview.appspot.com/257580043/diff/1/scm/lily-library.scm#newcode793
scm/lily-library.scm:793: char-set:whitespace))
A mostly theoretic musing: in the predicates documented in NR, we have
`char-whitespace?' listed in "R5RS secondary predicates".

I don't know how it's implemented, but if people compare results,
char-whitespace? might be the expected point of reference and it's not
like this
is performance-critical.

Using that would mean one less function in our codebase for the same
functionality.

I just thought I'd mention it: it's by no means a necessary change or
even a
showstopper.

Well, I could use `char-whitespace?' to write a different definition.
Ofcourse a new definition would still be there.
I've the feeling I don't really understand your proposal...

https://codereview.appspot.com/257580043/

_______________________________________________
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel

Reply via email to