Starlight220
Starlight220
So I see that the read-only SimValues take effect only for the SimGUI? In code, they can be written from everywhere. Is this the desired behavior?
If so, it should be documented. The SimDevice field keys don't need the direction prefix, right?
Do we want to make this change before beta?
I agree with what you said about the Command classes being designed for specific use cases. However, inheriting is the "traditional" way of creating commands, as evidenced by the `HatchbotTraditional`...
Due to Java's lack of free functions, inheriting commands like @28andrew's example is almost the only way of creating built-in commands with certain commands in a repeatable and self-documenting way....
A slight problem: the Gearsbot example overrides commands in a way that this would break, and I'm not sure there is a way to have the same functionality in the...
#4385 handled the four classic command group classes; do we want to mark anything else as `final`?
The main risky overrides were the command groups; that's solved now.
Needs to be rebased for correct formatting (old base is set to clang-format 12, we use 14).
`finally` would be an ideal naming, the problem is that afaik it's a reserved keyword. I don't like `onEnd`, I think it's too vague. `onFinish` is redundant; `andThen` accomplishes exactly...