fix build-CI

README.md

@@ -67,11 +67,6 @@ Serial.print(fr.toString());  //  prints "(355/113)"
 ```
 
 
-#### Related
-
-
-
-
 ## Interface
 
 ```cpp

examples/fraction_extensive/fraction_extensive.ino

@@ -6,8 +6,10 @@
 
 
 //  step size to test, typical 100000
-//  AVR UNO takes a bit less than 500 seconds 
-//  to process 100.000 tests
+//  for 100.000 tests
+//  - AVR UNO 16 MHz takes ~500 seconds at 115200 baud
+//  - ESP32 240 MHz takes ~400 seconds at 115200 baud
+//  - ESP32 240 MHz takes ~85 seconds at 500000 baud
 
 const uint32_t N = 100000;
 
@@ -20,16 +22,21 @@ uint32_t pos = 0;
 
 void setup()
 {
-  Serial.begin(115200);
+  //  NOTE BAUDRATE!
+  Serial.begin(500000);
   Serial.print(__FILE__);
   Serial.print("FRACTION_LIB_VERSION: ");
   Serial.println(FRACTION_LIB_VERSION);
   Serial.println();
   delay(100);
 
+  Fraction pi(PI);
+  Serial.println(pi.toString());
+  delay(1000);
+
   start = millis();
 
-  for (uint32_t n = 10; n <= N; n++)
+  for (uint32_t n = 100; n <= N; n++)
   {
     float g = n * (1.0 / N);
     Fraction frac( g );

fraction.cpp

@@ -357,7 +357,7 @@ void Fraction::simplify()
   //  in preventing overflow
   while (q > 10000)
   {
-    //  rounding might need improvement
+    //  rounding need improvement
     p = (p + 5)/10;
     q = (q + 5)/10;
     x = gcd(p, q);
@@ -412,7 +412,7 @@ void Fraction::fractionize(float val)
       int32_t count = (int32_t)test;    //  "N"
       int32_t n = (count + 1) * low.n + high.n;
       int32_t d = (count + 1) * low.d + high.d;
-      if ((n > 0x10000) || (d > 0x10000))
+      if ((n > 0x8000) || (d > 0x10000))   //   0x8000 0x10000
       break;
       high.n = n - low.n;  //  new "A"
       high.d = d - low.d;
@@ -425,7 +425,7 @@ void Fraction::fractionize(float val)
       int32_t count = (int32_t)test;     //  "N"
       int32_t n = low.n + (count + 1) * high.n;
       int32_t d = low.d + (count + 1) * high.d;
-      if ((n > 0x10000) || (d > 0x10000))
+      if ((n > 0x10000) || (d > 0x10000))   //   0x10000 0x10000
       break;
       low.n = n - high.n;  //  new "A"
       low.d = d - high.d;

test/unit_test_001.cpp

@@ -44,7 +44,7 @@ unittest_teardown()
 unittest(test_constructor)
 {
   Fraction pi(PI);
-  assertEqualFloat(PI, pi.toFloat(), 0.0001)
+  assertEqualFloat(PI, pi.toFloat(), 0.0001);
   //  what we wished it would find.
   //  assertEqual(355, pi.nominator());
   //  assertEqual(113, pi.denominator());
@@ -54,7 +54,7 @@ unittest(test_constructor)
   fprintf(stderr, "pi %1.8f\n", pi.toFloat());
 
   Fraction ee(EULER);
-  assertEqualFloat(EULER, ee.toFloat(), 0.0001)
+  assertEqualFloat(EULER, ee.toFloat(), 0.0001);
   //  what we wished it would find.
   //  assertEqual(3985, ee.nominator());
   //  assertEqual(1466, ee.denominator());