AerospaceNU / AerospaceNU/better-engine-control-software

Can we make `ISequence`s const?

Abierto
#25 0 comentarios 0 reacciones 0 asignados Ver en GitHub
question
Lenguaje dominante
C++
Estrellas
5
Forks
2
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

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

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Evaluación

Este issue todavía no se ha evaluado.

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.