doc: changing example for overriding positions

doc: changing example for overriding positions

example in LM 4.5.2 (fixing beam collision)
is outdated because beam collision has been
implemented, so there is nothing to fix.
Therefore I change the type of collision
to beam-tie collision.

[Graham Percival (old account), 2011-06-12 13:33:45]

thanks, added as
http://code.google.com/p/lilypond/issues/detail?id=1689

[Graham Percival (old account), 2011-06-12 13:52:11]

LGTM.  Could you send me the final patch for pushing?

[Neil Puttock, 2011-06-12 14:09:09]

http://codereview.appspot.com/4607043/diff/1/Documentation/learning/tweaks.itely
File Documentation/learning/tweaks.itely (right):

http://codereview.appspot.com/4607043/diff/1/Documentation/learning/tweaks.it...
Documentation/learning/tweaks.itely:2801: \time 4/2
judging by the bar checks, I think you mean 4/4

http://codereview.appspot.com/4607043/diff/1/Documentation/learning/tweaks.it...
Documentation/learning/tweaks.itely:2803: { c'1 ~ c'2. e'8 f'}
space before }

http://codereview.appspot.com/4607043/diff/1/Documentation/learning/tweaks.it...
Documentation/learning/tweaks.itely:2829: \override Beam #'positions = #'(-1 .
-1)
please put this inside the sequential block:

{
  \override Beam #'positions = #'(-1 . -1)
  e''8 ...
}

http://codereview.appspot.com/4607043/diff/1/Documentation/learning/tweaks.it...
Documentation/learning/tweaks.itely:2842: the second measure of quavers, but not
to any of the beams in the
of eighth notes,

[Janek Warchol, 2011-06-13 11:35:42]

Style problems corrected.
Thanks for review, Neil!

http://codereview.appspot.com/4607043/diff/1/Documentation/learning/tweaks.itely
File Documentation/learning/tweaks.itely (right):

http://codereview.appspot.com/4607043/diff/1/Documentation/learning/tweaks.it...
Documentation/learning/tweaks.itely:2801: \time 4/2
On 2011/06/12 14:09:09, Neil Puttock wrote:
> judging by the bar checks, I think you mean 4/4

Sorry, barchecks were wrong. It was 4/4 at first, but should be 4/2.

http://codereview.appspot.com/4607043/diff/1/Documentation/learning/tweaks.it...
Documentation/learning/tweaks.itely:2803: { c'1 ~ c'2. e'8 f'}
On 2011/06/12 14:09:09, Neil Puttock wrote:
> space before }

Done.

http://codereview.appspot.com/4607043/diff/1/Documentation/learning/tweaks.it...
Documentation/learning/tweaks.itely:2829: \override Beam #'positions = #'(-1 .
-1)
On 2011/06/12 14:09:09, Neil Puttock wrote:
> please put this inside the sequential block:
> 
> {
>   \override Beam #'positions = #'(-1 . -1)
>   e''8 ...
> }

Done.

http://codereview.appspot.com/4607043/diff/1/Documentation/learning/tweaks.it...
Documentation/learning/tweaks.itely:2842: the second measure of quavers, but not
to any of the beams in the
On 2011/06/12 14:09:09, Neil Puttock wrote:
> of eighth notes,

Done.

[Janek Warchol, 2011-06-13 11:40:05]

2011/6/12  <percival.music.ca@gmail.com>:
> LGTM.  Could you send me the final patch for pushing?

Attached.

[Graham Percival (old account), 2011-06-14 12:10:30]

thanks, pushed.