minizip: use single ZREAD64 call in unz64local_getShort/Long/Long64

[eugenegff]

minizip: use single ZREAD64 call in unz64local_getShort/Long/Long64 implementation, rather than read it byte by byte.

Code is still compatible with big endian architectures. At the same time both Clang and GCC are able to reinterpret local uchar c[N] array as appropriately sized little endian unsigned number on x64 arch, instead of performing shifting and or-ing calculations.

[gvollant]

@madler @Neustradamus this PR is a good idea and fine, can be merged

[madler]

@eugenegff Can you fix this commit to work on the develop branch?

[eugenegff]

@eugenegff Can you fix this commit to work on the develop branch?

Wow, real Mark Adler wrote to me! :)
Done.

[Neustradamus]

Linked to:

• minizip: use single ZREAD64 call in unz64local_getShort/Long/Long64 #557

[madler]

@eugenegff Thanks, but it still says there are conflicts.

[eugenegff]

@eugenegff Thanks, but it still says there are conflicts.

@madler I don't see any conflicts, only macOS GCC build agent says that

Could not find compiler set in environment variable CC:

Anyway, I rebased commit directly above the top of the madler:develop branch rather than merge with it. Working copy was not changed, only commits tree is simplified. macOS GCC build agent still complaining that it

Could not find compiler set in environment variable CC:

[madler]

@eugenegff Thanks! That worked. (No, not from the build agent. It said right here on this page that there were conflicts, where it now says "This branch has no conflicts.")

Applied.

[Neustradamus]

Merged commit:

• be7aa11