USARTのグローバル変数化やめたい

[Enchan1207]

これのせいでwarning出る

ターゲットMCUによってレジスタ名の定義が違ってたりするので、Arduinoにならって 必要なレジスタをコンストラクタで渡せるようにしたいなという

ただ流石に HardwareSerial[0-3] みたいなネーミングは避けたい

[Enchan1207]

avr/io.hに直接依存するのは event.cppとconfigure.cppで

• UDR0: データレジスタ
• UCSR0A: 制御レジスタA
• UCSR0B: 制御レジスタB
• UBRR0: ボーレートレジスタ

の4つのレジスタと

• TXEN0: 送信許可
• RXEN0: 受信許可
• UDRIE0: 送信割り込み許可
• RXCIE0: 受信割り込み許可

の4つの識別子が必要.

[Enchan1207]

データレジスタ(UDR)だけは共通化できそう(UDR, UDR[0-3])だから、event.cpp内のデータ取得処理だけメンバ呼ぶようにして、あとはvirtualの方がいいかもしれない

[Enchan1207]

とりあえずUDRだけインジェクションされた方を参照するように変更。これで avr/io.h に依存するのは configure.cpp だけになったわけですが…

[Enchan1207]

あとはグローバル変数をどうやめるかなんですよね… Singletonにするにしても、そもそも複数のUSARTインタフェースを持たれると話が終わってしまう

というか多分tiny85とかUSARTI/Fを持たないMCUをターゲットにするとまともに動作しないんじゃないだろうか?

[Enchan1207]

USARTを抽象基底クラスにしてしまって、対応するレジスタが存在する場合のみUSART0, 1, 2, 3を実体化させるとかどうでしょうか

[Enchan1207]

AVR-GCCで仮想関数を使うのに必要な事前準備が多すぎる

[Enchan1207]

USART[0-3] を BaseUSARTを継承したシングルトンクラスとして定義, Serial および Serial[0-3] をエイリアスとして設定

[Enchan1207]

これによりこんな感じの書き方ができるようになりました:

int main(){
    Serial.begin(115200);

    Serial.println((unsigned char)22);
    Serial.println(123);
    Serial.println((unsigned int)456);
    Serial.println(3.14);
    Serial.println(123456L);
    Serial.println(1234567890UL);
}

これでArduinoのSerialを使ったコードを(ほぼ)そのまま流用できそう.

[Enchan1207]

PROGMEM文字列の対応をやってないですが、それはこのissueでやることではないので一旦閉じますか