IS_DIRECTORY cannot deal with delimiter on windows('\\") about libzippp

[ctabin]

Hi @aijyo,

In a ZIP file, all the entry paths are separated by a slash (/), even on windows. Hence, this macro should never be used to match "real" directory paths under windows. Maybe a better name would be "ENTRY_PATH_SEPARATOR".

And for your remark for a addDir API, the problem is the compatibility between linux and windows, because the file & folder API is platform dependant (or you need to include an API that provides a generic access, like Qt, but that would then be very heavy).

[aijyo]

Hi @aijyo,

In a ZIP file, all the entry paths are separated by a slash (/), even on windows. Hence, this macro should never be used to match "real" directory paths under windows. Maybe a better name would be "ENTRY_PATH_SEPARATOR".

And for your remark for a addDir API, the problem is the compatibility between linux and windows, because the file & folder API is platform dependant (or you need to include an API that provides a generic access, like Qt, but that would then be very heavy).

For the DIRECTORY_SEPARATOR problem , I normalized the path with the slash (/) (just like the nodejs.path.normalize...,but not the same) before call the libzipp api;

Add the addDir, I found a solution that seems not too heavy, I use dirent.h(https://github.com/tronkko/dirent), so I can enum dir file on windows same as linux

[ctabin]

When you use the methods of libzippp, remember that you are giving an entry name, not a directory path. Hence, as an entry name, only slashes (/) are interpreted as separators. The methods are not intended to handle real file paths.

Thanks for the link to dirent.h. I'm not sure yet I will include it, but it can be very useful.

[aijyo]

When you use the methods of libzippp, remember that you are giving an entry name, not a directory path. Hence, as an entry name, only slashes (/) are interpreted as separators. The methods are not intended to handle real file paths.

Thanks for the link to dirent.h. I'm not sure yet I will include it, but it can be very useful.

Thanks, and got it ...."you are giving an entry name, not a directory path"