sebastiandev / zipper

C++ wrapper around minizip compression library
MIT License
515 stars 123 forks source link

Library ported to C++17 #80

Closed GiuseppeCesarano closed 3 years ago

GiuseppeCesarano commented 3 years ago

I know that pull request introduce a lot of changes, but as stated in #24, handling paths with strings adds complexity that's over the scope of this library, so i ported the library to C++17 because this standard adds the std::filesystem class that can fully replace the CDirEntry utility also being "more" cross platform. The changes are tested under Arch Linux with GCC 10.2 and Windows 10 with mingw-64 10.2. This pull request should fix #24.
Edit: Changes also tested in clang 10.0.1

Lecrapouille commented 3 years ago

@GiuseppeCesarano I merged and fixe merged conflict into dev-modernize. Please check if I did nothing wrong.