PATCHES: Countdown for Septemeber 3rd - 06:00 GMT

2013-08-30 Thread James
hello *Countdown – September 3rd – 06:00 GMT* * * * * * * * * 3517http://code.google.com/p/lilypond/issues/detail?id=3517q=label%3APatch-reviewsort=patchcolspec=ID%20Type%20Status%20Stars%20Owner%20Patch%20Needs%20Summary%20Modified Enhancement David Kastrup Push Patch: Parse composite

Re: stencil-integral: use box extents specified in markup; issue 3255 (issue 9295044)

2013-08-30 Thread Mike Solomon
On 30 août 2013, at 07:49, Keith OHara k-ohara5...@oco.net wrote: On Thu, 29 Aug 2013 21:24:25 -0700, lemzw...@googlemail.com wrote: Conceptually, I prefer (1), but this is based on your descriptions and the previous discussion, not on understanding the code... Then look at (2) and see

Re: stencil-integral: use box extents specified in markup; issue 3255 (issue 9295044)

2013-08-30 Thread dak
On 2013/08/30 07:41:49, mike7 wrote: I'd still argue that (2) is the best way to go as it is a one-stop-shopping way to clear all these bugs (and perhaps more) as they arise. Since we are currently in the process of recovering from the last one-stop-shopping way to clear bugs galore which

Re: parser: more specific error messages; issue 3300 (issue 8506043)

2013-08-30 Thread dak
https://codereview.appspot.com/8506043/diff/13001/lily/parser.yy File lily/parser.yy (right): https://codereview.appspot.com/8506043/diff/13001/lily/parser.yy#newcode2995 lily/parser.yy:2995: parser-parser_error (@1, _ (unexpected markup, without any ^ _ or -, and outside of \\lyricmode)); The

Re: stencil-integral: use box extents specified in markup; issue 3255 (issue 9295044)

2013-08-30 Thread Mike Solomon
On 30 août 2013, at 10:11, d...@gnu.org wrote: On 2013/08/30 07:41:49, mike7 wrote: I'd still argue that (2) is the best way to go as it is a one-stop-shopping way to clear all these bugs (and perhaps more) as they arise. Since we are currently in the process of recovering from the last

Re: stencil-integral: use box extents specified in markup; issue 3255 (issue 9295044)

2013-08-30 Thread Keith OHara
On Thu, 29 Aug 2013 23:57:59 -0700, Mike Solomon m...@mikesolomon.org wrote: On 30 août 2013, at 07:49, Keith OHara k-ohara5...@oco.net wrote: It hurts very little to find ourselves stuck with a second-choice implementation in this case. We won't know what our first-choice implementation is

Re: stencil-integral: use box extents specified in markup; issue 3255 (issue 9295044)

2013-08-30 Thread dak
On 2013/08/30 08:55:15, mike7 wrote: On 30 août 2013, at 10:11, mailto:d...@gnu.org wrote: On 2013/08/30 07:41:49, mike7 wrote: I'd still argue that (2) is the best way to go as it is a one-stop-shopping way to clear all these bugs (and perhaps more) as they arise. Since we are

Re: stencil-integral: use box extents specified in markup; issue 3255 (issue 9295044)

2013-08-30 Thread Mike Solomon
On 30 août 2013, at 11:39, d...@gnu.org wrote: I suggest that we refrain from embracing your somewhat optimistic estimates until after wrapping up a stable release. I don't see the relationship between the two. Which makes it a good thing that you are not pursuing a career as

Re: make sure that AmbitusLine is visible for small ambits (issue 4609041)

2013-08-30 Thread lemniskata . bernoullego
https://codereview.appspot.com/4609041/diff/29002/input/regression/ambitus-gap.ly File input/regression/ambitus-gap.ly (right): https://codereview.appspot.com/4609041/diff/29002/input/regression/ambitus-gap.ly#newcode5 input/regression/ambitus-gap.ly:5: note heads are set by the @code{gap}

Re: parser: more specific error messages; issue 3300 (issue 8506043)

2013-08-30 Thread k-ohara5a5a
https://codereview.appspot.com/8506043/diff/13001/lily/parser.yy File lily/parser.yy (right): https://codereview.appspot.com/8506043/diff/13001/lily/parser.yy#newcode2995 lily/parser.yy:2995: parser-parser_error (@1, _ (unexpected markup, without any ^ _ or -, and outside of \\lyricmode)); On