Cppcheck report - [project name]: NaoTHSoccer/Source/Tools/Xabsl/XabslFileInputSource.h

/**
* @file XabslFileInputSource.h
*
* Declaration of class XabslFileInputSource:
* reads a xabsl behavior from a file
*/

#ifndef _XabslFileInputSource_H
#define  _XabslFileInputSource_H

#include <XabslEngine/XabslTools.h>
#include <fstream>

class XabslFileInputSource : public xabsl::InputSource
{
public:
  XabslFileInputSource(std::string file);
<--- Class 'XabslFileInputSource' has a constructor with 1 argument that is not explicit. [+]
Class 'XabslFileInputSource' has a constructor with 1 argument that is not explicit. Such constructors should in general be explicit for type safety reasons. Using the explicit keyword in the constructor means some mistakes when using the class can be avoided.

  bool open();
  void close();
  double readValue();
  bool readString(char* destination, int maxLength);
  
  void skipComments();
  void skipWhiteSpace();

  virtual ~XabslFileInputSource();

private:
  std::string file;
  std::ifstream inFile;
};

#endif  /* _XabslFileInputSource_H */