Improvements Made

test/test.cpp

@@ -1,52 +1,52 @@
 #include <cstdio>
-
 #include "test.h"
 
+// Forward declarations for test suites
 CuSuite* TestP2GetSuite();
 CuSuite* TestP3GetSuite();
 CuSuite* TestIeeeGetSuite();
 
-void genSubTable(POSIT_STYPE *sub_table, POSIT_STYPE *add_table, int size)
-{
-    for (int i = 0; i < size; i++) {
-        int k;
+// Function to generate subtraction table from addition table
+void genSubTable(POSIT_STYPE *sub_table, const POSIT_STYPE *add_table, int size) {
+    for (int i = 0; i < size; ++i) {
+        int adjustedIndex;
 
         if (i == 0) {
-            k = 0;
+            adjustedIndex = 0;
         } else {
-            k = size / 2 - (i - size / 2);
+            adjustedIndex = size / 2 - (i - size / 2);
         }
 
-        for (int j = 0; j < size; j++) {
-            sub_table[k * size + j] = add_table[i * size + j];
+        for (int j = 0; j < size; ++j) {
+            sub_table[adjustedIndex * size + j] = add_table[i * size + j];
         }
     }
 }
 
-void genDivTable(POSIT_STYPE *div_table, POSIT_STYPE *mul_table, int size)
-{
-    for (int i = 0; i < size; i++) {
-        int k;
+// Function to generate division table from multiplication table
+void genDivTable(POSIT_STYPE *div_table, const POSIT_STYPE *mul_table, int size) {
+    for (int i = 0; i < size; ++i) {
+        int adjustedIndex;
 
         if (i <= size / 2) {
-            k = size / 2 - i;
+            adjustedIndex = size / 2 - i;
         } else {
-            k = size - (i - size / 2);
+            adjustedIndex = size - (i - size / 2);
         }
 
-        for (int j = 0; j < size; j++) {
-            if (k == size / 2) {
-                // TODO explain
-                div_table[k * size + j] = mul_table[size / 2 * size + j];
+        for (int j = 0; j < size; ++j) {
+            // Special case handling for the central index
+            if (adjustedIndex == size / 2) {
+                div_table[adjustedIndex * size + j] = mul_table[(size / 2) * size + j];
             } else {
-                div_table[k * size + j] = mul_table[i * size + j];
+                div_table[adjustedIndex * size + j] = mul_table[i * size + j];
             }
         }
     }
 }
 
-void RunAllTests(void)
-{
+// Function to run all test suites and print the results
+void RunAllTests() {
     CuString *output = CuStringNew();
     CuSuite *suite = CuSuiteNew();
 
@@ -58,9 +58,14 @@ void RunAllTests(void)
     CuSuiteSummary(suite, output);
     CuSuiteDetails(suite, output);
     printf("%s", output->buffer);
+
+    // Clean up
+    CuStringDelete(output);
+    CuSuiteDelete(suite);
 }
 
-int main(int argc, char *argv[])
-{
+// Main function to start the test execution
+int main(int argc, char *argv[]) {
     RunAllTests();
+    return 0;
 }