m5stack / M5Unified

Unified library for M5Stack series
MIT License
302 stars 54 forks source link

Add settings for serial1 and serial2 #75

Closed hashy0917 closed 1 year ago

hashy0917 commented 1 year ago

serial1_baudrateとserial2_baudrateの設定を使用できるように変更しました

lovyan03 commented 1 year ago

プルリクエストありがとうございます! すみません、利点がよくわからないので質問なのですが、 ボーレート設定のみでbeginする機能をM5Unifiedに頼って使いたい場面はありますか?

cfg.serial1_baudrate を書くのと、普通に Serial1.begin を書くのに労力の差はさほどないと思いますし、 Serial1.beginの第二引数以降のパラメータ指定ができないではあまり便利にも感じないのですが、、

つまり何というか、Unifiedに頼るよりも、ユーザーコード側に普通に各自で Serial1.begin使った方が便利ではないですか?

なお Serial.begin をUnifiedでサポートしている理由は、 M5Stackの機種別ライブラリがそれをサポートしていることと、 標準のSerialに関しては出力先が決まっていてボーレート設定のみで使用するケースが殆どだからです。

hashy0917 commented 1 year ago

cfg.serial1_baudrate を書くのと、普通に Serial1.begin を書くのに労力の差はさほどないと思いますし、 Serial1.beginの第二引数以降のパラメータ指定ができないではあまり便利にも感じないのですが、、 つまり何というか、Unifiedに頼るよりも、ユーザーコード側に普通に各自で Serial1.begin使った方が便利ではないですか?

全くその通りですね・・・

特にこれと言って利点という物はありませんが、cfg.serial_baudrate の後にSerial1.beginがあると統一感が無くて微妙な気持ちになったのでとりあえず変更しただけとなります・・・

lovyan03 commented 1 year ago

なるほど~、統一感という面では確かにアレではありますが… 機種別の相違があるわけではない箇所なので、これを手厚くサポートする理由がUnifiedにないこと、 マージすると今後 「2番目以降の引数の対応が無いの何とかしろ」的な要求が発生しそうなことから、 今回はマージを見送りとさせて頂きたいと思います。。。

hashy0917 commented 1 year ago

はい~ 承知しました。

ご確認ありがとうございました~ 🙇