static analysis clean up

Signed-off-by: Nathan Young <[email protected]>

avogadro/core/basisset.h

@@ -7,7 +7,9 @@
 #define AVOGADRO_CORE_BASISSET_H
 
 #include "avogadrocoreexport.h"
+#include "core/variant.h"
 
+#include <array>
 #include <string>
 
 namespace Avogadro::Core {
@@ -138,14 +140,14 @@ class AVOGADROCORE_EXPORT BasisSet
    * For closed shell calculations alpha is doubly occupied and there are no
    * beta electrons.
    */
-  unsigned int m_electrons[2];
+  std::array<unsigned int, 2> m_electrons = {};
 
   /**
    * The Molecule holds the atoms (and possibly bonds) read in from the output
    * file. Most basis sets have orbitals around these atoms, but this is not
    * necessarily the case.
    */
-  Molecule* m_molecule;
+  Molecule* m_molecule = nullptr;
 
   /**
    * The name of the basis set, this is usually a string identifier referencing

avogadro/core/residue.h

@@ -10,6 +10,7 @@
 
 #include "avogadrocore.h"
 
+#include <limits>
 #include <map>
 #include <string>
 
@@ -133,7 +134,7 @@ class AVOGADROCORE_EXPORT Residue
 
 protected:
   std::string m_residueName;
-  Index m_residueId;
+  Index m_residueId = std::numeric_limits<Index>::max();
   char m_chainId;
   AtomNameMap m_atomNameMap;
   bool m_heterogen;