cppcheck

src/OpenMesh/Apps/Decimating/DecimaterViewerWidget.hh

@@ -144,7 +144,7 @@ public:
 
 public: // inherited
 
-  bool open_mesh(const char* _filename, OpenMesh::IO::Options _opt)
+  bool open_mesh(const char* _filename, OpenMesh::IO::Options _opt) override
   {
     bool rc;
 
@@ -170,7 +170,7 @@ protected slots:
 
 protected:   
 
-  virtual void keyPressEvent(QKeyEvent* _event);  
+  virtual void keyPressEvent(QKeyEvent* _event) override;
 
 
 private:

src/OpenMesh/Apps/ProgViewer/ProgViewerWidget.hh

@@ -94,7 +94,7 @@ public:
 
 public:   
   /// default constructor
-  ProgViewerWidget(QWidget* _parent=0)
+  explicit ProgViewerWidget(QWidget* _parent=0)
     : MeshViewerWidget(_parent),
       n_base_vertices_(0),
       n_base_faces_(0),

src/OpenMesh/Apps/QtViewer/MeshViewerWidgetT.hh

@@ -117,7 +117,7 @@ public:
 protected:
   
   /// inherited drawing method
-  virtual void draw_scene(const std::string& _draw_mode);
+  virtual void draw_scene(const std::string& _draw_mode) override;
   
 protected:
   
@@ -171,7 +171,7 @@ protected: // Strip support
 
 protected: // inherited
    
-  virtual void keyPressEvent( QKeyEvent* _event);
+  virtual void keyPressEvent( QKeyEvent* _event) override;
 
 protected:
    

src/OpenMesh/Core/IO/reader/OBJReader.hh

@@ -105,7 +105,7 @@ private:
   {
   public:
 
-    Material() { cleanup(); }
+    Material():Tr_(0),index_Kd_(0) { cleanup(); }
 
     void cleanup()
     {
@@ -137,7 +137,7 @@ private:
     void set_Tr( float t )
     { Tr_=t;            Tr_is_set_=true; }
 
-    void set_map_Kd( std::string _name, int _index_Kd )
+    void set_map_Kd( const std::string& _name, int _index_Kd )
     { map_Kd_ = _name, index_Kd_ = _index_Kd; map_Kd_is_set_ = true; };
 
     const Vec3f& Kd( void ) const { return Kd_; }

src/OpenMesh/Tools/Smoother/JacobiLaplaceSmootherT.hh

@@ -79,7 +79,7 @@ private:
   
 public:
 
-  JacobiLaplaceSmootherT( Mesh& _mesh ) : LaplaceSmootherT<Mesh>(_mesh) {}
+  explicit JacobiLaplaceSmootherT( Mesh& _mesh ) : LaplaceSmootherT<Mesh>(_mesh) {}
 
   // override: alloc umbrellas
   void smooth(unsigned int _n);

src/OpenMesh/Tools/Smoother/LaplaceSmootherT.hh

@@ -83,7 +83,7 @@ public:
   typedef typename SmootherT<Mesh>::EdgeHandle    EdgeHandle;
   
 
-  LaplaceSmootherT( Mesh& _mesh );
+  explicit LaplaceSmootherT( Mesh& _mesh );
   virtual ~LaplaceSmootherT();
 
 

src/OpenMesh/Tools/Subdivider/Adaptive/Composite/CompositeT.hh

@@ -146,7 +146,7 @@ public:
 public:
 
   /// Constructor
-  CompositeT(Mesh& _mesh) 
+  explicit CompositeT(Mesh& _mesh)
     : subdiv_type_(0), 
       subdiv_rule_(NULL), /*first_rule_(NULL), last_rule_(NULL),*/ mesh_(_mesh)
   { }

src/OpenMesh/Tools/Subdivider/Adaptive/Composite/RuleInterfaceT.hh

@@ -118,7 +118,7 @@ public:
 protected:
 
   /// Default constructor
-  RuleInterfaceT(Mesh& _mesh) : mesh_(_mesh) {};
+  RuleInterfaceT(Mesh& _mesh) : mesh_(_mesh),prev_rule_(nullptr),subdiv_rule_(nullptr),subdiv_type_(0),number_(0),n_rules_(0) {};
 
 public: