AerospaceNU / AerospaceNU/better-engine-control-software

Can we make `ISequence`s const?

オープン
#25 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
question
主要言語
C++
スター
5
フォーク
2
PR マージ指標
30日以内にマージされた PR はありません

説明

https://github.com/AerospaceNU/better-engine-control-software/blob/main/src/constants/AllSequences.h

We define our sequences as global variables. Ideally I want some const protection on them so they don't get randomly mutated by some function.

However, this requires us to make `ISequence` methods all const.

I want

- [ ] A good argument as for why we should commit to `ISequence` being const. `ISequence` is an interface, all future deriving classes must now act const as well, should we commit to this design idea?
- [ ] Impl the const

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。