]> git.cworth.org Git - scherzo/blobdiff - scherzo.c
Fix bug preventing recognition of m6/9 and add m6/♭9
[scherzo] / scherzo.c
index 8c6d394934087dd61da9bab7fbd16dc0a6111d3a..5c24a543baaf4e4f63a38d304fabacfd57f8c786 100644 (file)
--- a/scherzo.c
+++ b/scherzo.c
@@ -450,10 +450,15 @@ static int
 _modified_degree_to_half_steps (const modified_degree_t *degree)
 {
     int half_steps;
+    int scale_degree = degree->degree;
+
+    /* Restrict to actual degrees within a scale. */
+    if (scale_degree > 7)
+       scale_degree = (scale_degree % 8) + 1;
 
     /* Number of half steps from root to specified degree within a
      * diatonic scaled. */
-    switch (degree->degree) {
+    switch (scale_degree) {
     case 1:
        half_steps = 0;
        break;
@@ -489,51 +494,126 @@ _chord_signature_matches (analyzed_note_t *notes,
                          int num_notes,
                          modified_degree_t *degrees,
                          int num_degrees,
-                         int *inversion_ret)
+                         int inversion,
+                         score_pitch_t *root)
 {
-#define MAX_DEGREES 4
+#define MAX_DEGREES 6
     int relative_pitches[MAX_DEGREES];
-    int inversion, max_inversions;
-    int i;
+    int i, root_index;
 
     assert (num_degrees <= MAX_DEGREES);
 
     if (num_notes != num_degrees)
-           return 0;
+       return 0;
 
-    max_inversions = num_degrees;
+    if (inversion >= num_degrees)
+       return 0;
 
     /* We never spell simple intervals as inversions. */
-    if (num_degrees == 2)
-       max_inversions = 1;
-
-    for (inversion = 0; inversion < max_inversions; inversion++) {
-       for (i = 0; i < num_degrees; i++) {
-           /* The num_degrees is in the addition just to ensure all
-            * inputs to the modulus operator remain positive. */
-           int index = (i + num_degrees - inversion) % num_degrees;
-
-           /* Again, adding a 12 to keep things positive. */
-           relative_pitches[index] =
-               (12 +
-                _modified_degree_to_half_steps (&degrees[i]) -
-                _modified_degree_to_half_steps (&degrees[inversion])) % 12;
+    if (num_degrees == 2 && inversion > 0)
+       return 0;
+
+    for (i = 0; i < num_degrees; i++) {
+       /* The num_degrees is in the addition just to ensure all
+        * inputs to the modulus operator remain positive. */
+       int index = (i + num_degrees - inversion) % num_degrees;
+
+       /* Again, adding a 12 to keep things positive. */
+       relative_pitches[index] =
+           (12 +
+            _modified_degree_to_half_steps (&degrees[i]) -
+            _modified_degree_to_half_steps (&degrees[inversion])) % 12;
                
-       }
+    }
 
-       for (i = 0; i < num_notes; i++)
-           if (notes[i].relative_pitch != relative_pitches[i])
-               goto NEXT_INVERSION;
+    for (i = 0; i < num_notes; i++)
+       if (notes[i].relative_pitch != relative_pitches[i])
+           return 0;
 
-       *inversion_ret = inversion;
+    root_index = (num_notes - inversion) % num_notes;
+    *root = notes[root_index].note->pitch;
 
-       return 1;
+    return 1;
+}
 
-    NEXT_INVERSION:
-       ;
+static const char *
+_pitch_str (score_pitch_t pitch)
+{
+    switch (pitch) {
+    case SCORE_PITCH_Cff:
+       return "C𝄫";
+    case SCORE_PITCH_Cf:
+       return "C♭";
+    case SCORE_PITCH_C:
+       return "C";
+    case SCORE_PITCH_Cs:
+       return "C♯";
+    case SCORE_PITCH_Css:
+       return "C𝄪";
+    case SCORE_PITCH_Dff:
+       return "D𝄫";
+    case SCORE_PITCH_Df:
+       return "D♭";
+    case SCORE_PITCH_D:
+       return "D";
+    case SCORE_PITCH_Ds:
+       return "D♯";
+    case SCORE_PITCH_Dss:
+       return "D𝄪";
+    case SCORE_PITCH_Eff:
+       return "E𝄫";
+    case SCORE_PITCH_Ef:
+       return "E♭";
+    case SCORE_PITCH_E:
+       return "E";
+    case SCORE_PITCH_Es:
+       return "E♯";
+    case SCORE_PITCH_Ess:
+       return "E𝄪";
+    case SCORE_PITCH_Fff:
+       return "F𝄫";
+    case SCORE_PITCH_Ff:
+       return "F♭";
+    case SCORE_PITCH_F:
+       return "F";
+    case SCORE_PITCH_Fs:
+       return "F♯";
+    case SCORE_PITCH_Fss:
+       return "F𝄪";
+    case SCORE_PITCH_Gff:
+       return "G𝄫";
+    case SCORE_PITCH_Gf:
+       return "G♭";
+    case SCORE_PITCH_G:
+       return "G";
+    case SCORE_PITCH_Gs:
+       return "G♯";
+    case SCORE_PITCH_Gss:
+       return "G𝄪";
+    case SCORE_PITCH_Aff:
+       return "A𝄫";
+    case SCORE_PITCH_Af:
+       return "A♭";
+    case SCORE_PITCH_A:
+       return "A";
+    case SCORE_PITCH_As:
+       return "A♯";
+    case SCORE_PITCH_Ass:
+       return "A𝄪";
+    case SCORE_PITCH_Bff:
+       return "B𝄫";
+    case SCORE_PITCH_Bf:
+       return "B♭";
+    case SCORE_PITCH_B:
+       return "B";
+    case SCORE_PITCH_Bs:
+       return "B♯";
+    case SCORE_PITCH_Bss:
+       return "B𝄪";
     }
 
-    return 0;
+    fprintf (stderr, "Internal error: Unknown pitch %d\n", pitch);
+    exit (1);
 }
 
 static void
@@ -544,6 +624,7 @@ scherzo_analyze_chord (scherzo_t *scherzo)
     note_group_t *note_group;
     unsigned i, j, num_notes;
     int bass_pitch, inversion;
+    score_pitch_t root;
     const char *chord_name = NULL;
 
     if (scherzo->pedal_pressed)
@@ -554,40 +635,148 @@ scherzo_analyze_chord (scherzo_t *scherzo)
     num_notes = note_group->num_notes;
 
     struct { modified_degree_t degrees[1]; const char *name; } octaves[] = {
-       { {{1, 0}}, "Octave"}
+       { {{1, 0}}, " Octave"}
     };
 
     struct { modified_degree_t degrees[2]; const char *name; } intervals[] = {
-       { {{1, 0}, {2, -1}}, "Minor 2nd"},
-       { {{1, 0}, {2,  0}}, "Major 2nd"},
-       { {{1, 0}, {3, -1}}, "Minor 3rd"},
-       { {{1, 0}, {3,  0}}, "Major 3rd"},
-       { {{1, 0}, {4,  0}}, "Perfect 4th"},
-       { {{1, 0}, {5, -1}}, "Diminished 5th"},
-       { {{1, 0}, {5,  0}}, "Perfect 5th"},
-       { {{1, 0}, {6, -1}}, "Minor 6th"},
-       { {{1, 0}, {6,  0}}, "Major 6th"},
-       { {{1, 0}, {7, -1}}, "Minor 7th"},
-       { {{1, 0}, {7,  0}}, "Major 7th"}
+       { {{1, 0}, {2, -1}}, " Minor 2nd"},
+       { {{1, 0}, {2,  0}}, " Major 2nd"},
+       { {{1, 0}, {3, -1}}, " Minor 3rd"},
+       { {{1, 0}, {3,  0}}, " Major 3rd"},
+       { {{1, 0}, {4,  0}}, " Perfect 4th"},
+       { {{1, 0}, {5, -1}}, " Diminished 5th"},
+       { {{1, 0}, {5,  0}}, " Perfect 5th"},
+       { {{1, 0}, {6, -1}}, " Minor 6th"},
+       { {{1, 0}, {6,  0}}, " Major 6th"},
+       { {{1, 0}, {7, -1}}, " Minor 7th"},
+       { {{1, 0}, {7,  0}}, " Major 7th"}
     };
 
-    struct { modified_degree_t degrees[3]; const char *name; } triads[] = {
-       { {{1, 0}, {3,  0}, {5, +1}}, "Augmented triad" },
-       { {{1, 0}, {3,  0}, {5,  0}}, "Major triad" },
-       { {{1, 0}, {3, -1}, {5,  0}}, "Minor triad" },
-       { {{1, 0}, {3, -1}, {5, -1}}, "Diminished triad" }
+/* XXX: The superscript rise value should be relative to the current
+ * font size. Best would be for pango to support this. See:
+ *
+ * https://bugzilla.gnome.org/show_bug.cgi?id=708780
+ *
+ * For now, these are emipirically-derived values that look reasonable
+ * at the default size that score.c is using to draw the staff. If the
+ * user scales the staff up or down then these will look wrong.
+ */
+
+#define SUP "<span font='33' rise='30000'>"
+#define PUS "</span>"
+
+    struct { modified_degree_t degrees[3]; const char *name; } trichords[] = {
+       /* Triads */
+       { {{1, 0}, {3, +1}, {5,  0}}, "sus" },
+       { {{1, 0}, {3,  0}, {5, +1}}, SUP "+" PUS },
+       { {{1, 0}, {3,  0}, {5,  0}}, "" },
+       { {{1, 0}, {3, -1}, {5,  0}}, "m" },
+       { {{1, 0}, {3, -1}, {5, -1}}, "°" },
+       { {{1, 0}, {2,  0}, {5,  0}}, "msus2" },
+       /* Seventh chords with no 3rd */
+       { {{1, 0}, {5, +1}, {7,  0}}, SUP "+M7" PUS },
+       { {{1, 0}, {5, +1}, {7, -1}}, SUP "+7" PUS },
+       { {{1, 0}, {5,  0}, {7,  0}}, "M7" },
+       { {{1, 0}, {5,  0}, {7, -1}}, "7" },
+       { {{1, 0}, {5, -1}, {7, -1}}, "7♭5" },
+       /* Seventh chords with no 5th */
+       { {{1, 0}, {4,  0}, {7,  0}}, "M7sus" },
+       { {{1, 0}, {4,  0}, {7, -1}}, "7sus" },
+       { {{1, 0}, {3,  0}, {7,  0}}, "M7" },
+       { {{1, 0}, {3,  0}, {7, -1}}, "7" },
+       { {{1, 0}, {3, -1}, {7,  0}}, "m" SUP "M7" PUS },
+       { {{1, 0}, {3, -1}, {7, -1}}, "m7" },
+       { {{1, 0}, {2,  0}, {7,  0}}, "m" SUP "M7" PUS "sus2" },
+       { {{1, 0}, {2,  0}, {7, -1}}, "m7sus2" },
+    };
+
+    struct { modified_degree_t degrees[4]; const char *name; } tetrachords[] = {
+       /* Sixth chords */
+       { {{1, 0}, {4,  0}, {5,  0}, {6,  0}}, "6sus" },
+       { {{1, 0}, {3,  0}, {5,  0}, {6,  0}}, "6" }, /* Ambiguous with m7 */
+       { {{1, 0}, {3, -1}, {5,  0}, {6,  0}}, "m6" },
+       { {{1, 0}, {2,  0}, {5,  0}, {6,  0}}, "m6sus2" },
+       /* Seventh chords */
+       { {{1, 0}, {4,  0}, {5, +1}, {7,  0}}, SUP "+M7" PUS "sus" },
+       { {{1, 0}, {4,  0}, {5, +1}, {7, -1}}, SUP "+7" PUS "sus" },
+       { {{1, 0}, {4,  0}, {5,  0}, {7,  0}}, "M7sus" },
+       { {{1, 0}, {4,  0}, {5,  0}, {7, -1}}, "7sus" },
+       { {{1, 0}, {4,  0}, {5, -1}, {7, -1}}, "7♭5sus" },
+       { {{1, 0}, {3,  0}, {5, +1}, {7,  0}}, SUP "+M7" PUS },
+       { {{1, 0}, {3,  0}, {5, +1}, {7, -1}}, SUP "+7" PUS },
+       { {{1, 0}, {3,  0}, {5,  0}, {7,  0}}, "M7" },
+       { {{1, 0}, {3,  0}, {5,  0}, {7, -1}}, "7" },
+       { {{1, 0}, {3,  0}, {5, -1}, {7, -1}}, "7♭5" },
+       { {{1, 0}, {3, -1}, {5,  0}, {7,  0}}, "m" SUP "M7" PUS },
+       { {{1, 0}, {3, -1}, {5,  0}, {7, -1}}, "m7" },
+       { {{1, 0}, {3, -1}, {5, -1}, {7,  0}}, "°" SUP "M7" PUS },
+       { {{1, 0}, {3, -1}, {5, -1}, {7, -1}}, "𝆩" SUP "7" PUS },
+       { {{1, 0}, {3, -1}, {5, -1}, {7, -2}}, "°" SUP "7" PUS },
+       { {{1, 0}, {2,  0}, {5,  0}, {7,  0}}, "m" SUP "M7" PUS "sus2" },
+       { {{1, 0}, {2,  0}, {5,  0}, {7, -1}}, "m7sus2" },
+       { {{1, 0}, {2,  0}, {5, -1}, {7,  0}}, "°" SUP "M7" PUS "sus2" },
+       { {{1, 0}, {2,  0}, {5, -1}, {7, -1}}, "𝆩" SUP "7" PUS "sus2" },
+       { {{1, 0}, {2,  0}, {5, -1}, {7, -2}}, "°" SUP "7" PUS "sus2" }, /* Ambiguous with 7 */
+       /* Ninth chords voiced with no 5th */
+       { {{1, 0}, {9,  0}, {4,  0}, {7,  0}}, "M9sus" },
+       { {{1, 0}, {9,  0}, {4,  0}, {7, -1}}, "9sus" },
+       { {{1, 0}, {9,  0}, {3,  0}, {7,  0}}, "M9" },
+       { {{1, 0}, {9,  0}, {3,  0}, {7, -1}}, "9" },
+       { {{1, 0}, {9,  0}, {3, -1}, {7,  0}}, "m" SUP "M9" PUS },
+       { {{1, 0}, {9,  0}, {3, -1}, {7, -1}}, "m9" },
+       { {{1, 0}, {9, -1}, {3, -1}, {7, -1}}, "m♭9" },
     };
 
-    struct { modified_degree_t degrees[4]; const char *name; } sevenths[] = {
-       { {{1, 0}, {3,  0}, {5, +1}, {7,  0}}, "Augmented/major 7" },
-       { {{1, 0}, {3,  0}, {5, +1}, {7, -1}}, "Augmented 7" },
-       { {{1, 0}, {3,  0}, {5,  0}, {7,  0}}, "Major 7" },
-       { {{1, 0}, {3,  0}, {5,  0}, {7, -1}}, "Dominant 7" },
-       { {{1, 0}, {3, -1}, {5,  0}, {7,  0}}, "Minor/major 7" },
-       { {{1, 0}, {3, -1}, {5,  0}, {7, -1}}, "Minor 7" },
-       { {{1, 0}, {3, -1}, {5, -1}, {7,  0}}, "Diminished/major 7" },
-       { {{1, 0}, {3, -1}, {5, -1}, {7, -1}}, "Half-diminished 7" },
-       { {{1, 0}, {3, -1}, {5, -1}, {7, -2}}, "Diminished 7" }
+    /* The sorting here is funny to keep the array in degree order
+     * after reducing each degree to an actual scale degree, (9 -> 2,
+     * 11 -> 4, 13 -> 6) */
+    struct { modified_degree_t degrees[5]; const char *name; } pentachords[] = {
+       /* Sixth plus 9 */
+       { {{1, 0}, {9,  0}, {3,  0}, {5,  0}, {6,  0}}, "6/9" },
+       { {{1, 0}, {9,  0}, {3, -1}, {5,  0}, {6,  0}}, "m6/9" },
+       { {{1, 0}, {9, -1}, {3, -1}, {5,  0}, {6,  0}}, "m6/♭9" },
+       /* Seventh plus altered 9 */
+       { {{1, 0}, {9, +1}, {3,  0}, {5,  0}, {7, -1}}, "7♯9" },
+       { {{1, 0}, {9, -1}, {3,  0}, {5,  0}, {7, -1}}, "7♭9" },
+       { {{1, 0}, {9, +1}, {3,  0}, {5, -1}, {7, -1}}, "7♭5♯9" },
+       { {{1, 0}, {9, -1}, {3,  0}, {5, -1}, {7, -1}}, "7♭5♭9" },
+       /* Ninth chords */
+       { {{1, 0}, {9,  0}, {4,  0}, {5, +1}, {7,  0}}, SUP "+M9" PUS "sus" },
+       { {{1, 0}, {9,  0}, {4,  0}, {5, +1}, {7, -1}}, SUP "+9" PUS "sus" },
+       { {{1, 0}, {9,  0}, {4,  0}, {5,  0}, {7,  0}}, "M9sus" },
+       { {{1, 0}, {9,  0}, {4,  0}, {5,  0}, {7, -1}}, "9sus" },
+
+       { {{1, 0}, {9,  0}, {3,  0}, {5, +1}, {7,  0}}, SUP "+M9" PUS },
+       { {{1, 0}, {9,  0}, {3,  0}, {5, +1}, {7, -1}}, SUP "+9" PUS },
+
+       { {{1, 0}, {9,  0}, {3,  0}, {5,  0}, {7,  0}}, "M9" },
+       { {{1, 0}, {9,  0}, {3,  0}, {5,  0}, {7, -1}}, "9" },
+       { {{1, 0}, {9,  0}, {3, -1}, {5,  0}, {7,  0}}, "m" SUP "M9" PUS },
+       { {{1, 0}, {9,  0}, {3, -1}, {5,  0}, {7, -1}}, "m9" },
+       { {{1, 0}, {9, -1}, {3, -1}, {5,  0}, {7, -1}}, "m♭9" },
+       { {{1, 0}, {9,  0}, {3,  0}, {5, -1}, {7,  0}}, "M9" SUP "♭5" PUS },
+       { {{1, 0}, {9,  0}, {3,  0}, {5, -1}, {7, -1}}, "9" SUP "♭5" PUS },
+       { {{1, 0}, {9,  0}, {3, -1}, {5, -1}, {7,  0}}, "m" SUP "M9♭5" PUS },
+       { {{1, 0}, {9,  0}, {3, -1}, {5, -1}, {7, -1}}, "𝆩" SUP "9" PUS },
+
+       /* FIXME: I don't have names for these last three after
+        * dropping the 5th from the voicing. That suggests to me that
+        * I'm missing names for these with a perfect 5th in the list
+        * above. */
+       { {{1, 0}, {9, -1}, {3, -1}, {5, -1}, {7, -1}}, "𝆩" SUP "♭9" PUS },
+       { {{1, 0}, {9,  0}, {3, -1}, {5, -1}, {7, -2}}, "°" SUP "9" PUS },
+       { {{1, 0}, {9, -1}, {3, -1}, {5, -1}, {7, -2}}, "°" SUP "♭9" PUS },
+    };
+
+    struct { modified_degree_t degrees[6]; const char *name; } hexachords[] = {
+       { {{1, 0}, {9,  0}, {3,  0}, {11,  0}, {5, +1}, {7,  0}}, SUP "+M11" PUS },
+       { {{1, 0}, {9,  0}, {3,  0}, {11,  0}, {5, +1}, {7, -1}}, SUP "+11" PUS },
+       { {{1, 0}, {9,  0}, {3,  0}, {11,  0}, {5,  0}, {7,  0}}, "M11" },
+       { {{1, 0}, {9,  0}, {3,  0}, {11,  0}, {5,  0}, {7, -1}}, "11" },
+       { {{1, 0}, {9,  0}, {3, -1}, {11,  0}, {5,  0}, {7,  0}}, "m" SUP "M11" PUS },
+       { {{1, 0}, {9,  0}, {3, -1}, {11,  0}, {5,  0}, {7, -1}}, "m11" },
+       { {{1, 0}, {9, -1}, {3, -1}, {11,  0}, {5, -1}, {7, -1}}, "𝆩" SUP "11" PUS },
+       { {{1, 0}, {9, -1}, {3, -1}, {11, -1}, {5, -1}, {7, -2}}, "°" SUP "11" PUS }
     };
 
     if (scherzo->chord) {
@@ -652,48 +841,77 @@ scherzo_analyze_chord (scherzo_t *scherzo)
            }
     }
 
-    switch (num_notes) {
-    case 1:
-       for (i = 0; i < ARRAY_SIZE (octaves); i++) {
-           if (_chord_signature_matches (notes, num_notes,
-                                         octaves[i].degrees, 1, &inversion))
-           {
-               chord_name = octaves[i].name;
-               break;
+    for (inversion = 0; inversion < 4; inversion++) {
+       switch (num_notes) {
+       case 1:
+           for (i = 0; i < ARRAY_SIZE (octaves); i++) {
+               if (_chord_signature_matches (notes, num_notes,
+                                             octaves[i].degrees, 1,
+                                             inversion, &root))
+               {
+                   chord_name = octaves[i].name;
+                   goto CHORD_NAME_KNOWN;
+               }
            }
-       }
-       break;
-    case 2:
-       for (i = 0; i < ARRAY_SIZE (intervals); i++) {
-           if (_chord_signature_matches (notes, num_notes,
-                                         intervals[i].degrees, 2, &inversion))
-           {
-               chord_name = intervals[i].name;
-               break;
+           break;
+       case 2:
+           for (i = 0; i < ARRAY_SIZE (intervals); i++) {
+               if (_chord_signature_matches (notes, num_notes,
+                                             intervals[i].degrees, 2,
+                                             inversion, &root))
+               {
+                   chord_name = intervals[i].name;
+                   goto CHORD_NAME_KNOWN;
+               }
            }
-       }
-       break;
-    case 3:
-       for (i = 0; i < ARRAY_SIZE (triads); i++) {
-           if (_chord_signature_matches (notes, num_notes,
-                                         triads[i].degrees, 3, &inversion))
-           {
-               chord_name = triads[i].name;
-               break;
+           break;
+       case 3:
+           for (i = 0; i < ARRAY_SIZE (trichords); i++) {
+               if (_chord_signature_matches (notes, num_notes,
+                                             trichords[i].degrees, 3,
+                                             inversion, &root))
+               {
+                   chord_name = trichords[i].name;
+                   goto CHORD_NAME_KNOWN;
+               }
            }
-       }
-       break;
-    case 4:
-       for (i = 0; i < ARRAY_SIZE(sevenths); i++) {
-           if (_chord_signature_matches (notes, num_notes,
-                                         sevenths[i].degrees, 4, &inversion))
-           {
-               chord_name = sevenths[i].name;
-               break;
+           break;
+       case 4:
+           for (i = 0; i < ARRAY_SIZE (tetrachords); i++) {
+               if (_chord_signature_matches (notes, num_notes,
+                                             tetrachords[i].degrees, 4,
+                                             inversion, &root))
+               {
+                   chord_name = tetrachords[i].name;
+                   goto CHORD_NAME_KNOWN;
+               }
+           }
+           break;
+       case 5:
+           for (i = 0; i < ARRAY_SIZE (pentachords); i++) {
+               if (_chord_signature_matches (notes, num_notes,
+                                             pentachords[i].degrees, 5,
+                                             inversion, &root))
+               {
+                   chord_name = pentachords[i].name;
+                   goto CHORD_NAME_KNOWN;
+               }
+           }
+           break;
+       case 6:
+           for (i = 0; i < ARRAY_SIZE (hexachords); i++) {
+               if (_chord_signature_matches (notes, num_notes,
+                                             hexachords[i].degrees, 6,
+                                             inversion, &root))
+               {
+                   chord_name = hexachords[i].name;
+                   goto CHORD_NAME_KNOWN;
+               }
            }
+           break;
        }
-       break;
     }
+    CHORD_NAME_KNOWN:
 
     if (chord_name) {
        if (inversion) {
@@ -713,10 +931,13 @@ scherzo_analyze_chord (scherzo_t *scherzo)
                         inversion);
                exit(1);
            }
-           chord_name = talloc_asprintf (local, "%s %s",
+           chord_name = talloc_asprintf (local, "%s%s %s",
+                                         _pitch_str (root),
                                          chord_name, inversion_str);
        } else {
-           chord_name = talloc_strdup (local, chord_name);
+           chord_name = talloc_asprintf (local, "%s%s",
+                                         _pitch_str (root),
+                                         chord_name);
        }
     } else {
        chord_name = talloc_strdup (local, "Unknown chord");