Robin Leroy

Or at least fixed descriptions, the current ones don't fit.

States and transitions in the plugin

1

We could use something more refined than the boolean initializing, and this would clarify the sequence of expected calls at every frame.

Classes should be final or have a virtual destructor

4

For sanity. This probably means that structs should be final.

Allow passing arguments by reference

2

The current style conflicts with our avoidance of pointers where the address is unneeded (and the wole "& is visible" argument is fallacious since many things are pointers already, and...

It is of no concern to the caller whether a non-reference parameter is const or not. This matters in the function definition (it has to be non-`const` when, e.g., moving,...

`constexpr` is a declaration specifier, but not a `cv` qualifier, thus may appear in the `decl-specifier-seq`, but not in the `declarator`; in other words, for the purposes of variable declarations,...

Use internal namespaces in header files, not unnamed namespaces.

1

Many instances.

Suggested by @ferram4, this sounds sensible, and would be a good reminder of which frame is in use.