Explicitly initialize all member variables in constructors #196
Labels
good first issue
This issue is easy to resolve for a novice
misimplementation
Circumlocutious or redundant implementation
When constructing new objects, it is best practice to initialize all variables, such that they are assured not to be undefined later when they are to be used. While
g++
does not specifically raise a complaint for this (it's not always undefined behavior if used carefully),msvc
does, and so does Cppcheck.Resolving this issue would mean setting all object constructors to set all member variables to some value (0 or
nullptr
is fine). There are multiple transgressions of this best practice in many files, so a significant number of commits (and potentially multiple pull requests) are expected to resolve this.The text was updated successfully, but these errors were encountered: