AZero13

If allocation for dinfoptr fails, output an error and exit

This isn't really a cleanup. This is the only case where an allocation error does not get printed UNLESS it is verbose, which doesn't align with the other behavior. Let's...

Replace old workarounds from when C wasn't standardized

1

Replace many outdated workarounds from when C wasn’t standardized yet with their modern day equivalents.

Use C++ features and headers in C++ code instead of C

1

We don’t need to use integers as booleans or the C standard headers when we have the C++ standard headers and booleans

Fix potential overflow when comparing wide char types

2

Comparing wide char types may result in an overflow when subtracting them if one value happens to be extremely large and the other isn’t, as the difference is evaluated as...

Convert isipv6 to bool since it is used as one

2

I also removed an unused hw_tls struct member that I found was completely unused.

memchr and memcmp both work by comparing the values as unsigned chars. This is mostly an issue when memchr is called, as the (int) cast causes a sign extension instead...

Memmove is a valid way to implement memcpy. However, memcpy isn’t a valid way to implement memmove. For this reason, I switched them around. No ABI breakage should occur

Some code clearly was written to please the compiler and avoid a warning, as Werror is enabled, or trick older compilers into producing better code generation. However, the way this...