When a negative parameter is fed for the number of digits then default to the default number of parameters

aentinger · aentinger · commit 4bcdd1300d27 · 2020-11-02T14:48:25.000+01:00
diff --git a/api/Print.cpp b/api/Print.cpp
@@ -332,8 +332,11 @@ size_t Print::printULLNumber(unsigned long long n64, uint8_t base)
   return bytes;
 }
 
-size_t Print::printFloat(double number, uint8_t digits)
+size_t Print::printFloat(double number, int digits)
 {
+  if (digits < 0)
+    digits = 2;
+
   size_t n = 0;
 
   if (isnan(number)) return print("nan");
diff --git a/api/Print.h b/api/Print.h
@@ -37,7 +37,7 @@ class Print
     int write_error;
     size_t printNumber(unsigned long, uint8_t);
     size_t printULLNumber(unsigned long long, uint8_t);
-    size_t printFloat(double, uint8_t);
+    size_t printFloat(double, int);
   protected:
     void setWriteError(int err = 1) { write_error = err; }
   public:
diff --git a/test/src/Print/test_print.cpp b/test/src/Print/test_print.cpp
@@ -111,6 +111,12 @@ TEST_CASE ("Print::print(double, int = 2)", "[Print-print-07]")
     WHEN("digits = 5")           { mock.print(val, 5); REQUIRE(mock._str  == "3.14590"); }
   }
 
+  WHEN ("digits are negative")
+  {
+    double const val = 3.1459;
+    WHEN("digits = -1") { mock.print(val, -1); REQUIRE(mock._str  == "3.15"); }
+  }
+
   WHEN ("val is a negative floating point value")
   {
     double const val = -3.1459;

Original file line number	Diff line number	Diff line change
`@@ -332,8 +332,11 @@ size_t Print::printULLNumber(unsigned long long n64, uint8_t base)`
`332`	`332`	`return bytes;`
`333`	`333`	`}`
`334`	`334`
`335`		`-size_t Print::printFloat(double number, uint8_t digits)`
	`335`	`+size_t Print::printFloat(double number, int digits)`
`336`	`336`	`{`
	`337`	`+ if (digits < 0)`
	`338`	`+ digits = 2;`
	`339`	`+`
`337`	`340`	`size_t n = 0;`
`338`	`341`
`339`	`342`	`if (isnan(number)) return print("nan");`
Original file line number	Diff line number	Diff line change
`@@ -111,6 +111,12 @@ TEST_CASE ("Print::print(double, int = 2)", "[Print-print-07]")`
`111`	`111`	`WHEN("digits = 5") { mock.print(val, 5); REQUIRE(mock._str == "3.14590"); }`
`112`	`112`	`}`
`113`	`113`
	`114`	`+ WHEN ("digits are negative")`
	`115`	`+ {`
	`116`	`+ double const val = 3.1459;`
	`117`	`+ WHEN("digits = -1") { mock.print(val, -1); REQUIRE(mock._str == "3.15"); }`
	`118`	`+ }`
	`119`	`+`
`114`	`120`	`WHEN ("val is a negative floating point value")`
`115`	`121`	`{`
`116`	`122`	`double const val = -3.1459;`