https://codereview.appspot.com/13256053/diff/1/lily/lexer.ll
File lily/lexer.ll (right):
https://codereview.appspot.com/13256053/diff/1/lily/lexer.ll#newcode588
lily/lexer.ll:588: [^|*.=$#{}\"\\ \t\n\r\f0-9][^$#{}\"\\ \t\n\r\f0-9]* {
On 2013/09/24 16:15:39, dak wrote:
On 2013/09/24 07:44:44, janek wrote:
> Hmm. From my point of view, this deserves some comment (but i don't
insist).
I have a problem thinking of a comment that would add any information
that is
not immediately apparent from the pattern.
Suggestions?
Maybe "because of regex greediness, we have to additionally exclude
patterns beginning with |*.= (without this \lyricmode { \skip 1.*3 }
fails)"?
For me the regex itself required a few moments of thinking to
understand.
Janek
https://codereview.appspot.com/13256053/
_______________________________________________
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel
