http://codereview.appspot.com/4518045/diff/1/scm/translation-functions.scm
File scm/translation-functions.scm (right):
http://codereview.appspot.com/4518045/diff/1/scm/translation-functions.scm#newcode351
scm/translation-functions.scm:351: (note-pitch note) string))
(car
On 2011/05/26 03:26:32, Keith wrote:
`make check` was crashing on an unbound variable 'note'. Given line
345 above
the fix was so obvious that I just pushed it.
Thanks for the fix. I had found that fix as well, and was getting ready
to push it. But there's still another problem. Right
Reviewers: MikeSol,
Message:
In response to Mike's request, I've fixed the problem in determine-frets
that sometimes changed the order of the notes in the chord.
This keeps the glissando between the same notes in the TabStaff as in
the Staff.
I've made enough changes that I'd like to get a
Hi Carl,
LGTM, apart from some indentation issues.
One question for you though: do you think the refactoring is an
improvement on the current code? You could for example achieve the same
result with the following:
(sort string-fret-fingering-tuples
(lambda (a b) ( (car a) (car
On 5/9/11 12:34 PM, n.putt...@gmail.com n.putt...@gmail.com wrote:
Hi Carl,
LGTM, apart from some indentation issues.
One question for you though: do you think the refactoring is an
improvement on the current code? You could for example achieve the same
result with the following:
On 9 May 2011 19:44, Carl Sorensen c_soren...@byu.edu wrote:
There's no guarantee that the notes will be in decreasing string number
order.
That's the general case, but it's not required.
In absolute mode I can write a C major chord as
c' e' g'
or
g' e' c'
and as far as I know, we
http://codereview.appspot.com/4518045/diff/1/scm/translation-functions.scm
File scm/translation-functions.scm (right):
http://codereview.appspot.com/4518045/diff/1/scm/translation-functions.scm#newcode280
scm/translation-functions.scm:280: (define free-strings (map 1+ (iota
(length tuning
