On 2010/11/20 08:44:51, Trevor Daniels wrote:
LGTM
Patch pushed and issue closed.
http://codereview.appspot.com/3199041/
___
lilypond-devel mailing list
lilypond-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/lilypond-devel
LGTM
http://codereview.appspot.com/3199041/
___
lilypond-devel mailing list
lilypond-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/lilypond-devel
On 2010/11/20 01:23:07, Keith wrote:
Misplaced underscore tag for translation?
Oops. Fixed, thanks.
On 2010/11/20 00:12:19, Graham Percival wrote:
I would rather wait another 23.5 hours, to give people in
all timezones (regardless of geographic location; I'm
living in Pacific Standard Time a
http://codereview.appspot.com/3199041/diff/10001/python/convertrules.py
File python/convertrules.py (right):
http://codereview.appspot.com/3199041/diff/10001/python/convertrules.py#newcode3101
python/convertrules.py:3101: stderr_write("Adjust settings for
top-system-spacing instead.\n")
Misplace
On 2010/11/19 23:23:20, Mark Polesky wrote:
Okay, this small patch is finished. Graham, would you like
me to wait to push this for any reason?
The patch looks ok to me. I agree with the 2.13.40 convert-ly rule, and
the NOT_SMART-ness. At the moment, I can't think of any reason why
somebody w
Okay, this small patch is finished. Graham, would you like
me to wait to push this for any reason?
- Mark
http://codereview.appspot.com/3199041/
___
lilypond-devel mailing list
lilypond-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/lilypond-dev
On 2010/11/19 10:06:33, perpeduumimmobile wrote:
On 2010/11/19 07:55:46, Patrick McCarty wrote:
> IIUC, this would not be a straightforward conversion that regular
> expressions can (easily) deal with. I would suggest a NOT SMART
> rule that recommends the use of the new spacing alists.
I sec
On 2010/11/19 17:26:49, Mark Polesky wrote:
On 2010/11/19 07:55:46, Patrick McCarty wrote:
> I would suggest a NOT SMART rule that recommends the
> use of the new spacing alists.
At which release point should this rule be entered? This
commit was pushed 2.13.4:
2f38710 Rewrite the vertical la
On 2010/11/19 07:55:46, Patrick McCarty wrote:
I would suggest a NOT SMART rule that recommends the
use of the new spacing alists.
At which release point should this rule be entered? This
commit was pushed 2.13.4:
2f38710 Rewrite the vertical layout of staves/systems.
- Mark
http://coderevie
On 2010/11/19 07:55:46, Patrick McCarty wrote:
I would suggest a NOT SMART rule that recommends the
use of the new spacing alists.
At which release point should this rule be entered? This
commit was pushed 2.13.4:
2f38710 Rewrite the vertical layout of staves/systems.
- Mark
http://coderevie
On 2010/11/19 07:55:46, Patrick McCarty wrote:
IIUC, this would not be a straightforward conversion that regular
expressions can (easily) deal with. I would suggest a NOT SMART
rule that recommends the use of the new spacing alists.
I second this. It would be possible to convert the *-separat
On 2010/11/19 07:02:35, Mark Polesky wrote:
When they ceased to be, the functionality
they provided was presumably achievable using the new
spacing alists, but a convert-ly rule to automate this was
never written.
IIUC, this would not be a straightforward conversion that regular
expressions can
On 2010/11/18 17:51:55, Graham Percival wrote:
I'm getting a bit confused with all the renamings,
reorganizations, etc., and I can't be the only one. Could
we slow things down slightly? I'd like to see only one
spacing patch "on the table" at once, and leaving 24 hours
after the final draft of
On 2010/11/18 09:12:23, Mark Polesky wrote:
Is that precisely how the conversion should be made? Can
someone verify this? Should I push this patch or wait for
a convert-ly rule?
I'm getting a bit confused with all the renamings, reorganizations,
etc., and I can't be the only one. Could we sl
Reviewers: ,
Message:
If I understand correctly, head-separation and
foot-separation are gone. This patch just removes the
vestiges from scm/ and ly/, but I suppose we should provide
a convert-ly rule?
So, in an earlier patch by Alexander...
http://codereview.appspot.com/1710046/diff/19001/Docu
15 matches
Mail list logo
