Skip to content
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.

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

Add extra types to Memcache::getExtendedStats and Memcached::getStats #8700

Merged
merged 2 commits into from Nov 11, 2022
Merged

Add extra types to Memcache::getExtendedStats and Memcached::getStats #8700

merged 2 commits into from Nov 11, 2022

Conversation

bram123
Copy link
Contributor

@bram123 bram123 commented Nov 11, 2022

Tightened the return types of Memcached::getStats and Memcache::getExtendedStats, following the examples from:
https://www.php.net/manual/en/memcached.getstats.php
https://www.php.net/manual/en/memcache.getextendedstats.php

@orklah orklah added the release:fix The PR will be included in 'Fixes' section of the release notes label Nov 11, 2022
@orklah orklah merged commit 8807800 into vimeo:master Nov 11, 2022
@orklah
Copy link
Collaborator

orklah commented Nov 11, 2022

Thanks!

@bram123 bram123 deleted the memcache_callmap branch November 11, 2022 13:47
@danog
Copy link
Collaborator

danog commented Nov 11, 2022

Hmm, wouldn't it be better to add shaped arrays instead of generic array<...> arrays?

@bram123
Copy link
Contributor Author

bram123 commented Nov 11, 2022

Hmm, wouldn't it be better to add shaped arrays instead of generic array<...> arrays?

Ye probably, but afaik this depends on the $type input. And I'd have no idea on how to do this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release:fix The PR will be included in 'Fixes' section of the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants