fix arginfo and build warnings

[remicollet]

No description provided.

[codemasher]

I remember changing this here because it broke the build in Windows. I've checked with this PR and still getting ext\xz\utils.c(32): error C2036: 'void *': unknown size [...].

[remicollet]

Please try with new version

[codemasher]

That looks good! (tested with the php-8.0 dev branch)

[remicollet]

Perhaps worth a new version ?

I still like to check the read/write for stream, especially the error check, and the API in PHP >= 7.4 (error return -1) but won't have time in the next days

[codemasher]

Yes, i'm going to tag a new release - i have no problem to wait a few more days as i'm no C expert and i appreciate your contributions!
One thing i'd like to do is to add Windows builds to CI but from what it looks it's... complicated. Do you perhaps know of a php extension repo that has already accomplished that for GitHub actions?

[remicollet]

Yes, i'm going to tag a new release - i have no problem to wait a few more days as i'm no C expert and i appreciate your contributions!
One thing i'd like to do is to add Windows builds to CI but from what it looks it's... complicated. Do you perhaps know of a php extension repo that has already accomplished that for GitHub actions?

Perhaps @cmb69 may help for Windows CI ?

[cmb69]

See https://github.com/php/pecl-database-dbase/tree/master/.github/workflows (which uses https://github.com/cmb69/setup-php-sdk). There is no support for installing the dependency libraries yet (not even for the core dependencies), but in your case just fetching and unpacking liblzma might be sufficient for now.

[codemasher]

This is what i was looking for - thank you so much!