[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Redo autobeam settings to make resetting easier (issue1667041)
From: |
n . puttock |
Subject: |
Re: Redo autobeam settings to make resetting easier (issue1667041) |
Date: |
Sun, 13 Jun 2010 19:01:48 +0000 |
Hi Carl,
Are you sure this is ready for review?
I see comments like this,
// I removed this to solve a bug; need to make sure it's right -- cds
and commented debug code,
+;(display "\nIn auto-beam.scm\n")
and alarm bells start to ring. :)
Also, there are several places where beamSettings is capitalized (most
importantly, in bagpipe.ly.)
BTW, what happens if a time signature has no default settings?
Cheers,
Neil
http://codereview.appspot.com/1667041/show
- Redo autobeam settings to make resetting easier (issue1667041), Carl . D . Sorensen, 2010/06/12
- Re: Redo autobeam settings to make resetting easier (issue1667041),
n . puttock <=
- Re: Redo autobeam settings to make resetting easier (issue1667041), n . puttock, 2010/06/14
- Re: Redo autobeam settings to make resetting easier (issue1667041), Graham Percival, 2010/06/14
- Re: Redo autobeam settings to make resetting easier (issue1667041), Carl Sorensen, 2010/06/14
- Re: Redo autobeam settings to make resetting easier (issue1667041), Neil Puttock, 2010/06/14
- Re: Redo autobeam settings to make resetting easier (issue1667041), Trevor Daniels, 2010/06/14
- Re: Redo autobeam settings to make resetting easier (issue1667041), Carl Sorensen, 2010/06/14
- Re: Redo autobeam settings to make resetting easier (issue1667041), Trevor Daniels, 2010/06/14
- Re: Redo autobeam settings to make resetting easier (issue1667041), Graham Percival, 2010/06/14