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

Use player.Iterator() instead of player.GetAll() for optimization #11

Closed

Winkarst-cpu wants to merge 1 commit into glua:master from Winkarst-cpu:iterator

Winkarst-cpu commented Jan 7, 2025

player.Iterator() works faster than ipairs(player.GetAll()).

https://wiki.facepunch.com/gmod/player.Iterator


          Use player.Iterator() instead of player.GetAll()

5df522d

Collaborator

FPtje commented Jan 7, 2025

Thanks for the PR! Sadly the iterator do not always work as intended, and therefore should not be used at all. See the discussion here:

FPtje/DarkRP#3255

FPtje closed this

Winkarst-cpu deleted the iterator branch

January 7, 2025 18:49

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

Labels

None yet