fixed compilation in MSVC 2013 and enable IOWIN32_USING_WINRT_API only on non-desktop Windows platforms

[kolomenkin]

On my computer I have WINAPI_FAMILY_ONE_PARTITION undefined and WINAPI_FAMILY_PARTITION defined. This generates compilation error with current development sources.

I have MSVC 2013 update 5, windows 8.1, SDK 8.1

I modified the code to have few changes:

• check for WINAPI_FAMILY defined to decide current SDK knows about different Microsoft platforms.
• include <winapifamily.h> when it is available. It guarantees WINAPI_PARTITION_DESKTOP is really defined here.
• define IOWIN32_USING_WINRT_API only on non-desktop platforms. So we support very old API on desktop and support as much Windows version as possible. In other case we would support Windows 8 only.

[kolomenkin]

pull requests was updated

[asfaque-alam]

The zlib 1.2.11 still don't have this commit . Is it going to be in next releases.

[kolomenkin]

unfortunately I can't check this in nearest future

[kolomenkin]

@timofonic, I found some time.

I can't build latest develop branch of https://github.com/Dead2/zlib-ng/ because of

1. missing zconf-ng.h
2. missing minizip contrib subdir

Maybe you are generating zconf-ng.h by some configure script, but original zlib did not need configure step. And it could be built in Windows by just adding existing sources to another project.

[Neustradamus]

Please note that there is https://github.com/nmoinvaz/minizip.

[kolomenkin]

I do not get the status of this PR. Do you need any actions from me?

[philippeVerney]

Already fixed in https://github.com/madler/zlib/commit/ce12773790517034317274f5c65ba70cfeea29f7 ?

[tbeu]

Already fixed in ce12773 ?

Yes, finally. Duplicate of #233.

[Neustradamus]

Linked to:

• https://github.com/madler/zlib/pull/610