Fixed classes with undefined constructor.

IterativeRobot and BinaryImage had no constructor
defined (and so would give linker errors).
Error just had an empty constructor defined,
so I switched to "=default".

Change-Id: Ia8efb4282928227878dfefeda58ccb8cf06aabb2

wpilibc/wpilibC++/include/Error.h

@@ -20,7 +20,7 @@ class Error {
  public:
   typedef int32_t Code;
 
-  Error();
+  Error() = default;
   void Clone(const Error& error);
   Code GetCode() const;
   std::string GetMessage() const;

wpilibc/wpilibC++/include/ErrorBase.h

@@ -52,7 +52,7 @@
 class ErrorBase {
   // TODO: Consider initializing instance variables and cleanup in destructor
  public:
-  ErrorBase();
+  ErrorBase() = default;
   virtual ~ErrorBase() = default;
   virtual Error& GetError();
   virtual const Error& GetError() const;

wpilibc/wpilibC++/src/Error.cpp

@@ -17,8 +17,6 @@
 #include "Timer.h"
 #include "Utility.h"
 
-Error::Error() {}
-
 void Error::Clone(const Error& error) {
   m_code = error.m_code;
   m_message = error.m_message;

wpilibc/wpilibC++/src/ErrorBase.cpp

@@ -16,10 +16,6 @@
 
 priority_mutex ErrorBase::_globalErrorMutex;
 Error ErrorBase::_globalError;
-/**
- * @brief Initialize the instance status to 0 for now.
- */
-ErrorBase::ErrorBase() {}
 
 /**
  * @brief Retrieve the current error.

wpilibc/wpilibC++Devices/include/IterativeRobot.h

@@ -73,7 +73,7 @@ class IterativeRobot : public RobotBase {
   virtual void Prestart();
 
   virtual ~IterativeRobot() = default;
-  IterativeRobot();
+  IterativeRobot() = default;
 
  private:
   bool m_disabledInitialized = false;

wpilibc/wpilibC++Devices/include/Vision/BinaryImage.h

@@ -18,7 +18,6 @@
 
 class BinaryImage : public MonoImage {
  public:
-  BinaryImage();
   virtual ~BinaryImage() = default;
   int GetNumberParticles();
   ParticleAnalysisReport GetParticleAnalysisReport(int particleNumber);