Doc: Added @knownissue to Grace Notes for MIDI

Doc: Added @knownissue to Grace Notes for MIDI

As per thread

http://article.gmane.org/gmane.comp.gnu.lilypond.general/63450

When using MIDI with grace notes some manual tweaking made be required to
get the 'correct' length of the grace note to be played.

[Carl, 2011-04-07 17:48:37]

Looks generally good, but I think we need to include the error message text in
the @knownissue.

Thanks,

Carl

http://codereview.appspot.com/4388041/diff/1/Documentation/notation/rhythms.i...
File Documentation/notation/rhythms.itely (right):

http://codereview.appspot.com/4388041/diff/1/Documentation/notation/rhythms.i...
Documentation/notation/rhythms.itely:3241: 
I think the known issue should include the warning; something like "If the total
length of the grace notes is greater than the duration of the preceding note,
the error message "Going back in MIDI time" will be generated. "  That will give
the user the clue about their particular music, I think.

[Trevor Daniels, 2011-04-07 22:31:25]

Suggested rewording, accommodating Carl's suggestion.

Otherwise, looks fine.

Trevor

http://codereview.appspot.com/4388041/diff/1/Documentation/notation/rhythms.i...
File Documentation/notation/rhythms.itely (right):

http://codereview.appspot.com/4388041/diff/1/Documentation/notation/rhythms.i...
Documentation/notation/rhythms.itely:3239: duration. So some manual tweaking may
be needed to fit in time after the
duration, and this is stolen from the preceding note.  [Here follows Carl's
sentence]  To avoid this problem, the grace notes may be made shorter in
duration:

[pkx166h, 2011-04-08 16:45:56]

On 2011/04/07 22:31:25, Trevor Daniels wrote:
> Suggested rewording, accommodating Carl's suggestion.
> 
> Otherwise, looks fine.
> 
I did what was suggested and then re-worded that.

So it's 'done' but not exactly as suggested. I think this is better now.

Thanks for the comments.

[Trevor Daniels, 2011-04-08 23:25:32]

Nearly there :)

http://codereview.appspot.com/4388041/diff/4001/Documentation/notation/rhythm...
File Documentation/notation/rhythms.itely (right):

http://codereview.appspot.com/4388041/diff/4001/Documentation/notation/rhythm...
Documentation/notation/rhythms.itely:3240: length of the preceding note a
@emph{"Going back in MIDI time"} error
use @qq{}

http://codereview.appspot.com/4388041/diff/4001/Documentation/notation/rhythm...
Documentation/notation/rhythms.itely:3243: Some manual tweaking will be needed
to correct this. Either make the
"tweaking" has a special meaning in LP, which makes this use
potentially misleading and/or confusing.

[Graham Percival (old account), 2011-04-08 23:29:07]

http://codereview.appspot.com/4388041/diff/4001/Documentation/notation/rhythm...
File Documentation/notation/rhythms.itely (right):

http://codereview.appspot.com/4388041/diff/4001/Documentation/notation/rhythm...
Documentation/notation/rhythms.itely:3240: length of the preceding note a
@emph{"Going back in MIDI time"} error
On 2011/04/08 23:25:32, Trevor Daniels wrote:
> use @qq{}

Actually, I suggest using @qq{@code{Going back in MIDI time}}.  There's no
reason to use an @emph{} here!

[pkx166h, 2011-04-09 16:19:42]

Third Draft.

Regards

http://codereview.appspot.com/4388041/diff/4001/Documentation/notation/rhythm...
File Documentation/notation/rhythms.itely (right):

http://codereview.appspot.com/4388041/diff/4001/Documentation/notation/rhythm...
Documentation/notation/rhythms.itely:3240: length of the preceding note a
@emph{"Going back in MIDI time"} error
On 2011/04/08 23:25:32, Trevor Daniels wrote:
> use @qq{}

Done.

http://codereview.appspot.com/4388041/diff/4001/Documentation/notation/rhythm...
Documentation/notation/rhythms.itely:3243: Some manual tweaking will be needed
to correct this. Either make the
On 2011/04/08 23:25:32, Trevor Daniels wrote:
> "tweaking" has a special meaning in LP, which makes this use
> potentially misleading and/or confusing.

Done.

[Trevor Daniels, 2011-04-09 17:43:44]

LGTM

Trevor

[Graham Percival (old account), 2011-04-09 23:39:44]

LGTM

[pkx166h, 2011-04-10 13:12:40]

Pushed as commit	53390e0570f4544c131e77323cdc4f4ea738bddd