Skip to content
New issue

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

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

Already on GitHub? # to your account

Use spl_object_id instead of spl_object_hash #605

Merged
merged 1 commit into from
Feb 7, 2023

Conversation

mabar
Copy link
Contributor

@mabar mabar commented Feb 7, 2023

spl_object_id() should be bit more performant than spl_object_hash()

I found in PHP internals communication that there is one difference - spl_object_hash() should return same hash for equal objects. But it does not seem to be the case https://3v4l.org/pSNBF

@hrach hrach merged commit 94d1d44 into nextras:main Feb 7, 2023
@mabar mabar deleted the spl-object-id branch February 7, 2023 19:48
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants