Partially address review, stil need to change invalid utf8 tests #108

Job	Run time
Interpreter (Debug)	6m 35s
i686-pc-windows-msvc/msvc (Debug)	16m 21s
Free-Threaded (Debug)	10m 2s
i686-pc-windows-msvc/msvc (Release)	20m 5s
x86_64-pc-windows-msvc/msvc (Debug)	16m 12s
x86_64-pc-windows-msvc/msvc (Release)	20m 45s
aarch64-pc-windows-msvc/msvc (Debug)	3m 42s
aarch64-pc-windows-msvc/msvc (Release)	6m 18s
x86_64-apple-darwin/clang (Debug)	10m 16s
x86_64-apple-darwin/clang (Release)	17m 24s
aarch64-apple-darwin/clang (Debug)	9m 3s
aarch64-apple-darwin/clang (Release)	13m 30s
x86_64-unknown-linux-gnu/gcc (Debug)	7m 56s
x86_64-unknown-linux-gnu/gcc (Release)	17m 24s
x86_64-unknown-linux-gnu/clang (Debug)	7m 24s
x86_64-unknown-linux-gnu/clang (Release)	13m 35s
aarch64-unknown-linux-gnu/gcc (Debug)	1h 10m 5s
aarch64-unknown-linux-gnu/gcc (Release)	23m 11s
aarch64-unknown-linux-gnu/clang (Debug)	1h 5m 5s
aarch64-unknown-linux-gnu/clang (Release)	35m 52s
	6h 30m 45s

Provide feedback