[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Issue 4154: Compact Chord Symbols Patch (issue 153160043 by address@
Re: Issue 4154: Compact Chord Symbols Patch (issue 153160043 by address@hidden)
Thu, 09 Oct 2014 11:36:30 +0000
On 2014/10/09 11:08:16, richard_rshann.plus.com wrote:
On Wed, 2014-10-08 at 17:41 +0000, mailto:address@hidden wrote:
> Reviewers: ,
> File scm/chord-ignatzek-names.scm (right):
> scm/chord-ignatzek-names.scm:98: (vector-ref #("C" "D" "E" "F" "G"
> "B") (ly:pitch-notename
> This looks like a bad idea. It does not obey the various chord name
> languages. It does not use the same callbacks. It is a large
> duplication of code not connected with the other code and not using
> same options, functionality and interfaces.
When I saw this (on the lilypond-devel mailing list) I thought this
a comment about the existing code. The code quoted is the existing
which I haven't changed.
It most certainly isn't the existing code. It is not even present
anywhere in the same file. You might have copied parts of it into the
file from some other file, but even git blame -w -C -C -C does not
point to any purported other file as a potential source.