Reverted to old code then changed to wh_test_print for cleaner hexdump output

Author: jackctj117

test/wh_test_nvm_flash.c

@@ -55,44 +55,28 @@ static void _HexDump(const char* p, size_t data_len)
     const size_t         bytesPerLine = 16;
     const unsigned char  two_digits   = 0x10;
     const unsigned char* u            = (const unsigned char*)p;
-    char                 line[80];  /* Buffer for a line of hex output */
-    size_t               linePos     = 0;
-    size_t               off         = 0;
-    const char           hex[]       = "0123456789ABCDEF";
-    
+
+    /* Header line - use WH_TEST_DEBUG_PRINT for context */
     WH_TEST_DEBUG_PRINT("    HD:%p for %lu bytes\n", p, data_len);
     if ((p == NULL) || (data_len == 0))
         return;
-    
+
+    size_t off = 0;
     for (off = 0; off < data_len; off++) {
-        if ((off % bytesPerLine) == 0) {
-            linePos = 0;
-            line[linePos++] = ' ';
-            line[linePos++] = ' ';
-            line[linePos++] = ' ';
-            line[linePos++] = ' ';
-        }
+        /* Use WH_TEST_PRINT for hex data - no prefix */
+        if ((off % bytesPerLine) == 0)
+            WH_TEST_PRINT("    ");
         if (u[off] < two_digits) {
-            line[linePos++] = '0';
-            line[linePos++] = hex[u[off]];
-            line[linePos++] = ' ';
+            WH_TEST_PRINT("0%X ", u[off]);
         }
         else {
-            line[linePos++] = hex[u[off] >> 4];
-            line[linePos++] = hex[u[off] & 0x0F];
-            line[linePos++] = ' ';
+            WH_TEST_PRINT("%X ", u[off]);
         }
-        if ((off % bytesPerLine) == (bytesPerLine - 1)) {
-            line[linePos++] = '\n';
-            line[linePos] = '\0';
-            WOLFHSM_CFG_PRINTF("%s", line);
-        }
-    }
-    if ((off % bytesPerLine) != 0) {
-        line[linePos++] = '\n';
-        line[linePos] = '\0';
-        WOLFHSM_CFG_PRINTF("%s", line);
+        if ((off % bytesPerLine) == (bytesPerLine - 1))
+            WH_TEST_PRINT("\n");
     }
+    if ((off % bytesPerLine) != 0)
+        WH_TEST_PRINT("\n");
 }
 
 static void _ShowAvailable(const whNvmCb* cb, void* context)