adapter.cpp

@@ -16,7 +16,7 @@ VailAdapter::VailAdapter(unsigned int PiezoPin) {
 
 // Send a MIDI Key Event
 void VailAdapter::midiKey(uint8_t key, bool down) {
-    midiEventPacket_t event = {uint8_t(down?9:8), uint8_t(down?0x90:0x80), key, 0x7f};
+    midiEventPacket_t event = {down?9:8, down?0x90:0x80, key, 0x7f};
     MidiUSB.sendMIDI(event);
     MidiUSB.flush();
 }
@@ -97,16 +97,10 @@ void VailAdapter::HandleMIDI(midiEventPacket_t event) {
                 break;
             case 1: // set dit duration (0-254) *2ms
                 this->ditDuration = event.byte3 * 2 * MILLISECOND;
-                if (this->keyer) {
-                    this->keyer->SetDitDuration(this->ditDuration);
-                }
                 break;
         }
         break;
     case 0xC0: // Program Change
-        if (this->keyer) {
-            this->keyer->Release();
-        }
         this->keyer = GetKeyerByNumber(event.byte2, this);
         break;
     case 0x80: // Note off
@@ -117,9 +111,3 @@ void VailAdapter::HandleMIDI(midiEventPacket_t event) {
         break;
     }
 }
-
-void VailAdapter::Tick(unsigned millis) {
-    if (this->keyer) {
-        this->keyer->Tick(millis);
-    }
-}

adapter.h

@@ -22,5 +22,4 @@ public:
     void HandleMIDI(midiEventPacket_t event);
     void BeginTx();
     void EndTx();
-    void Tick(unsigned millis);
 };

keyers.cpp

@@ -35,7 +35,7 @@ public:
             return;
         }
         for (int i = 0; i < arrlen; i++) {
-            if (arr[arrlen] == val) {
+            if (arr[arrlen] == i) {
                 return;
             }
         }
@@ -51,28 +51,21 @@ public:
     unsigned int ditDuration;
     bool txRelays[2];
 
-    StraightKeyer() {
+    StraightKeyer(Transmitter *output) {
+        this->output = output;
+        this->ditDuration = 100;
         this->Reset();
     }
 
-    void SetOutput(Transmitter *output) {
-        this->output = output;
-    }
-
     void Reset() {
-        if (this->output) {
         this->output->EndTx();
     }
-        this->ditDuration = 100;
-    }
 
     void SetDitDuration(int duration) {
         this->ditDuration = duration;
     }
 
-    void Release() {
-        this->Reset();
-    }
+    void Release() {}
 
     bool TxClosed() {
         for (int i = 0; i < len(this->txRelays); i++) {
@@ -110,14 +103,14 @@ public:
 
 class BugKeyer: public StraightKeyer {
 public:
-    unsigned int nextPulse = 0;
+    unsigned int pulseTime = 0;
     bool keyPressed[2];
 
     using StraightKeyer::StraightKeyer;
 
     void Reset() {
         StraightKeyer::Reset();
-        this->nextPulse = 0;
+        this->pulseTime = 0;
         this->keyPressed[0] = false;
         this->keyPressed[1] = false;
     }
@@ -131,28 +124,26 @@ public:
         }
     }
 
-    void Tick(unsigned int millis) {
-        if (this->nextPulse && (millis >= this->nextPulse)) {
-            this->pulse(millis);
-        }
-    }
-
     void beginPulsing() {
-        if (!this->nextPulse) {
-            this->nextPulse = 1;
-        }
+        this->pulseTime = 1;
     }
 
-    virtual void pulse(unsigned int millis) {
+    void pulse(unsigned int millis) {
         if (this->TxClosed(0)) {
             this->Tx(0, false);
         } else if (this->keyPressed[0]) {
             this->Tx(0, true);
         } else {
-            this->nextPulse = 0;
+            this->pulseTime = 0;
             return;
         }
-        this->nextPulse = millis + this->ditDuration;
+        this->pulseTime = millis + this->ditDuration;
+    }
+
+    void Tick(unsigned int millis) {
+        if (this->pulseTime && (millis >= this->pulseTime)) {
+            this->pulse(millis);
+        }
     }
 };
 
@@ -181,34 +172,34 @@ public:
         this->keyPressed[key] = pressed;
         if (pressed) {
             this->nextRepeat = key;
-            this->beginPulsing();
         } else {
             this->nextRepeat = this->whichKeyPressed();
         }
+        this->beginPulsing();
     }
 
     unsigned int keyDuration(int key) {
         switch (key) {
-        case PADDLE_DIT:
+        case 0:
             return this->ditDuration;
-        case PADDLE_DAH:
-            return 3 * (this->ditDuration);
+        case 1:
+            return 3 * this->ditDuration;
         }
-        return this->ditDuration; // XXX
+        return 0;
     }
 
-    virtual int nextTx() {
+    int nextTx() {
         if (this->whichKeyPressed() == -1) {
             return -1;
         }
         return this->nextRepeat;
     }
 
-    virtual void pulse(unsigned int millis) {
+    void pulse(unsigned int millis) {
         int nextPulse = 0;
         if (this->TxClosed(0)) {
             // Pause if we're currently transmitting
-            nextPulse = this->keyDuration(PADDLE_DIT);
+            nextPulse = this->ditDuration;
             this->Tx(0, false);
         } else {
             int next = this->nextTx();
@@ -219,28 +210,33 @@ public:
         }
 
         if (nextPulse) {
-            this->nextPulse = millis + nextPulse;
+            this->pulseTime = millis + nextPulse;
         } else {
-            this->nextPulse = 0;
+            this->pulseTime = 0;
         }
     }
 };
 
 class UltimaticKeyer: public ElBugKeyer {
 public:
-    QSet queue;
+    QSet *queue;
     
     using ElBugKeyer::ElBugKeyer;
 
+    void Reset() {
+        ElBugKeyer::Reset();
+        this->queue = new QSet();
+    }
+
     void Key(Paddle key, bool pressed) {
         if (pressed) {
-            this->queue.add(key);
+            this->queue->add(key);
         }
         ElBugKeyer::Key(key, pressed);
     }
 
-    virtual int nextTx() {
-        int key = this->queue.shift();
+    int nextTx() {
+        int key = this->queue->shift();
         if (key != -1) {
             return key;
         }
@@ -250,19 +246,24 @@ public:
 
 class SingleDotKeyer: public ElBugKeyer {
 public:
-    QSet queue;
+    QSet *queue;
 
     using ElBugKeyer::ElBugKeyer;
 
+    void Reset() {
+        ElBugKeyer::Reset();
+        this->queue = new QSet();
+    }
+    
     void Key(Paddle key, bool pressed) {
-        if (pressed && (key == PADDLE_DIT)) {
-            this->queue.add(key);
+        if (pressed && (key == 0)) {
+            this->queue->add(key);
         }
         ElBugKeyer::Key(key, pressed);
     }
 
-    virtual int nextTx() {
-        int key = this->queue.shift();
+    int nextTx() {
+        int key = this->queue->shift();
         if (key != -1) {
             return key;
         }
@@ -277,9 +278,9 @@ public:
 
     using ElBugKeyer::ElBugKeyer;
 
-    virtual int nextTx() {
+    int nextTx() {
         int next = ElBugKeyer::nextTx();
-        if (this->keyPressed[PADDLE_DIT] && this->keyPressed[PADDLE_DAH]) {
+        if (this->whichKeyPressed() != -1) {
             this->nextRepeat = 1 - this->nextRepeat;
         }
         return next;
@@ -288,20 +289,25 @@ public:
 
 class IambicAKeyer: public IambicKeyer {
 public:
-    QSet queue;
+    QSet *queue;
 
     using IambicKeyer::IambicKeyer;
     
+    void Reset() {
+        IambicKeyer::Reset();
+        this->queue = new QSet();
+    }
+
     void Key(Paddle key, bool pressed) {
-        if (pressed && (key == PADDLE_DIT)) {
-            this->queue.add(key);
+        if (pressed && (key == 0)) {
+            this->queue->add(key);
         }
         IambicKeyer::Key(key, pressed);
     }
 
-    virtual int nextTx() {
+    int nextTx() {
         int next = IambicKeyer::nextTx();
-        int key = this->queue.shift();
+        int key = this->queue->shift();
         if (key != -1) {
             return key;
         }
@@ -311,29 +317,30 @@ public:
 
 class IambicBKeyer: public IambicKeyer {
 public:
-    QSet queue;
+    QSet *queue;
 
     using IambicKeyer::IambicKeyer;
 
     void Reset() {
         IambicKeyer::Reset();
+        this->queue = new QSet();
     }
 
     void Key(Paddle key, bool pressed) {
         if (pressed) {
-            this->queue.add(key);
+            this->queue->add(key);
         }
         IambicKeyer::Key(key, pressed);
     }
 
-    virtual int nextTx() {
-        for (int key = 0; key < len(this->keyPressed); key++) {
+    int nextTx() {
+        for (int key = 0; key < 2; key++) {
             if (this->keyPressed[key]) {
-                this->queue.add(key);
+                this->queue->add(key);
             }
         }
 
-        return this->queue.shift();
+        return this->queue->shift();
     }
 };
 
@@ -358,7 +365,7 @@ public:
         ElBugKeyer::Key(key, pressed);
     }
 
-    virtual int nextTx() {
+    int nextTx() {
         if (this->qlen > 0) {
             int next = this->queue[0];
             this->qlen--;
@@ -371,34 +378,27 @@ public:
     }
 };
 
-StraightKeyer straightKeyer = StraightKeyer();
-BugKeyer bugKeyer = BugKeyer();
-ElBugKeyer elBugKeyer = ElBugKeyer();
-SingleDotKeyer singleDotKeyer = SingleDotKeyer();
-UltimaticKeyer ultimaticKeyer = UltimaticKeyer();
-IambicKeyer iambicKeyer = IambicKeyer();
-IambicAKeyer iambicAKeyer = IambicAKeyer();
-IambicBKeyer iambicBKeyer = IambicBKeyer();
-KeyaheadKeyer keyaheadKeyer = KeyaheadKeyer();
-
-Keyer *keyers[] = {
-    &straightKeyer,
-    &bugKeyer,
-    &elBugKeyer,
-    &singleDotKeyer,
-    &ultimaticKeyer,
-    &iambicKeyer,
-    &iambicAKeyer,
-    &iambicBKeyer,
-    &keyaheadKeyer,
-};
-
 Keyer *GetKeyerByNumber(int n, Transmitter *output) {
-    if (n >= len(keyers)) {
+    switch (n) {
+    case 1:
+        return new StraightKeyer(output);
+    case 2:
+        return new BugKeyer(output);
+    case 3:
+        return new ElBugKeyer(output);
+    case 4:
+        return new SingleDotKeyer(output);
+    case 5:
+        return new UltimaticKeyer(output);
+    case 6:
+        return new IambicKeyer(output);
+    case 7:
+        return new IambicAKeyer(output);
+    case 8:
+        return new IambicBKeyer(output);
+    case 9:
+        return new KeyaheadKeyer(output);
+    default:
         return NULL;
     }
-
-    Keyer *k = keyers[n];
-    k->SetOutput(output);
-    return k;
 }

keyers.h

@@ -4,7 +4,7 @@
 
 typedef enum {
     PADDLE_DIT = 0,
-    PADDLE_DAH = 1,
+    PADDLE_DAH,
     PADDLE_STRAIGHT,
 } Paddle;
 
@@ -16,7 +16,6 @@ public:
 
 class Keyer {
 public:
-    virtual void SetOutput(Transmitter *output);
     virtual void Reset();
     virtual void SetDitDuration(int d);
     virtual void Release();

touchbounce.cpp

@@ -1,11 +1,11 @@
 #include "touchbounce.h"
 
 void TouchBounce::attach(int pin) {
-    this->qt = Adafruit_FreeTouch(pin, OVERSAMPLE_2, RESISTOR_50K, FREQ_MODE_NONE);
+    this->qt = Adafruit_FreeTouch(pin);
     this->qt.begin();
 }
 
 bool TouchBounce::readCurrentState() {
     int val = this->qt.measure();
-    return val > QT_THRESHOLD/2;
+    return val < QT_THRESHOLD;
 }

vail-adapter.ino

@@ -87,11 +87,8 @@ void setLED() {
 }
 
 void loop() {
-  unsigned now = millis();  
   midiEventPacket_t event = MidiUSB.read();
-
   setLED();
-  adapter.Tick(now);
 
   if (event.header) {
     adapter.HandleMIDI(event);
@@ -99,7 +96,7 @@ void loop() {
 
   // Monitor straight key pin
   if (key.update() || qt_key.update()) {
-    bool pressed = !key.read() || qt_key.read();
+    bool pressed = key.read() || qt_key.read();
     adapter.HandlePaddle(PADDLE_STRAIGHT, pressed);
   }
 
@@ -110,12 +107,12 @@ void loop() {
   }
 
   if (dit.update() || qt_dit.update()) {
-    bool pressed = !dit.read() || qt_dit.read();
+    bool pressed = dit.read() || qt_dit.read();
     adapter.HandlePaddle(PADDLE_DIT, pressed);
   }
   
   if (dah.update() || qt_dah.update()) {
-    bool pressed = !dah.read() || qt_dah.read();
+    bool pressed = dah.read() | qt_dah.read();
     adapter.HandlePaddle(PADDLE_DAH, pressed);
   }
 }