[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Make a PostEvents container class for packaging several postevents (
From: |
ianhulin44 |
Subject: |
Re: Make a PostEvents container class for packaging several postevents (issue 7742044) |
Date: |
Sun, 24 Mar 2013 13:16:20 +0000 |
On 2013/03/16 20:51:05, dak wrote:
On 2013/03/16 19:19:57, lemzwerg wrote:
> LGTM. The combination `\' `\n' `\(' is probably worth a comment in
the
> contributors' manual.
If \( was defined, things would be so much easier: one would just put
\( in the
first column. \ <newline> \n( is the best I could manage. Or reformat
in order
to get the ( away from the starts of line.
Total nuisance.
LGTM apart from a nitpick with the actual doc-string in
define-music-types.scm. You probably got distracted by all the
formatting stuff.
https://codereview.appspot.com/7742044/