PATCHES - Countdown for July 7th

2019-07-07 Thread James

Hello,

Here is the current patch countdown list. The next countdown will be on 
July 9th


A quick synopsis of all patches currently in the review process can be 
found here:


http://philholmes.net/lilypond/allura/




 Push:

5524 New function css-color (accompanying x11-color) - Urs Liska
https://sourceforge.net/p/testlilyissues/issues/5524
http://codereview.appspot.com/580780043

5512 Use new syntax consistently throughout docs and examples - Valentin 
Villenave

https://sourceforge.net/p/testlilyissues/issues/5512
http://codereview.appspot.com/572640043


 Countdown:


 Review:

5531 NR: 2.10 Arabic Music - inlcude references to hel-arabic.ly - James 
Lowe

https://sourceforge.net/p/testlilyissues/issues/5531
http://codereview.appspot.com/560790043

5520 Improvements to \ambitusAfter - Malte Meyn
https://sourceforge.net/p/testlilyissues/issues/5520
http://codereview.appspot.com/552740043

5303 layout-set-staff-space confuses chord note head placement in a 
certain situation - James Lowe

https://sourceforge.net/p/testlilyissues/issues/5303
http://codereview.appspot.com/570830043

4362 Completion_heads_engraver and dotted breve - Malte Meyn
https://sourceforge.net/p/testlilyissues/issues/4362
http://codereview.appspot.com/560810043


 New:

No new patches at this time.



***

Regards,


James
___
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel


Re: stem.cc - issue 5303 - misplaced notehead (issue 570830043 by pkxgnugi...@runbox.com)

2019-07-07 Thread Carl . D . Sorensen

Perhaps this patch should also revert
87eb2f9fe1be3a532675fe4b7322bbba5a60ba5c

since that patch was a workaround, rather than a real fix, as
demonstrated during this troubleshooting thread.

Carl


https://codereview.appspot.com/570830043/

___
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel


Re: stem.cc - issue 5303 - misplaced notehead (issue 570830043 by pkxgnugi...@runbox.com)

2019-07-07 Thread lilypond

On 2019/07/07 17:32:35, Lukas-Fabian Moser wrote:

> The last patch reverts the rounding in stem.cc. stem.cc should be

taken from

the
> first patch set.



No, this was intentional: I agreed with David K that keeping the

calculations in

floats would be the most convincing solution. (It's not a revert: The

original

state was an (int) cast, which I first replaced by rounding+int-cast

and now

(finally?) by keeping the original float value.


Sorry, I hadn’t noticed that the cast was gone.

https://codereview.appspot.com/570830043/
___
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel


Re: stem.cc - issue 5303 - misplaced notehead (issue 570830043 by pkxgnugi...@runbox.com)

2019-07-07 Thread lukasfabianmoser



The last patch reverts the rounding in stem.cc. stem.cc should be

taken from the

first patch set.


No, this was intentional: I agreed with David K that keeping the
calculations in floats would be the most convincing solution. (It's not
a revert: The original state was an (int) cast, which I first replaced
by rounding+int-cast and now (finally?) by keeping the original float
value.

https://codereview.appspot.com/570830043/

___
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel


Re: stem.cc - issue 5303 - misplaced notehead (issue 570830043 by pkxgnugi...@runbox.com)

2019-07-07 Thread lilypond

On 2019/07/07 16:31:47, lilypond-pkx wrote:

With changes from Lukas based on dev's comments


The last patch reverts the rounding in stem.cc. stem.cc should be taken
from the first patch set.

https://codereview.appspot.com/570830043/

___
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel


Re: NR: 2.10 Arabic Music - inlcude references to hel-arabic.ly (issue 560790043 by pkxgnugi...@runbox.com)

2019-07-07 Thread hassan . elfatihi



Hello everyone 
About hel-arabic.ly 
It seems to me that the example given is incomplete 
You must write 


\include "hel-arabic.ly" 

\relative { 
\key c \rast 
c' d edb f | g a bdb c | c bb a g | f d c 
} 

Best regards 

hassan el fatihi 
___
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel