On 2012/01/20 14:08:18, MikeSol wrote:

http://codereview.appspot.com/5440084/diff/11001/lily/rhythmic-music-iterator.cc
File lily/rhythmic-music-iterator.cc (right):


http://codereview.appspot.com/5440084/diff/11001/lily/rhythmic-music-iterator.cc#newcode62
lily/rhythmic-music-iterator.cc:62: if (scm_is_true
ly_lily_module_constant should only be called for things that don't
exist in
C++.

I've seen that elsewhere.  Maybe to support redefining the function
before it is first memoized.

Otherwise, it's better to use the C++ function.  In this case:
ly_is_listened_event_class
(in translator.cc)

Will do.

http://codereview.appspot.com/5440084/

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

Reply via email to