From 3aaf09031e9c5a517e9dadaad20387f4412d817d Mon Sep 17 00:00:00 2001
From: Avril <flanchan@cumallover.me>
Date: Fri, 13 Nov 2020 22:45:49 +0000
Subject: [PATCH] removed some use of `fmt`

~50% improved performance
---
 src/hex.cpp | 84 +++++++++++++++++++++++++++++++++++++----------------
 1 file changed, 59 insertions(+), 25 deletions(-)

diff --git a/src/hex.cpp b/src/hex.cpp
index 7cdded2..e32f543 100644
--- a/src/hex.cpp
+++ b/src/hex.cpp
@@ -20,6 +20,25 @@ const static constexpr char ascii_map[256] = {
   '.', '.', '.', '.', '.', '.', '.', '.', '.', '.', '.', '.', '.', '.', '.', '.', 
 };
 
+constexpr const char* hex_map[256] = {
+	"00", "01", "02", "03", "04", "05", "06", "07", "08", "09", "0a", "0b", "0c", "0d", "0e", "0f",
+	"10", "11", "12", "13", "14", "15", "16", "17", "18", "19", "1a", "1b", "1c", "1d", "1e", "1f",
+	"20", "21", "22", "23", "24", "25", "26", "27", "28", "29", "2a", "2b", "2c", "2d", "2e", "2f",
+	"30", "31", "32", "33", "34", "35", "36", "37", "38", "39", "3a", "3b", "3c", "3d", "3e", "3f",
+	"40", "41", "42", "43", "44", "45", "46", "47", "48", "49", "4a", "4b", "4c", "4d", "4e", "4f",
+	"50", "51", "52", "53", "54", "55", "56", "57", "58", "59", "5a", "5b", "5c", "5d", "5e", "5f",
+	"60", "61", "62", "63", "64", "65", "66", "67", "68", "69", "6a", "6b", "6c", "6d", "6e", "6f",
+	"70", "71", "72", "73", "74", "75", "76", "77", "78", "79", "7a", "7b", "7c", "7d", "7e", "7f",
+	"80", "81", "82", "83", "84", "85", "86", "87", "88", "89", "8a", "8b", "8c", "8d", "8e", "8f",
+	"90", "91", "92", "93", "94", "95", "96", "97", "98", "99", "9a", "9b", "9c", "9d", "9e", "9f",
+	"a0", "a1", "a2", "a3", "a4", "a5", "a6", "a7", "a8", "a9", "aa", "ab", "ac", "ad", "ae", "af",
+	"b0", "b1", "b2", "b3", "b4", "b5", "b6", "b7", "b8", "b9", "ba", "bb", "bc", "bd", "be", "bf",
+	"c0", "c1", "c2", "c3", "c4", "c5", "c6", "c7", "c8", "c9", "ca", "cb", "cc", "cd", "ce", "cf",
+	"d0", "d1", "d2", "d3", "d4", "d5", "d6", "d7", "d8", "d9", "da", "db", "dc", "dd", "de", "df",
+	"e0", "e1", "e2", "e3", "e4", "e5", "e6", "e7", "e8", "e9", "ea", "eb", "ec", "ed", "ee", "ef",
+	"f0", "f1", "f2", "f3", "f4", "f5", "f6", "f7", "f8", "f9", "fa", "fb", "fc", "fd", "fe", "ff",
+};
+
 #ifndef FIXED_ROW_SIZE
 #include <alloca.h>
 #endif
@@ -28,35 +47,50 @@ const static constexpr char ascii_map[256] = {
 constexpr const int ROW_SZ = FIXED_ROW_SIZE;
 #endif
 
+template<typename... S>
+inline __attribute__((always_inline)) void prints(const S&... strings)
+{
+	(fputs((const char*)strings, stdout), ...);
+}
+
+inline void hex02(std::uint8_t byte, char buffer[3])
+{
+	buffer[0] =hex_map[byte][0];
+	buffer[1] =hex_map[byte][1];
+}
+
 namespace hv {
-  void print_screen(const span<unsigned char> memory, unsigned long offset)
-  {
+	void print_screen(const span<unsigned char> memory, unsigned long offset)
+	{
 #ifndef FIXED_ROW_SIZE
 #define S (const char*)
-    int ROW_SZ = 24;
+		int ROW_SZ = 24;
 #else
 #define S
 #endif
-    fmt::print("0x{:016x}  ", offset);
-    char ascii[1 + ROW_SZ];
-    ascii[ROW_SZ] = 0;
-    std::size_t i=0;
-    for(;i<memory.size();i++) {
-      if (i && i % ROW_SZ == 0) {
-	fmt::print("  {}", S ascii);
-	fmt::print("\n0x{:016x}  ", i+offset);
-      }
-      fmt::print("{:02x} ", memory[i]);
-      ascii[i%ROW_SZ] = ascii_map[memory[i]];
-    }
-    if (memory.size() % ROW_SZ != 0)
-      {
-	auto rest = memory.size() % ROW_SZ;
-	ascii[rest] = 0;
-	for(std::size_t j=0;j< ROW_SZ - rest;j++)
-	  fmt::print("   ");
-	  
-      }
-    fmt::print("  {}", S ascii);    
-  }
+		fmt::print("0x{:016x}  ", offset);
+		char ascii[1 + ROW_SZ];
+		ascii[ROW_SZ] = 0;
+		std::size_t i=0;
+		char hxbuf[3];
+		hxbuf[2] = 0;
+		for(;i<memory.size();i++) {
+			if (i && i % ROW_SZ == 0) {
+				prints("  ", S ascii);
+				fmt::print("\n0x{:016x}  ", i+offset);
+			}
+			hex02(memory[i], hxbuf);
+			prints(hxbuf, " ");
+			ascii[i%ROW_SZ] = ascii_map[memory[i]];
+		}
+		if (memory.size() % ROW_SZ != 0)
+		{
+			auto rest = memory.size() % ROW_SZ;
+			ascii[rest] = 0;
+			for(std::size_t j=0;j< ROW_SZ - rest;j++)
+				prints("   ");
+
+		}
+		fmt::print("  {}", S ascii);    
+	}
 }