lilypond-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: line_count related patches in a single commit for review (issue 6419


From: Colin Campbell
Subject: Re: line_count related patches in a single commit for review (issue 6419064)
Date: Sun, 19 Aug 2012 23:30:51 -0600
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:14.0) Gecko/20120714 Thunderbird/14.0

On 12-08-17 06:14 AM, Benkő Pál wrote:
hi all,

Keith, I hope I fixed lyrics-bar.ly.

Yes, it comes out nicely.


1. in repeat-sign.ly the thick-lined staff has now the dots
    outside of staff, while the c++ version had it inside -
    there may be a difference how line-thickness is handled.

The latest Scheme version you sent uses the correct staff line thickness.
The last C version you posted used the layout line thickness, for bar lines
and span bars.
thanks for spotting, I use Marc's Scheme code now.

The gap you require to set the dot, dot-y-length + line-thickness + 2 * 0.75
line-thickness,
includes a lot of space for the gaps on each side of the dot, when the staff
lines are thick.  You might be happy with a rule to ensure a minimum
proportion of white-space  1.2( dot-y-length + line-thickness )
ok, I applied this (and did a bit of restructuring to keep
lyrics-bar.ly the way we like).
I also changed those regtests that show where's the change of dot placement
inside or outside staff.
I have now four commits: regtest changes plus one change
in three (sort of unrelated) functions each in bar-line.scm
(colon, extent and line-span computation).
what review process do you prefer/suggest?
one review per commit or review in one, push in four?

p




Sorry for being a bit thick, Pál, but I don't know which tracker item this patch belongs to. Also, would one of the developers have an opinion in answer to Pál's question: it would be good to get this on the Tuesday countdown for Thursday.

Cheers,
Colin

--
I've learned that you shouldn't go through life with a catcher's mitt on both 
hands.
You need to be able to throw something back.
-Maya Angelou, poet (1928- )




reply via email to

[Prev in Thread] Current Thread [Next in Thread]