New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

start making curl types more specific #11140

Open

kkmuffme wants to merge 1 commit into vimeo:5.x from kkmuffme:start-making-curl-types-more-specific

Contributor

kkmuffme commented Oct 23, 2024 •

edited

Loading

Start to make curl param and return types more specific.

Things that still need improving:

curl_getinfo
curl_setopt
param types where many are listed can potentially be more restricted when one goes through the actual constant values and their meaning. I just used the widest set possible according to the docs now.

Note: for the return type https://curl.se/libcurl/c/libcurl-errors.html needs to be used, bc the PHP docs https://www.php.net/manual/en/curl.constants.php are missing various values (the non-listed values can be returned, but not used as param, that's maybe why some of them are missing in the docs, but not sure)

kkmuffme force-pushed the start-making-curl-types-more-specific branch from c6bc1e6 to 91f22d0 Compare

October 23, 2024 20:10


          start making curl types more specific

79a587b

kkmuffme force-pushed the start-making-curl-types-more-specific branch from 91f22d0 to 79a587b Compare

October 23, 2024 20:15

kkmuffme marked this pull request as ready for review

October 23, 2024 20:22

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet