From c5013bf98086ad9ed938248f4bb9669ca221a43f Mon Sep 17 00:00:00 2001
From: Andy Maloney <asmaloney@gmail.com>
Date: Thu, 11 Jan 2024 10:32:16 -0500
Subject: [PATCH] Remove default arguments on implementation class constructors

These defaults are never used since they are always constructed with all arguments.
---
 src/FloatNodeImpl.h         |  5 ++---
 src/IntegerNodeImpl.h       |  4 ++--
 src/ScaledIntegerNodeImpl.h | 12 ++++++------
 src/SourceDestBufferImpl.h  |  2 +-
 src/StringNodeImpl.h        |  2 +-
 5 files changed, 12 insertions(+), 13 deletions(-)

diff --git a/src/FloatNodeImpl.h b/src/FloatNodeImpl.h
index a479ecd..b7ff10c 100644
--- a/src/FloatNodeImpl.h
+++ b/src/FloatNodeImpl.h
@@ -33,9 +33,8 @@ namespace e57
    class FloatNodeImpl : public NodeImpl
    {
    public:
-      explicit FloatNodeImpl( ImageFileImplWeakPtr destImageFile, double value = 0,
-                              bool validValue = true, FloatPrecision precision = PrecisionDouble,
-                              double minimum = DOUBLE_MIN, double maximum = DOUBLE_MAX );
+      explicit FloatNodeImpl( ImageFileImplWeakPtr destImageFile, double value, bool validValue,
+                              FloatPrecision precision, double minimum, double maximum );
       ~FloatNodeImpl() override = default;
 
       NodeType type() const override
diff --git a/src/IntegerNodeImpl.h b/src/IntegerNodeImpl.h
index faa42e3..0f7d4ea 100644
--- a/src/IntegerNodeImpl.h
+++ b/src/IntegerNodeImpl.h
@@ -33,8 +33,8 @@ namespace e57
    class IntegerNodeImpl : public NodeImpl
    {
    public:
-      explicit IntegerNodeImpl( ImageFileImplWeakPtr destImageFile, int64_t value = 0,
-                                int64_t minimum = 0, int64_t maximum = 0 );
+      explicit IntegerNodeImpl( ImageFileImplWeakPtr destImageFile, int64_t value, int64_t minimum,
+                                int64_t maximum );
       ~IntegerNodeImpl() override = default;
 
       NodeType type() const override
diff --git a/src/ScaledIntegerNodeImpl.h b/src/ScaledIntegerNodeImpl.h
index b6b5e16..6d1c5ad 100644
--- a/src/ScaledIntegerNodeImpl.h
+++ b/src/ScaledIntegerNodeImpl.h
@@ -33,13 +33,13 @@ namespace e57
    class ScaledIntegerNodeImpl : public NodeImpl
    {
    public:
-      explicit ScaledIntegerNodeImpl( ImageFileImplWeakPtr destImageFile, int64_t value = 0,
-                                      int64_t minimum = 0, int64_t maximum = 0, double scale = 1.0,
-                                      double offset = 0.0 );
+      explicit ScaledIntegerNodeImpl( ImageFileImplWeakPtr destImageFile, int64_t value,
+                                      int64_t minimum, int64_t maximum, double scale,
+                                      double offset );
 
-      explicit ScaledIntegerNodeImpl( ImageFileImplWeakPtr destImageFile, double scaledValue = 0.,
-                                      double scaledMinimum = 0., double scaledMaximum = 0.,
-                                      double scale = 1.0, double offset = 0.0 );
+      explicit ScaledIntegerNodeImpl( ImageFileImplWeakPtr destImageFile, double scaledValue,
+                                      double scaledMinimum, double scaledMaximum, double scale,
+                                      double offset );
 
       ~ScaledIntegerNodeImpl() override = default;
 
diff --git a/src/SourceDestBufferImpl.h b/src/SourceDestBufferImpl.h
index f981a9a..55fdd74 100644
--- a/src/SourceDestBufferImpl.h
+++ b/src/SourceDestBufferImpl.h
@@ -37,7 +37,7 @@ namespace e57
    {
    public:
       SourceDestBufferImpl( ImageFileImplWeakPtr destImageFile, const ustring &pathName,
-                            size_t capacity, bool doConversion = false, bool doScaling = false );
+                            size_t capacity, bool doConversion, bool doScaling );
 
       template <typename T> void setTypeInfo( T *base, size_t stride = sizeof( T ) );
 
diff --git a/src/StringNodeImpl.h b/src/StringNodeImpl.h
index f6d1c51..a7cbca4 100644
--- a/src/StringNodeImpl.h
+++ b/src/StringNodeImpl.h
@@ -33,7 +33,7 @@ namespace e57
    class StringNodeImpl : public NodeImpl
    {
    public:
-      explicit StringNodeImpl( ImageFileImplWeakPtr destImageFile, const ustring &value = "" );
+      explicit StringNodeImpl( ImageFileImplWeakPtr destImageFile, const ustring &value );
       ~StringNodeImpl() override = default;
 
       NodeType type() const override