Re: Uses single algorithm for side-position spacing. (issue 6827072)

2012-11-30 Thread mike
On 30 nov. 2012, at 07:25, k-ohara5...@oco.net wrote: > 'finger-chords.ly' is still in disagreement with its texidoc (therefore > failing). You could adjust that regtest in light of the new defaults, > of course. > > Better might be to make Fingering.add-stem-support = #only-if-beamed > the new

Re: How to do proper indentation?

2012-11-30 Thread Marc Hohl
Am 30.11.2012 02:16, schrieb Thomas Morley: Hi, reviewing https://codereview.appspot.com/6850073/ Pál admonished to do better indentation on lists at different levels. Trying to minimize my lack of knowledge on this topic I searched the CG about It. CG 10.5.3 Indentation mentions possibilities

Use define-void-function rather than define-music-function in several places (issue 6854104)

2012-11-30 Thread benko . pal
LGTM https://codereview.appspot.com/6854104/ ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel

Re: Uses single algorithm for side-position spacing. (issue 6827072)

2012-11-30 Thread Keith OHara
On Fri, 30 Nov 2012 01:37:48 -0800, wrote: Ugh...this is a really nasty cyclic dependency. There is sometimes a cyclic dependency message, but that is not the problem. I'm pretty sure I used \stemUp to determine the stem directions and break this dependency chain. 1) note_head_x_shift 2)

Re: Uses single algorithm for side-position spacing. (issue 6827072)

2012-11-30 Thread m...@mikesolomon.org
On 30 nov. 2012, at 20:42, Keith OHara wrote: > On Fri, 30 Nov 2012 01:37:48 -0800, wrote: > >> Ugh...this is a really nasty cyclic dependency. > > There is sometimes a cyclic dependency message, but that is not the problem. > I'm pretty sure I used \stemUp to determine the stem directions

Re: markup-commands rest-by-number and rest (issue 6850073)

2012-11-30 Thread David Kastrup
thomasmorle...@googlemail.com writes: > On 2012/11/27 20:03:06, benko.pal wrote: > >> > Did you had a look on the compiled output of the new reg-tests? > >> no. could you push to a dev/ branch? > > Sorry. I'm still a newbie in devel tasks. If you have a branch where you checked in your changes,

Re: Uses single algorithm for side-position spacing. (issue 6827072)

2012-11-30 Thread k-ohara5a5a
Ugh...this is a really nasty cyclic dependency. Then you could use as a test-case which has no fingerings, no cyclic dependencies, but still shows the problem of confused beams. https://codereview.appspot.com/6827072/ __

git-cl error message

2012-11-30 Thread Aleksandr Andreev
Hello list, I am getting the following message when I try to run git cl upload origin/master: Traceback (most recent call last): File "/usr/bin/git-cl", line 888, in sys.exit(main(sys.argv)) File "/usr/bin/git-cl", line 882, in main return func(argv[2:]) File "/usr/bin/git-cl", lin

Re: Uses single algorithm for side-position spacing. (issue 6827072)

2012-11-30 Thread m...@mikesolomon.org
On 30 nov. 2012, at 22:14, k-ohara5...@oco.net wrote: >>> Ugh...this is a really nasty cyclic dependency. > > Then you could use as a test-case > > which has no fingerings, no cyclic dependencies, but still shows the > problem of confu

Re: How to do proper indentation?

2012-11-30 Thread Thomas Morley
2012/11/30 Marc Hohl : > Am 30.11.2012 02:16, schrieb Thomas Morley: >> >> Hi, >> >> reviewing >> https://codereview.appspot.com/6850073/ >> Pál admonished to do better indentation on lists at different levels. >> >> Trying to minimize my lack of knowledge on this topic I searched the >> CG about I