Nathan Moinvaziri
Nathan Moinvaziri
Still failing on some ARM.. weird.
I worry about adding union into the code, since people are prone to try and compile zlib-ng in C++. For now, I have excluded `__arm__` from `Z_TOUCH` macro definition. If...
@dougallj I have enabled it now only for platforms we know work. Take a look at that and let me know if it looks good to you.
Rebased on top of latest `develop`. Benchmarks may yield better results now.
@KungFuJesus that sounds like an interesting idea.
> MSVC failures: `inftrees.c(234): warning C4242: 'function': conversion from 'unsigned int' to 'uint16_t', possible loss of data` Fixed.
@Dead2 done, I also found another instance where we can use `uint16_t`.
I'm not sure, that stuff doesn't makes sense to me. @prateek9623 do you have any idea?
## Benchmarks ``` Running /Users/nathan/Source/zlib-ng/build/test/benchmarks/benchmark_zlib Run on (8 X 24 MHz CPU s) CPU Caches: L1 Data 64 KiB L1 Instruction 128 KiB L2 Unified 4096 KiB (x8) Load Average:...
~~I will rebase after PR #2028 lands.~~ Not necessary.