]> git.cworth.org Git - scherzo/blobdiff - scherzo.c
Rework computer keyboard input to be mapped piano keyboard
[scherzo] / scherzo.c
index d7f78fdf6f372ba4b94d5020deeaba4547bea5e7..e57ccbe9b3fe7f831d535bf73d77544d62dce04e 100644 (file)
--- a/scherzo.c
+++ b/scherzo.c
@@ -24,7 +24,7 @@
 #include "score.h"
 #include "mnemon.h"
 
-#define ARRAY_SIZE(arr) (sizeof(arr) / sizeof(arr[0]))
+#define ARRAY_SIZE(arr) ((int) (sizeof(arr) / sizeof(arr[0])))
 
 #define unused(foo) foo __attribute__((unused))
 
 
 typedef struct challenge
 {
+    int active;
     bin_t *bin;
     int item_index;
     score_staff_t *staff;
-    score_note_t *note;
+    pitch_t pitch;
 
     int satisfied;
     int mistaken;
 } challenge_t;
 
-typedef struct note_group
+typedef struct pitch_group
 {
     void *ctx;
-    score_note_t **notes;
+    pitch_t *pitches;
     int size;
-    int num_notes;
-} note_group_t;
+    int num_pitches;
+} pitch_group_t;
 
 typedef struct scherzo
 {
@@ -56,12 +57,14 @@ typedef struct scherzo
     GtkWidget *window;
     score_t *score;
     int staff_height;
+
     score_staff_t *treble;
     score_staff_t *bass;
     score_chord_t *chord;
 
-    /* This is for a "computer keyboard". Any "piano keyboard" key
-     * knows its own octave. */
+    /* The word "keyboard" here is referring to a "computer
+     * keyboard". Any "piano keyboard" key knows its own octave and
+     * accidental already. */
     int keyboard_octave;
 
     int midi_fd;
@@ -70,18 +73,20 @@ typedef struct scherzo
     mnemon_t mnemon;
     challenge_t challenge;
 
-    note_group_t notes_pressed;
-    note_group_t notes_pedaled;
+    pitch_group_t notes_pressed;
+    pitch_group_t notes_pedaled;
 
     int pedal_pressed;
+
+    pitch_t key;
 } scherzo_t;
 
 /* Forward declarations. */
-static score_note_t *
-scherzo_press_note (scherzo_t *scherzo, score_pitch_t pitch, int octave);
+static void
+scherzo_press_note (scherzo_t *scherzo, pitch_t pitch);
 
 static void
-scherzo_release_note (scherzo_t *scherzo, score_pitch_t pitch, int octave);
+scherzo_release_note (scherzo_t *scherzo, pitch_t pitch);
 
 static void
 scherzo_press_pedal (scherzo_t *scherzo);
@@ -90,7 +95,7 @@ static void
 scherzo_release_pedal (scherzo_t *scherzo);
 
 static void
-_judge_note (scherzo_t *scherzo, score_note_t *note);
+_judge_pitch (scherzo_t *scherzo, pitch_t pitch);
 
 static void
 _score_challenge (scherzo_t *scherzo);
@@ -145,10 +150,10 @@ on_key_press_event (GtkWidget *widget,
 {
     scherzo_t *scherzo = user_data;
     int octave;
-    score_pitch_t pitch;
+    pitch_t pitch = PITCH_NOT_A_PITCH;
 
-    if (scherzo->challenge.note)
-       octave = scherzo->challenge.note->octave;
+    if (scherzo->challenge.active)
+       octave = PITCH_OCTAVE (scherzo->challenge.pitch);
     else
        octave = scherzo->keyboard_octave;
 
@@ -167,38 +172,89 @@ on_key_press_event (GtkWidget *widget,
        gtk_widget_queue_draw (widget);
        return TRUE;
        break;
-    case GDK_KEY_q:
-    case GDK_KEY_Q:
     case GDK_KEY_Escape:
        gtk_main_quit ();
        return FALSE;
-    case GDK_KEY_c:
-    case GDK_KEY_C:
-       pitch = SCORE_PITCH_C;
+    /* Map a piano keyboard onto the computer keyboard with white keys
+     * along the home row, (piano C on computer A, up to next octave
+     * higher C on computer K). Black keys are in the expected
+     * positions, (C# is W, D# is E, etc.).
+     */
+    case GDK_KEY_a:
+    case GDK_KEY_A:
+       pitch = PITCH_LITERAL (C, NATURAL, octave);
+       break;
+    case GDK_KEY_s:
+    case GDK_KEY_S:
+       pitch = PITCH_LITERAL (D, NATURAL, octave);
        break;
     case GDK_KEY_d:
     case GDK_KEY_D:
-       pitch = SCORE_PITCH_D;
-       break;
-    case GDK_KEY_e:
-    case GDK_KEY_E:
-       pitch = SCORE_PITCH_E;
+       pitch = PITCH_LITERAL (E, NATURAL, octave);
        break;
     case GDK_KEY_f:
     case GDK_KEY_F:
-       pitch = SCORE_PITCH_F;
+       pitch = PITCH_LITERAL (F, NATURAL, octave);
        break;
     case GDK_KEY_g:
     case GDK_KEY_G:
-       pitch = SCORE_PITCH_G;
+       pitch = PITCH_LITERAL (G, NATURAL, octave);
        break;
-    case GDK_KEY_a:
-    case GDK_KEY_A:
-       pitch = SCORE_PITCH_A;
+    case GDK_KEY_h:
+    case GDK_KEY_H:
+       pitch = PITCH_LITERAL (A, NATURAL, octave);
+       break;
+    case GDK_KEY_j:
+    case GDK_KEY_J:
+       pitch = PITCH_LITERAL (B, NATURAL, octave);
+       break;
+    case GDK_KEY_k:
+    case GDK_KEY_K:
+       pitch = PITCH_LITERAL (C, NATURAL, octave + 1);
+       break;
+    case GDK_KEY_l:
+    case GDK_KEY_L:
+       pitch = PITCH_LITERAL (D, NATURAL, octave + 1);
+       break;
+    case GDK_KEY_semicolon:
+    case GDK_KEY_colon:
+       pitch = PITCH_LITERAL (E, NATURAL, octave + 1);
        break;
-    case GDK_KEY_b:
-    case GDK_KEY_B:
-       pitch = SCORE_PITCH_B;
+    case GDK_KEY_apostrophe:
+    case GDK_KEY_quotedbl:
+       pitch = PITCH_LITERAL (F, NATURAL, octave + 1);
+       break;
+    case GDK_KEY_w:
+    case GDK_KEY_W:
+       pitch = PITCH_LITERAL (C, SHARP, octave);
+       break;
+    case GDK_KEY_e:
+    case GDK_KEY_E:
+       pitch = PITCH_LITERAL (D, SHARP, octave);
+       break;
+    case GDK_KEY_t:
+    case GDK_KEY_T:
+       pitch = PITCH_LITERAL (F, SHARP, octave);
+       break;
+    case GDK_KEY_y:
+    case GDK_KEY_Y:
+       pitch = PITCH_LITERAL (G, SHARP, octave);
+       break;
+    case GDK_KEY_u:
+    case GDK_KEY_U:
+       pitch = PITCH_LITERAL (A, SHARP, octave);
+       break;
+    case GDK_KEY_o:
+    case GDK_KEY_O:
+       pitch = PITCH_LITERAL (C, SHARP, octave + 1);
+       break;
+    case GDK_KEY_p:
+    case GDK_KEY_P:
+       pitch = PITCH_LITERAL (D, SHARP, octave + 1);
+       break;
+    case GDK_KEY_bracketright:
+    case GDK_KEY_braceright:
+       pitch = PITCH_LITERAL (F, SHARP, octave + 1);
        break;
     case GDK_KEY_0:
     case GDK_KEY_1:
@@ -216,13 +272,10 @@ on_key_press_event (GtkWidget *widget,
        break;
     }
 
-    if ((key->keyval >= GDK_KEY_A && key->keyval <= GDK_KEY_G) ||
-       (key->keyval >= GDK_KEY_a && key->keyval <= GDK_KEY_g))
+    if (pitch != PITCH_NOT_A_PITCH) 
     {
-       score_note_t *note;
-
-       note = scherzo_press_note (scherzo, pitch, octave);
-       _judge_note (scherzo, note);
+       scherzo_press_note (scherzo, pitch);
+       _judge_pitch (scherzo, pitch);
        gtk_widget_queue_draw (scherzo->window);
 
        return TRUE;
@@ -240,51 +293,98 @@ on_key_release_event (unused (GtkWidget *widget),
 {
     scherzo_t *scherzo = user_data;
     int octave;
-    score_pitch_t pitch;
+    pitch_t pitch = PITCH_NOT_A_PITCH;
 
-    if (scherzo->challenge.note)
-       octave = scherzo->challenge.note->octave;
+    if (scherzo->challenge.active)
+       octave = PITCH_OCTAVE (scherzo->challenge.pitch);
     else
        octave = scherzo->keyboard_octave;
 
     switch (key->keyval) {
-    case GDK_KEY_c:
-    case GDK_KEY_C:
-       pitch = SCORE_PITCH_C;
+    case GDK_KEY_a:
+    case GDK_KEY_A:
+       pitch = PITCH_LITERAL (C, NATURAL, octave);
+       break;
+    case GDK_KEY_s:
+    case GDK_KEY_S:
+       pitch = PITCH_LITERAL (D, NATURAL, octave);
        break;
     case GDK_KEY_d:
     case GDK_KEY_D:
-       pitch = SCORE_PITCH_D;
-       break;
-    case GDK_KEY_e:
-    case GDK_KEY_E:
-       pitch = SCORE_PITCH_E;
+       pitch = PITCH_LITERAL (E, NATURAL, octave);
        break;
     case GDK_KEY_f:
     case GDK_KEY_F:
-       pitch = SCORE_PITCH_F;
+       pitch = PITCH_LITERAL (F, NATURAL, octave);
        break;
     case GDK_KEY_g:
     case GDK_KEY_G:
-       pitch = SCORE_PITCH_G;
+       pitch = PITCH_LITERAL (G, NATURAL, octave);
        break;
-    case GDK_KEY_a:
-    case GDK_KEY_A:
-       pitch = SCORE_PITCH_A;
+    case GDK_KEY_h:
+    case GDK_KEY_H:
+       pitch = PITCH_LITERAL (A, NATURAL, octave);
+       break;
+    case GDK_KEY_j:
+    case GDK_KEY_J:
+       pitch = PITCH_LITERAL (B, NATURAL, octave);
+       break;
+    case GDK_KEY_k:
+    case GDK_KEY_K:
+       pitch = PITCH_LITERAL (C, NATURAL, octave + 1);
+       break;
+    case GDK_KEY_l:
+    case GDK_KEY_L:
+       pitch = PITCH_LITERAL (D, NATURAL, octave + 1);
+       break;
+    case GDK_KEY_semicolon:
+    case GDK_KEY_colon:
+       pitch = PITCH_LITERAL (E, NATURAL, octave + 1);
+       break;
+    case GDK_KEY_apostrophe:
+    case GDK_KEY_quotedbl:
+       pitch = PITCH_LITERAL (F, NATURAL, octave + 1);
+       break;
+    case GDK_KEY_w:
+    case GDK_KEY_W:
+       pitch = PITCH_LITERAL (C, SHARP, octave);
+       break;
+    case GDK_KEY_e:
+    case GDK_KEY_E:
+       pitch = PITCH_LITERAL (D, SHARP, octave);
        break;
-    case GDK_KEY_b:
-    case GDK_KEY_B:
-       pitch = SCORE_PITCH_B;
+    case GDK_KEY_t:
+    case GDK_KEY_T:
+       pitch = PITCH_LITERAL (F, SHARP, octave);
+       break;
+    case GDK_KEY_y:
+    case GDK_KEY_Y:
+       pitch = PITCH_LITERAL (G, SHARP, octave);
+       break;
+    case GDK_KEY_u:
+    case GDK_KEY_U:
+       pitch = PITCH_LITERAL (A, SHARP, octave);
+       break;
+    case GDK_KEY_o:
+    case GDK_KEY_O:
+       pitch = PITCH_LITERAL (C, SHARP, octave + 1);
+       break;
+    case GDK_KEY_p:
+    case GDK_KEY_P:
+       pitch = PITCH_LITERAL (D, SHARP, octave + 1);
+       break;
+    case GDK_KEY_bracketright:
+    case GDK_KEY_braceright:
+       pitch = PITCH_LITERAL (F, SHARP, octave + 1);
        break;
     case GDK_KEY_space:
        scherzo_release_pedal (scherzo);
        break;
     }
 
-    if ((key->keyval >= GDK_KEY_A && key->keyval <= GDK_KEY_G) ||
-       (key->keyval >= GDK_KEY_a && key->keyval <= GDK_KEY_g))
+    if (pitch != PITCH_NOT_A_PITCH)
     {
-       scherzo_release_note (scherzo, pitch, octave);
+       scherzo_release_note (scherzo, pitch);
        _score_challenge (scherzo);
        gtk_widget_queue_draw (scherzo->window);
 
@@ -297,47 +397,47 @@ on_key_release_event (unused (GtkWidget *widget),
 }
 
 static unsigned char
-_score_pitch_and_octave_to_midi (score_pitch_t pitch,
-                                int octave)
+_pitch_to_midi (pitch_t pitch)
 {
+    int octave = PITCH_OCTAVE (pitch);
     unsigned char midi_note = 12 * (octave + 1);
 
-    switch (SCORE_PITCH_NAME (pitch)) {
-    case SCORE_PITCH_NAME_C:
+    switch (PITCH_NAME (pitch)) {
+    case PITCH_NAME_C:
        break;
-    case SCORE_PITCH_NAME_D:
+    case PITCH_NAME_D:
        midi_note += 2;
        break;
-    case SCORE_PITCH_NAME_E:
+    case PITCH_NAME_E:
        midi_note += 4;
        break;
-    case SCORE_PITCH_NAME_F:
+    case PITCH_NAME_F:
        midi_note += 5;
        break;
-    case SCORE_PITCH_NAME_G:
+    case PITCH_NAME_G:
        midi_note += 7;
        break;
-    case SCORE_PITCH_NAME_A:
+    case PITCH_NAME_A:
        midi_note += 9;
        break;
-    case SCORE_PITCH_NAME_B:
+    case PITCH_NAME_B:
        midi_note += 11;
        break;
     }
 
-    switch (SCORE_PITCH_ACCIDENTAL (pitch)) {
-    case SCORE_PITCH_ACCIDENTAL_DOUBLE_FLAT:
+    switch (PITCH_ACCIDENTAL (pitch)) {
+    case PITCH_ACCIDENTAL_DOUBLE_FLAT:
        midi_note -= 2;
        break;
-    case SCORE_PITCH_ACCIDENTAL_FLAT:
+    case PITCH_ACCIDENTAL_FLAT:
        midi_note -= 1;
        break;
-    case SCORE_PITCH_ACCIDENTAL_NATURAL:
+    case PITCH_ACCIDENTAL_NATURAL:
        break;
-    case SCORE_PITCH_ACCIDENTAL_SHARP:
+    case PITCH_ACCIDENTAL_SHARP:
        midi_note += 1;
        break;
-    case SCORE_PITCH_ACCIDENTAL_DOUBLE_SHARP:
+    case PITCH_ACCIDENTAL_DOUBLE_SHARP:
        midi_note += 2;
        break;
     }
@@ -345,199 +445,520 @@ _score_pitch_and_octave_to_midi (score_pitch_t pitch,
     return midi_note;
 }
 
-static void
-_midi_to_score_pitch_and_octave (unsigned char midi_note,
-                                score_pitch_t *pitch,
-                                int *octave)
+/* octave can optionally by NULL */
+static pitch_t
+_midi_to_pitch (unsigned char midi_note)
 {
-    *octave = midi_note / 12 - 1;
+    int octave = octave = midi_note / 12 - 1;
 
     switch (midi_note % 12)
     {
+    default:
     case 0:
-       *pitch = SCORE_PITCH_C;
+       return PITCH_LITERAL (C, NATURAL, octave);
        break;
     case 1:
-       *pitch = SCORE_PITCH_Cs;
+       return PITCH_LITERAL (C, SHARP, octave);
        break;
     case 2:
-       *pitch = SCORE_PITCH_D;
+       return PITCH_LITERAL (D, NATURAL, octave);
        break;
     case 3:
-       *pitch = SCORE_PITCH_Ds;
+       return PITCH_LITERAL (D, SHARP, octave);
        break;
     case 4:
-       *pitch = SCORE_PITCH_E;
+       return PITCH_LITERAL (E, NATURAL, octave);
        break;
     case 5:
-       *pitch = SCORE_PITCH_F;
+       return PITCH_LITERAL (F, NATURAL, octave);
        break;
     case 6:
-       *pitch = SCORE_PITCH_Fs;
+       return PITCH_LITERAL (F, SHARP, octave);
        break;
     case 7:
-       *pitch = SCORE_PITCH_G;
+       return PITCH_LITERAL (G, NATURAL, octave);
        break;
     case 8:
-       *pitch = SCORE_PITCH_Gs;
+       return PITCH_LITERAL (G, SHARP, octave);
        break;
     case 9:
-       *pitch = SCORE_PITCH_A;
+       return PITCH_LITERAL (A, NATURAL, octave);
        break;
     case 10:
-       *pitch = SCORE_PITCH_As;
+       return PITCH_LITERAL (A, SHARP, octave);
        break;
     case 11:
-       *pitch = SCORE_PITCH_B;
+       return PITCH_LITERAL (B, NATURAL, octave);
        break;
     }
 }
 
+/* Return true if 'a' and 'b' sound identical, (even if spelled differently)
+ *
+ * This comparison considers octaves as significant. So Bb and A# in
+ * the same octave are considered enharmonic, but Bb and A# in
+ * different octaves are not. */
+static int
+_pitches_are_enharmonic (pitch_t a, pitch_t b)
+{
+    return _pitch_to_midi (a) == _pitch_to_midi (b);
+}
+
 /* Determine a chord name (if any) from the current notes pressed */
 
-typedef struct analyzed_note {
+typedef struct analyzed_pitch {
     /* Original note being analzyed. */
-    score_note_t *note;
+    pitch_t pitch;
 
     /* Absolute pitch (expressed as midi number). */
     int midi_pitch;
 
     /* Pitch relative to bass note. */
     int relative_pitch;
-} analyzed_note_t;
+} analyzed_pitch_t;
 
 static int
-_compare_analyzed_note_by_midi_pitch (const void *va, const void *vb)
+_compare_analyzed_pitch_by_midi_pitch (const void *va, const void *vb)
 {
-    const analyzed_note_t *a = va, *b = vb;
+    const analyzed_pitch_t *a = va, *b = vb;
 
     return a->midi_pitch - b->midi_pitch;
 }
 
 static int
-_compare_analyzed_note_by_relative_pitch (const void *va, const void *vb)
+_compare_analyzed_pitch_by_relative_pitch (const void *va, const void *vb)
 {
-    const analyzed_note_t *a = va, *b = vb;
+    const analyzed_pitch_t *a = va, *b = vb;
 
     return a->relative_pitch - b->relative_pitch;
 }
 
+typedef struct modified_degree
+{
+    int degree;
+    int modification;
+} modified_degree_t;
+
+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 scale. */
+    switch (scale_degree) {
+    case 1:
+       half_steps = 0;
+       break;
+    case 2:
+       half_steps = 2;
+       break;
+    case 3:
+       half_steps = 4;
+       break;
+    case 4:
+       half_steps = 5;
+       break;
+    case 5:
+       half_steps = 7;
+       break;
+    case 6:
+       half_steps = 9;
+       break;
+    case 7:
+       half_steps = 11;
+       break;
+    default:
+       fprintf (stderr, "Internal: Invalid degree %d\n", degree->degree);
+       exit (1);
+       break;
+    }
+
+    return half_steps + degree->modification;
+}
+
+/* Number of half steps from 'root' up to 'pitch' (that is, counting
+ * the steps up a chromatic scale), within the same octave. */
 static int
-_chord_signature_matches (analyzed_note_t *notes,
-                         int num_notes,
-                         int *signature_pitches,
-                         int num_signature_pitches)
+_pitch_from_root_in_half_steps (pitch_t pitch, pitch_t root)
 {
-    int i;
+    int root_midi = _pitch_to_midi (root);
+    int pitch_midi = _pitch_to_midi (pitch);
 
-    if (num_notes != num_signature_pitches)
-           return 0;
+    while (pitch_midi < root_midi)
+       pitch_midi += 12;
+
+    return (pitch_midi - root_midi) % 12;
+}
 
-    for (i = 0; i < num_notes; i++)
-       if (notes[i].relative_pitch != signature_pitches[i])
+static int
+_chord_signature_matches (analyzed_pitch_t *pitches,
+                         int num_pitches,
+                         modified_degree_t *degrees,
+                         int num_degrees,
+                         int inversion,
+                         pitch_t *root)
+{
+#define MAX_DEGREES 6
+    int relative_pitches[MAX_DEGREES];
+    int i, root_index;
+
+    assert (num_degrees <= MAX_DEGREES);
+
+    if (num_pitches != num_degrees)
+       return 0;
+
+    if (inversion >= num_degrees)
+       return 0;
+
+    /* We never spell simple intervals as inversions. */
+    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_pitches; i++)
+       if (pitches[i].relative_pitch != relative_pitches[i])
            return 0;
 
+    root_index = (num_pitches - inversion) % num_pitches;
+    *root = pitches[root_index].pitch;
+
     return 1;
 }
 
+typedef struct chord_signature
+{
+    int num_degrees;
+    modified_degree_t degrees[MAX_DEGREES];
+    const char *name;
+} chord_signature_t;
+
+/* 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>"
+
+chord_signature_t signatures[] = {
+    /* Octave */
+    { 1, {{1, 0}}, "P8" },
+
+    /* Intervals */
+    { 2, {{1, 0}, {2, -1}}, "m2" },
+    { 2, {{1, 0}, {2,  0}}, "M2" },
+    { 2, {{1, 0}, {3, -1}}, "m3" },
+    { 2, {{1, 0}, {3,  0}}, "M3" },
+    { 2, {{1, 0}, {4,  0}}, "P4" },
+    { 2, {{1, 0}, {5, -1}}, "Tri-tone" },
+    { 2, {{1, 0}, {5,  0}}, "P5" },
+    { 2, {{1, 0}, {6, -1}}, "m6" },
+    { 2, {{1, 0}, {6,  0}}, "M6" },
+    { 2, {{1, 0}, {7, -1}}, "m7" },
+    { 2, {{1, 0}, {7,  0}}, "M7" },
+
+    /* Triads */
+    { 3, {{1, 0}, {4,  0}, {5,  0}}, "sus" },
+    { 3, {{1, 0}, {3,  0}, {5, +1}}, SUP "+" PUS },
+    { 3, {{1, 0}, {3,  0}, {5,  0}}, "" },
+    { 3, {{1, 0}, {3, -1}, {5,  0}}, "m" },
+    { 3, {{1, 0}, {3, -1}, {5, -1}}, "°" },
+    { 3, {{1, 0}, {2,  0}, {5,  0}}, "msus2" },
+
+    /* Seventh chords with no 3rd */
+    { 3, {{1, 0}, {5, +1}, {7,  0}}, SUP "+M7" PUS },
+    { 3, {{1, 0}, {5, +1}, {7, -1}}, SUP "+7" PUS },
+    { 3, {{1, 0}, {5,  0}, {7,  0}}, "M7" },
+    { 3, {{1, 0}, {5,  0}, {7, -1}}, "7" },
+    { 3, {{1, 0}, {5, -1}, {7, -1}}, "7♭5" },
+
+    /* Seventh chords with no 5th */
+    { 3, {{1, 0}, {4,  0}, {7,  0}}, "M7sus" },
+    { 3, {{1, 0}, {4,  0}, {7, -1}}, "7sus" },
+    { 3, {{1, 0}, {3,  0}, {7,  0}}, "M7" },
+    { 3, {{1, 0}, {3,  0}, {7, -1}}, "7" },
+    { 3, {{1, 0}, {3, -1}, {7,  0}}, "m" SUP "M7" PUS },
+    { 3, {{1, 0}, {3, -1}, {7, -1}}, "m7" },
+    { 3, {{1, 0}, {2,  0}, {7,  0}}, "m" SUP "M7" PUS "sus2" },
+    { 3, {{1, 0}, {2,  0}, {7, -1}}, "m7sus2" },
+
+    /* Sixth chords */
+    { 4, {{1, 0}, {4,  0}, {5,  0}, {6,  0}}, "6sus" },
+    { 4, {{1, 0}, {3,  0}, {5,  0}, {6,  0}}, "6" }, /* Ambiguous with m7 */
+    { 4, {{1, 0}, {3, -1}, {5,  0}, {6,  0}}, "m6" },
+    { 4, {{1, 0}, {2,  0}, {5,  0}, {6,  0}}, "m6sus2" },
+
+    /* Seventh chords */
+    { 4, {{1, 0}, {4,  0}, {5, +1}, {7,  0}}, SUP "+M7" PUS "sus" },
+    { 4, {{1, 0}, {4,  0}, {5, +1}, {7, -1}}, SUP "+7" PUS "sus" },
+    { 4, {{1, 0}, {4,  0}, {5,  0}, {7,  0}}, "M7sus" },
+    { 4, {{1, 0}, {4,  0}, {5,  0}, {7, -1}}, "7sus" },
+    { 4, {{1, 0}, {4,  0}, {5, -1}, {7, -1}}, "7♭5sus" },
+    { 4, {{1, 0}, {3,  0}, {5, +1}, {7,  0}}, SUP "+M7" PUS },
+    { 4, {{1, 0}, {3,  0}, {5, +1}, {7, -1}}, SUP "+7" PUS },
+    { 4, {{1, 0}, {3,  0}, {5,  0}, {7,  0}}, "M7" },
+    { 4, {{1, 0}, {3,  0}, {5,  0}, {7, -1}}, "7" },
+    { 4, {{1, 0}, {3,  0}, {5, -1}, {7, -1}}, "7♭5" },
+    { 4, {{1, 0}, {3, -1}, {5,  0}, {7,  0}}, "m" SUP "M7" PUS },
+    { 4, {{1, 0}, {3, -1}, {5,  0}, {7, -1}}, "m7" },
+    { 4, {{1, 0}, {3, -1}, {5, -1}, {7,  0}}, "°" SUP "M7" PUS },
+    { 4, {{1, 0}, {3, -1}, {5, -1}, {7, -1}}, "𝆩" SUP "7" PUS },
+    { 4, {{1, 0}, {3, -1}, {5, -1}, {7, -2}}, "°" SUP "7" PUS },
+    { 4, {{1, 0}, {2,  0}, {5,  0}, {7,  0}}, "m" SUP "M7" PUS "sus2" },
+    { 4, {{1, 0}, {2,  0}, {5,  0}, {7, -1}}, "m7sus2" },
+    { 4, {{1, 0}, {2,  0}, {5, -1}, {7,  0}}, "°" SUP "M7" PUS "sus2" },
+    { 4, {{1, 0}, {2,  0}, {5, -1}, {7, -1}}, "𝆩" SUP "7" PUS "sus2" },
+    { 4, {{1, 0}, {2,  0}, {5, -1}, {7, -2}}, "°" SUP "7" PUS "sus2" }, /* Ambiguous with 7 */
+
+    /* The sorting for chords with degree higher than 9 is funny
+     * to keep the array in degree order after reducing each
+     * degree to an actual scale degree, (9 -> 2, 11 -> 4, 13 ->
+     * 6) */
+
+    /* Ninth chords voiced with no 5th */
+    { 4, {{1, 0}, {9,  0}, {4,  0}, {7,  0}}, "M9sus" },
+    { 4, {{1, 0}, {9,  0}, {4,  0}, {7, -1}}, "9sus" },
+    { 4, {{1, 0}, {9,  0}, {3,  0}, {7,  0}}, "M9" },
+    { 4, {{1, 0}, {9,  0}, {3,  0}, {7, -1}}, "9" },
+    { 4, {{1, 0}, {9,  0}, {3, -1}, {7,  0}}, "m" SUP "M9" PUS },
+    { 4, {{1, 0}, {9,  0}, {3, -1}, {7, -1}}, "m9" },
+    { 4, {{1, 0}, {9, -1}, {3, -1}, {7, -1}}, "m♭9" },
+    { 4, {{1, 0}, {9,  0}, {3, -1}, {6,  0}}, "m6/9" },
+    { 4, {{1, 0}, {9, -1}, {3, -1}, {6,  0}}, "m6/♭9" },
+
+    /* Sixth plus 9 */
+    { 5, {{1, 0}, {9,  0}, {3,  0}, {5,  0}, {6,  0}}, "6/9" },
+    { 5, {{1, 0}, {9,  0}, {3, -1}, {5,  0}, {6,  0}}, "m6/9" },
+    { 5, {{1, 0}, {9, -1}, {3, -1}, {5,  0}, {6,  0}}, "m6/♭9" },
+
+    /* Seventh plus altered 9 */
+    { 5, {{1, 0}, {9, +1}, {3,  0}, {5,  0}, {7, -1}}, "7♯9" },
+    { 5, {{1, 0}, {9, -1}, {3,  0}, {5,  0}, {7, -1}}, "7♭9" },
+    { 5, {{1, 0}, {9, +1}, {3,  0}, {5, -1}, {7, -1}}, "7♭5♯9" },
+    { 5, {{1, 0}, {9, -1}, {3,  0}, {5, -1}, {7, -1}}, "7♭5♭9" },
+
+    /* Ninth chords */
+    { 5, {{1, 0}, {9,  0}, {4,  0}, {5, +1}, {7,  0}}, SUP "+M9" PUS "sus" },
+    { 5, {{1, 0}, {9,  0}, {4,  0}, {5, +1}, {7, -1}}, SUP "+9" PUS "sus" },
+    { 5, {{1, 0}, {9,  0}, {4,  0}, {5,  0}, {7,  0}}, "M9sus" },
+    { 5, {{1, 0}, {9,  0}, {4,  0}, {5,  0}, {7, -1}}, "9sus" },
+    { 5, {{1, 0}, {9,  0}, {3,  0}, {5, +1}, {7,  0}}, SUP "+M9" PUS },
+    { 5, {{1, 0}, {9,  0}, {3,  0}, {5, +1}, {7, -1}}, SUP "+9" PUS },
+    { 5, {{1, 0}, {9,  0}, {3,  0}, {5,  0}, {7,  0}}, "M9" },
+    { 5, {{1, 0}, {9,  0}, {3,  0}, {5,  0}, {7, -1}}, "9" },
+    { 5, {{1, 0}, {9,  0}, {3, -1}, {5,  0}, {7,  0}}, "m" SUP "M9" PUS },
+    { 5, {{1, 0}, {9,  0}, {3, -1}, {5,  0}, {7, -1}}, "m9" },
+    { 5, {{1, 0}, {9, -1}, {3, -1}, {5,  0}, {7, -1}}, "m♭9" },
+    { 5, {{1, 0}, {9,  0}, {3,  0}, {5, -1}, {7,  0}}, "M9" SUP "♭5" PUS },
+    { 5, {{1, 0}, {9,  0}, {3,  0}, {5, -1}, {7, -1}}, "9" SUP "♭5" PUS },
+    { 5, {{1, 0}, {9,  0}, {3, -1}, {5, -1}, {7,  0}}, "m" SUP "M9♭5" PUS },
+    { 5, {{1, 0}, {9,  0}, {3, -1}, {5, -1}, {7, -1}}, "𝆩" SUP "9" PUS },
+    { 5, {{1, 0}, {9, -1}, {3, -1}, {5, -1}, {7, -1}}, "𝆩" SUP "♭9" PUS },
+    { 5, {{1, 0}, {9,  0}, {3, -1}, {5, -1}, {7, -2}}, "°" SUP "9" PUS },
+    { 5, {{1, 0}, {9, -1}, {3, -1}, {5, -1}, {7, -2}}, "°" SUP "♭9" PUS },
+
+    /* Eleventh chords */
+    { 6, {{1, 0}, {9,  0}, {3,  0}, {11,  0}, {5, +1}, {7,  0}}, SUP "+M11" PUS },
+    { 6, {{1, 0}, {9,  0}, {3,  0}, {11,  0}, {5, +1}, {7, -1}}, SUP "+11" PUS },
+    { 6, {{1, 0}, {9,  0}, {3,  0}, {11,  0}, {5,  0}, {7,  0}}, "M11" },
+    { 6, {{1, 0}, {9,  0}, {3,  0}, {11,  0}, {5,  0}, {7, -1}}, "11" },
+    { 6, {{1, 0}, {9,  0}, {3, -1}, {11,  0}, {5,  0}, {7,  0}}, "m" SUP "M11" PUS },
+    { 6, {{1, 0}, {9,  0}, {3, -1}, {11,  0}, {5,  0}, {7, -1}}, "m11" },
+    { 6, {{1, 0}, {9, -1}, {3, -1}, {11,  0}, {5, -1}, {7, -1}}, "𝆩" SUP "11" PUS },
+    { 6, {{1, 0}, {9, -1}, {3, -1}, {11, -1}, {5, -1}, {7, -2}}, "°" SUP "11" PUS }
+};
+
 static void
-scherzo_analyze_chord (scherzo_t *scherzo)
+scherzo_adjust_pitch_to_match_modified_degree (pitch_t *pitch,
+                                              pitch_t root,
+                                              modified_degree_t *degree)
 {
-    void *local = talloc_new (NULL);
-    analyzed_note_t *notes;
-    note_group_t *note_group;
-    unsigned i, j, num_notes;
-    int bass_pitch;
-    const char *chord_name = NULL;
+    pitch_name_t name = PITCH_NAME (*pitch);
+    pitch_accidental_t accidental = PITCH_ACCIDENTAL (*pitch);
+    int octave = PITCH_OCTAVE (*pitch);
+    int degree_zero_based = (degree->degree - 1) % 7;
+    int degree_delta;
 
-    if (scherzo->pedal_pressed)
-       note_group = &scherzo->notes_pedaled;
-    else
-       note_group = &scherzo->notes_pressed;
-
-    num_notes = note_group->num_notes;
-
-    struct { int pitches[1]; const char *name; } octaves[] = {
-       { {0}, "Octave"}
-    };
-
-    struct { int pitches[2]; const char *name; } intervals[] = {
-       { {0, 1}, "Minor 2nd"},
-       { {0, 2}, "Major 2nd"},
-       { {0, 3}, "Minor 3rd"},
-       { {0, 4}, "Major 3rd"},
-       { {0, 5}, "Perfect 4th"},
-       { {0, 6}, "Diminished 5th/Augmented 4th"},
-       { {0, 7}, "Perfect 5th"},
-       { {0, 8}, "Minor 6th"},
-       { {0, 9}, "Major 6th"},
-       { {0, 10}, "Minor 7th"},
-       { {0, 11}, "Major 7th"}
-    };
-
-    struct { int pitches[3]; const char *name; } triads[] = {
-       { {0, 4, 8}, "Augmented triad" },
-       { {0, 4, 7}, "Major triad" },
-       { {0, 3, 7}, "Minor triad" },
-       { {0, 3, 6}, "Diminished triad" }
-    };
-
-    struct { int pitches[4]; const char *name; } sevenths[] = {
-       { {0, 4, 8, 11}, "Augmented/major 7" },
-       { {0, 4, 8, 10}, "Augmented 7" },
-       { {0, 4, 7, 11}, "Major 7" },
-       { {0, 4, 7, 10}, "Dominant 7" },
-       { {0, 3, 7, 11}, "Minor/major 7" },
-       { {0, 3, 7, 10}, "Minor 7" },
-       { {0, 3, 6, 11}, "Diminished/major 7" },
-       { {0, 3, 6, 10}, "Half-diminished 7" },
-       { {0, 3, 6, 9},  "Diminished 7" }
-    };
-
-    if (scherzo->chord) {
-       score_remove_chord (scherzo->chord);
-       scherzo->chord = NULL;
+    int note_degree_zero_based = name - PITCH_NAME (root);
+    if (note_degree_zero_based < 0)
+       note_degree_zero_based += 7;
+
+    if (note_degree_zero_based == degree_zero_based)
+       return;
+
+    degree_delta = note_degree_zero_based - degree_zero_based;
+    if (degree_delta > 3)
+       degree_delta = - (7 - degree_delta);
+    if (degree_delta < -3)
+       degree_delta = - (-7 - degree_delta);
+
+    if (abs (degree_delta) != 1) {
+       fprintf (stderr, "Internal error: Cannot adjust a note more than one degree (%d vs. %d).\n", note_degree_zero_based + 1, degree->degree);
+       exit (1);
+    }
+
+    if (degree_delta == -1) {
+       if (name == PITCH_NAME_B) {
+           name = PITCH_NAME_C;
+           octave++;
+       } else {
+           name++;
+       }
+       switch (name) {
+       case PITCH_NAME_D:
+       case PITCH_NAME_E:
+       case PITCH_NAME_G:
+       case PITCH_NAME_A:
+       case PITCH_NAME_B:
+           accidental -= 2;
+           break;
+       case PITCH_NAME_C:
+       case PITCH_NAME_F:
+           accidental -= 1;
+           break;
+       }
     }
 
-    if (num_notes <= 1)
+    if (degree_delta == +1) {
+       if (name == PITCH_NAME_C) {
+           name = PITCH_NAME_B;
+           octave--;
+       } else {
+           name--;
+       }
+       switch (name) {
+       case PITCH_NAME_C:
+       case PITCH_NAME_D:
+       case PITCH_NAME_F:
+       case PITCH_NAME_G:
+       case PITCH_NAME_A:
+           accidental += 2;
+           break;
+       case PITCH_NAME_E:
+       case PITCH_NAME_B:
+           accidental += 1;
+       }
+    }
+
+    if (accidental < 0 || accidental > PITCH_ACCIDENTAL_DOUBLE_SHARP) {
+       fprintf (stderr, "Internal error: Trying to adjust an accidental out of range (degree_delta == %d (%d - %d), new accidental == %d).\n", degree_delta, note_degree_zero_based, degree_zero_based, accidental);
+       exit (1);
+    }
+
+    *pitch = PITCH (name, accidental, octave);
+}
+
+static void
+_spell_chord_by_signature (pitch_group_t *chord,
+                          chord_signature_t *signature,
+                          pitch_t root)
+{
+    int i, degree, note_half_steps, degree_half_steps;
+    int root_midi;
+
+    /* Sanitize root to eliminate things like double-flats,
+     * double-sharps, Cb, E#, etc. */
+    root_midi = _pitch_to_midi (root);
+    root = _midi_to_pitch (root_midi);
+    
+    for (i = 0; i < chord->num_pitches; i++) {
+       note_half_steps = _pitch_from_root_in_half_steps (chord->pitches[i],
+                                                         root);
+       for (degree = 0; degree < signature->num_degrees; degree++) {
+           degree_half_steps = _modified_degree_to_half_steps (&signature->degrees[degree]);
+           if (note_half_steps == degree_half_steps) {
+               scherzo_adjust_pitch_to_match_modified_degree (&chord->pitches[i],
+                                                              root,
+                                                              &signature->degrees[degree]);
+               break;
+           }
+       }
+       if (note_half_steps != degree_half_steps) {
+           fprintf (stderr, "Internal error: Chord and degree mis-match\n");
+           exit (1);
+       }
+    }
+}
+
+static void
+_analyze_chord (pitch_group_t *chord,
+               chord_signature_t **signature_ret,
+               pitch_t **root_ret)
+{
+    void *local = talloc_new (NULL);
+    analyzed_pitch_t *pitches;
+    int i, j, num_pitches;
+    int bass_pitch, inversion;
+    pitch_t root;
+    chord_signature_t *match = NULL;
+
+    /* Default to no-match for any early return */
+    *signature_ret = NULL;
+    *root_ret = NULL;
+
+    num_pitches = chord->num_pitches;
+
+    if (chord->num_pitches <= 1)
        goto DONE;
 
-    notes = talloc_array (local, analyzed_note_t, num_notes);
-    if (notes == NULL)
+    pitches = talloc_array (local, analyzed_pitch_t, num_pitches);
+    if (pitches == NULL)
        goto DONE;
 
-    for (i = 0; i < num_notes; i++) {
-       score_note_t *note = note_group->notes[i];
-       notes[i].note = note;
-       notes[i].midi_pitch = _score_pitch_and_octave_to_midi (note->pitch,
-                                                              note->octave);
+    for (i = 0; i < num_pitches; i++) {
+       pitch_t pitch = chord->pitches[i];
+       pitches[i].pitch = pitch;
+       pitches[i].midi_pitch = _pitch_to_midi (pitch);
        /* Relative pitch will be filled in after sorting. */
-       notes[i].relative_pitch = 0;
+       pitches[i].relative_pitch = 0;
     }
 
     /* First, sort by midi pitch to find the bass note. */
-    qsort (notes, num_notes, sizeof (analyzed_note_t),
-          _compare_analyzed_note_by_midi_pitch);
+    qsort (pitches, num_pitches, sizeof (analyzed_pitch_t),
+          _compare_analyzed_pitch_by_midi_pitch);
     
-    bass_pitch = notes[0].midi_pitch;
+    bass_pitch = pitches[0].midi_pitch;
 
     /* With the bass note identified, we can find all relative pitches. */
-    for (i = 0; i < num_notes; i++) {
-       notes[i].relative_pitch = notes[i].midi_pitch - bass_pitch;
-       while (notes[i].relative_pitch >= 12)
-           notes[i].relative_pitch -= 12;
+    for (i = 0; i < num_pitches; i++) {
+       pitches[i].relative_pitch = pitches[i].midi_pitch - bass_pitch;
+       while (pitches[i].relative_pitch >= 12)
+           pitches[i].relative_pitch -= 12;
     }
 
     /* Now, sort again by relative pitch. */
-    qsort (notes, num_notes, sizeof (analyzed_note_t),
-          _compare_analyzed_note_by_relative_pitch);
+    qsort (pitches, num_pitches, sizeof (analyzed_pitch_t),
+          _compare_analyzed_pitch_by_relative_pitch);
 
     /* Finally, eliminate all duplicate notes. */
-    for (i = 0; i < num_notes - 1; i++) {
-           if (notes[i+1].relative_pitch == notes[i].relative_pitch) {
+    for (i = 0; i < num_pitches - 1; i++) {
+           if (pitches[i+1].relative_pitch == pitches[i].relative_pitch) {
                    j = i+1;
-                   while (j < num_notes &&
-                          notes[j].relative_pitch == notes[i].relative_pitch)
+                   while (j < num_pitches &&
+                          pitches[j].relative_pitch == pitches[i].relative_pitch)
                    {
                            j++;
                    }
@@ -547,178 +968,228 @@ scherzo_analyze_chord (scherzo_t *scherzo)
                     * the array bounds).*/
                    j--;
 
-                   if (j < num_notes - 1) {
-                           memmove (&notes[i+1], &notes[j+1],
-                                    (num_notes - j) * sizeof (analyzed_note_t));
+                   if (j < num_pitches - 1) {
+                           memmove (&pitches[i+1], &pitches[j+1],
+                                    (num_pitches - j) * sizeof (analyzed_pitch_t));
                    }
 
-                   num_notes -= (j - i);
+                   num_pitches -= (j - i);
            }
     }
 
-    for (i = 0; i < ARRAY_SIZE (octaves); i++) {
-       if (_chord_signature_matches (notes, num_notes, octaves[i].pitches, 1))
-           chord_name = octaves[i].name;
-    }
-
-    for (i = 0; i < ARRAY_SIZE (intervals); i++) {
-       if (_chord_signature_matches (notes, num_notes, intervals[i].pitches, 2))
-           chord_name = intervals[i].name;
-    }
-
-    for (i = 0; i < ARRAY_SIZE (triads); i++) {
-       if (_chord_signature_matches (notes, num_notes, triads[i].pitches, 3))
-           chord_name = triads[i].name;
+    for (inversion = 0; inversion < 4; inversion++) {
+       for (i = 0; i < ARRAY_SIZE (signatures); i++) {
+           if (_chord_signature_matches (pitches, num_pitches,
+                                         signatures[i].degrees,
+                                         signatures[i].num_degrees,
+                                         inversion, &root))
+           {
+               match = &signatures[i];
+               goto HAVE_MATCH;
+           }
+       }
     }
-
-    for (i = 0; i < ARRAY_SIZE(sevenths); i++) {
-       if (_chord_signature_matches (notes, num_notes, sevenths[i].pitches, 4))
-           chord_name = sevenths[i].name;
+HAVE_MATCH:
+
+    if (match) {
+       *signature_ret = match;
+       for (i = 0; i < chord->num_pitches; i++) {
+           if (chord->pitches[i] == root) {
+               *root_ret = &chord->pitches[i];
+               break;
+           }
+       }
     }
 
-    if (chord_name)
-       scherzo->chord = score_add_chord (scherzo->treble, chord_name);
-    else
-       scherzo->chord = score_add_chord (scherzo->treble, "Unknown or not a chord");
-
 DONE:
     talloc_free (local);
 }
 
 static void
-note_group_init (void *ctx, note_group_t *group)
+pitch_group_init (void *ctx, pitch_group_t *group)
 {
     group->ctx = ctx;
-    group->notes = NULL;
+    group->pitches = NULL;
     group->size = 0;
-    group->num_notes = 0;
+    group->num_pitches = 0;
 }
 
 static void
-note_group_add_note (note_group_t *group, score_note_t *note)
+pitch_group_add_pitch (pitch_group_t *group, pitch_t pitch)
 {
     int i;
 
-    /* Do nothing if note is already in group. */
-    for (i = 0; i < group->num_notes; i++) {
-       if (group->notes[i]->pitch == note->pitch &&
-           group->notes[i]->octave == note->octave)
-       {
+    /* Do nothing if pitch is already in group. */
+    for (i = 0; i < group->num_pitches; i++)
+       if (group->pitches[i] == pitch)
            return;
-       }
-    }
 
-    group->num_notes++;
+    group->num_pitches++;
 
-    if (group->num_notes > group->size) {
+    if (group->num_pitches > group->size) {
        group->size++;
-       group->notes = talloc_realloc (group->ctx, group->notes,
-                                      score_note_t*, group->size);
+       group->pitches = talloc_realloc (group->ctx, group->pitches,
+                                        pitch_t, group->size);
 
-       if (group->notes == NULL) {
+       if (group->pitches == NULL) {
            fprintf (stderr, "Out of memory.\n");
            exit (1);
        }
     }
 
-    group->notes[group->num_notes - 1] = note;
+    group->pitches[group->num_pitches - 1] = pitch;
 }
 
 static void
-note_group_remove_note_at (note_group_t *group, int i)
+pitch_group_remove_pitch_at (pitch_group_t *group, int i)
 {
-    if (i >= group->num_notes) {
-       fprintf (stderr, "Internal error: No note to remove at index %d\n", i);
+    if (i > group->num_pitches) {
+       fprintf (stderr, "Internal error: Cannot remove pitch %d from group with only %d %s\n",
+                i, group->num_pitches,
+                group->num_pitches == 1 ? "pitch" : "pitches");
        exit (1);
     }
 
-    if (i < group->num_notes - 1) {
-       memmove (group->notes + i, group->notes + i + 1,
-                (group->num_notes - 1 - i) * sizeof (score_note_t*));
+    if (i < group->num_pitches - 1) {
+       memmove (group->pitches + i, group->pitches + i + 1,
+                (group->num_pitches - 1 - i) * sizeof (pitch_t));
     }
-    group->num_notes--;
+
+    group->num_pitches--;
 }
 
-static score_note_t *
-scherzo_press_note (scherzo_t *scherzo, score_pitch_t pitch, int octave)
+/* Remove a 'pitch' from 'group', (including any enharmonic pitch)
+ * See also: pitch_group_remove_pitch
+ */
+static void
+pitch_group_remove_pitch_enharmonic (pitch_group_t *group, pitch_t pitch)
 {
-    score_staff_t *staff;
-    score_note_t *note;
     int i;
 
-    if (scherzo->challenge.note) {
-       staff = scherzo->challenge.staff;
-    } else if (octave >= 4) {
-       staff = scherzo->treble;
-    } else {
-       staff = scherzo->bass;
-    }
-
-    /* Do nothing if this note is already pressed. */
-    for (i = 0; i < scherzo->notes_pressed.num_notes; i++) {
-       if (scherzo->notes_pressed.notes[i]->pitch == pitch &&
-           scherzo->notes_pressed.notes[i]->octave == octave)
-       {
-           return scherzo->notes_pressed.notes[i];
-       }
-    }
+    for (i = 0; i < group->num_pitches; i++)
+       if (_pitches_are_enharmonic (group->pitches[i], pitch))
+           pitch_group_remove_pitch_at (group, i);
+}
 
-    note = score_add_note (staff, pitch, octave, SCORE_DURATION_WHOLE);
+/* Remove an exactly-matching pitch from 'group'
+ * See also: pitch_group_remove_pitch_enharmonic
+ *
+static void
+pitch_group_remove_pitch (pitch_group_t *group, pitch_t pitch)
+{
+    int i;
 
-    note_group_add_note (&scherzo->notes_pressed, note);
+    for (i = 0; i < group->num_pitches; i++)
+       if (group->pitches[i] == pitch)
+           pitch_group_remove_pitch_at (group, i);
+}
+*/
 
-    if (scherzo->pedal_pressed)
-       note_group_add_note (&scherzo->notes_pedaled, note);
+static void
+scherzo_update_notes_and_chord (scherzo_t *scherzo)
+{
+    if (scherzo->notes_pressed.num_pitches == 0 &&
+       scherzo->notes_pedaled.num_pitches == 0)
+    {
+       score_remove_notes (scherzo->score);
 
-    scherzo_analyze_chord (scherzo);
+       score_remove_chords (scherzo->score);
 
-    return note;
+       gtk_widget_queue_draw (scherzo->window);
+    }
 }
 
 static void
-scherzo_release_note (scherzo_t *scherzo, score_pitch_t pitch, int octave)
+scherzo_press_note (scherzo_t *scherzo, pitch_t pitch)
 {
-    score_note_t *note;
+    void *local = talloc_new (NULL);
+    pitch_group_t *chord;
+    chord_signature_t *signature;
+    char *chord_name = NULL;
+    pitch_t *root;
     int i;
-    int found = 0;
-
-    for (i = scherzo->notes_pressed.num_notes - 1; i >=0; i--) {
-       note = scherzo->notes_pressed.notes[i];
-       if (note->pitch == pitch && note->octave == octave) {
-           found = 1;
-           if (! scherzo->pedal_pressed)
-               score_remove_note (note);
-           note_group_remove_note_at (&scherzo->notes_pressed, i);
+
+    /* Do nothing if this note is already pressed. */
+    for (i = 0; i < scherzo->notes_pressed.num_pitches; i++)
+       if (scherzo->notes_pressed.pitches[i] == pitch)
+           return;
+
+    pitch_group_add_pitch (&scherzo->notes_pressed, pitch);
+
+    if (scherzo->pedal_pressed)
+       pitch_group_add_pitch (&scherzo->notes_pedaled, pitch);
+
+    /* Remove all notes/chords from score, then add current notes. */
+    score_remove_notes (scherzo->score);
+    score_remove_chords (scherzo->score);
+
+    if (scherzo->pedal_pressed)
+       chord = &scherzo->notes_pedaled;
+    else
+       chord = &scherzo->notes_pressed;
+
+    _analyze_chord (chord, &signature, &root);
+
+    if (signature) {
+       /* Don't print root pitch for octaves and inversions,
+        * (since a pitch name alone looks like a major triad) */
+       if (signature->num_degrees < 3) {
+           chord_name = talloc_strdup (local, signature->name);
+       } else {
+           _spell_chord_by_signature (&scherzo->notes_pressed,
+                                      signature, *root);
+           _spell_chord_by_signature (&scherzo->notes_pedaled,
+                                      signature, *root);
+
+           chord_name = talloc_asprintf (local, "%s%s",
+                                         pitch_string (*root),
+                                         signature->name);
        }
+    } else if (chord->num_pitches > 2) {
+       chord_name = talloc_strdup (local, "?");
     }
 
-    if (found == 0) {
-       fprintf (stderr, "Internal error: Failed to find note to release.\n");
+    if (chord_name)
+       scherzo->chord = score_add_chord (scherzo->treble, chord_name);
+
+    for (i = 0; i < scherzo->notes_pressed.num_pitches; i++)
+    {
+       score_add_note (scherzo->score, scherzo->notes_pressed.pitches[i],
+                       SCORE_DURATION_WHOLE);
     }
 
-    scherzo_analyze_chord (scherzo);
+    for (i = 0; i < scherzo->notes_pedaled.num_pitches; i++)
+    {
+       score_add_note (scherzo->score, scherzo->notes_pedaled.pitches[i],
+                       SCORE_DURATION_WHOLE);
+    }
+
+    talloc_free (local);
+}
+
+static void
+scherzo_release_note (scherzo_t *scherzo, pitch_t pitch)
+{
+    pitch_group_remove_pitch_enharmonic (&scherzo->notes_pressed, pitch);
+
+    if (scherzo->notes_pressed.num_pitches == 0)
+       scherzo_update_notes_and_chord (scherzo);
 }
 
-static score_note_t *
+static pitch_t
 scherzo_press_note_midi (scherzo_t *scherzo, unsigned char midi_note)
 {
-    score_pitch_t pitch;
-    int octave;
+    pitch_t pitch = _midi_to_pitch (midi_note);
 
-    _midi_to_score_pitch_and_octave (midi_note, &pitch, &octave);
+    scherzo_press_note (scherzo, pitch);
 
-    return scherzo_press_note (scherzo, pitch, octave);
+    return pitch;
 }
 
 static void
 scherzo_release_note_midi (scherzo_t *scherzo, unsigned char midi_note)
 {
-    score_pitch_t pitch;
-    int octave;
-    _midi_to_score_pitch_and_octave (midi_note, &pitch, &octave);
-
-    scherzo_release_note (scherzo, pitch, octave);
+    scherzo_release_note (scherzo, _midi_to_pitch (midi_note));
 }
 
 static void
@@ -726,56 +1197,44 @@ scherzo_press_pedal (scherzo_t *scherzo)
 {
     int i;
 
+    if (scherzo->pedal_pressed)
+       return;
+
     scherzo->pedal_pressed = 1;
 
     /* Copy all pressed notes to pedaled notes */
-    for (i = 0; i < scherzo->notes_pressed.num_notes; i++)
-       note_group_add_note (&scherzo->notes_pedaled, scherzo->notes_pressed.notes[i]);
+    for (i = 0; i < scherzo->notes_pressed.num_pitches; i++)
+       pitch_group_add_pitch (&scherzo->notes_pedaled, scherzo->notes_pressed.pitches[i]);
 }
 
 static void
 scherzo_release_pedal (scherzo_t *scherzo)
 {
-    score_note_t *note, *new_note;
     int i;
 
-    /* Make new notes in score for all pressed notes. */
-    for (i = 0; i < scherzo->notes_pressed.num_notes; i++) {
-       note = scherzo->notes_pressed.notes[i];
-       new_note = score_add_note (note->staff, note->pitch, note->octave, note->duration);
-       scherzo->notes_pressed.notes[i] = new_note;
-    }
+    if (! scherzo->pedal_pressed)
+       return;
 
-    /* Then remove all previously pedaled notes from the score. */
-    for (i = scherzo->notes_pedaled.num_notes - 1; i >=0; i--) {
-       note = scherzo->notes_pedaled.notes[i];
-       score_remove_note (note);
-       note_group_remove_note_at (&scherzo->notes_pedaled, i);
-    }
+    /* Remove all previously pedaled notes. */
+    for (i = scherzo->notes_pedaled.num_pitches - 1; i >=0; i--)
+       pitch_group_remove_pitch_at (&scherzo->notes_pedaled, i);
 
     scherzo->pedal_pressed = 0;
 
-    scherzo_analyze_chord (scherzo);
-
-    gtk_widget_queue_draw (scherzo->window);
+    scherzo_update_notes_and_chord (scherzo);
 }
 
-void
+static void
 _select_challenge (scherzo_t *scherzo)
 {
     category_t *category_unused;
     bool_t introduced_unused;
     item_t *item;
     challenge_t *challenge = &scherzo->challenge;
-    score_pitch_t pitch;
+    pitch_name_t pitch_name;
     int octave;
     char *s;
 
-    if (challenge->note) {
-       score_remove_note (challenge->note);
-       challenge->note = NULL;
-    }
-
     mnemon_select_item (&scherzo->mnemon,
                        &challenge->bin,
                        &challenge->item_index,
@@ -800,25 +1259,25 @@ _select_challenge (scherzo_t *scherzo)
 
     switch (*s) {
     case 'C':
-       pitch = SCORE_PITCH_VALUE(C, NATURAL);
+       pitch_name = PITCH_NAME_C;
        break;
     case 'D':
-       pitch = SCORE_PITCH_VALUE(D, NATURAL);
+       pitch_name = PITCH_NAME_D;
        break;
     case 'E':
-       pitch = SCORE_PITCH_VALUE(E, NATURAL);
+       pitch_name = PITCH_NAME_E;
        break;
     case 'F':
-       pitch = SCORE_PITCH_VALUE(F, NATURAL);
+       pitch_name = PITCH_NAME_F;
        break;
     case 'G':
-       pitch = SCORE_PITCH_VALUE(G, NATURAL);
+       pitch_name = PITCH_NAME_G;
        break;
     case 'A':
-       pitch = SCORE_PITCH_VALUE(A, NATURAL);
+       pitch_name = PITCH_NAME_A;
        break;
     case 'B':
-       pitch = SCORE_PITCH_VALUE(B, NATURAL);
+       pitch_name = PITCH_NAME_B;
        break;
     default:
        fprintf (stderr, "Malformed pitch name: %s (expected 'A' - 'G')\n", s);
@@ -833,34 +1292,26 @@ _select_challenge (scherzo_t *scherzo)
 
     octave = *s - '0';
 
-    challenge->note = score_add_note (challenge->staff, pitch, octave,
-                                     SCORE_DURATION_WHOLE);
+    challenge->pitch = PITCH (pitch_name, PITCH_ACCIDENTAL_NATURAL, octave);
+
+    challenge->active = 1;
     challenge->satisfied = 0;
     challenge->mistaken = 0;
 }
 
 /* Determine whether the user hit the correct note. */
 static void
-_judge_note (scherzo_t *scherzo, score_note_t *note)
+_judge_pitch (scherzo_t *scherzo, pitch_t pitch)
 {
     challenge_t *challenge = &scherzo->challenge;
 
-    if (! scherzo->challenge.note) {
-       score_set_note_color_rgb (note, 0.0, 0.0, 0.0); /* black */
+    if (! challenge->active)
        return;
-    }
 
-    if (note->pitch == challenge->note->pitch &&
-       note->octave == challenge->note->octave)
-    {
+    if (pitch == challenge->pitch)
        challenge->satisfied = 1;
-       score_set_note_color_rgb (note, 18/256., 130/256., 28/256.); /* green */
-    }
     else
-    {
        challenge->mistaken = 1;
-       score_set_note_color_rgb (note, 184/256., 4/256., 22/256.); /* red */
-    }
 }
 
 /* If the user got the right note (eventually), then score it in
@@ -870,7 +1321,7 @@ _score_challenge (scherzo_t *scherzo)
 {
     challenge_t *challenge = &scherzo->challenge;
 
-    if (! challenge->note)
+    if (! challenge->active)
        return;
 
     if (! challenge->satisfied)
@@ -891,7 +1342,7 @@ on_midi_input (unused (GIOChannel *channel),
     scherzo_t *scherzo = user_data;
     ssize_t remaining;
     snd_seq_event_t event;
-    score_note_t *note;
+    pitch_t pitch;
     int need_redraw = FALSE;
 
     remaining = read (scherzo->midi_fd, buf, MIDI_BUF_SIZE);
@@ -911,8 +1362,8 @@ on_midi_input (unused (GIOChannel *channel),
            /* Incomplete event. Nothing to do. */
            break;
        case SND_SEQ_EVENT_NOTEON:
-           note = scherzo_press_note_midi (scherzo, event.data.note.note);
-           _judge_note (scherzo, note);
+           pitch = scherzo_press_note_midi (scherzo, event.data.note.note);
+           _judge_pitch (scherzo, pitch);
            need_redraw = TRUE;
            break;
        case SND_SEQ_EVENT_NOTEOFF:
@@ -979,16 +1430,19 @@ main (int argc, char *argv[])
     /* Default to octave 4 for computer keyboard keypresses. */
     scherzo.keyboard_octave = 4;
 
-    note_group_init (scherzo.ctx, &scherzo.notes_pressed);
-    note_group_init (scherzo.ctx, &scherzo.notes_pedaled);
+    pitch_group_init (scherzo.ctx, &scherzo.notes_pressed);
+    pitch_group_init (scherzo.ctx, &scherzo.notes_pedaled);
 
     scherzo.pedal_pressed = 0;
 
+    scherzo.key = PITCH_CLASS_LITERAL (C, NATURAL);
+    score_set_key (scherzo.score, scherzo.key);
+
     mnemon_init (&scherzo.mnemon);
     /* XXX: Should create a default file if one cannot be loaded. */
     mnemon_load_category (&scherzo.mnemon, "scherzo-notes");
 
-    scherzo.challenge.note = NULL;
+    scherzo.challenge.active = 0;
 /*
     _select_challenge (&scherzo);
 */