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

fix(aos): only remove handler if name matches #366

Merged

twilson63 merged 1 commit into main from jfrain99/only-remove-handler-if-name-match

Oct 15, 2024

Contributor

jfrain99 commented Oct 14, 2024

If nil is passed as index to table.remove, it removes the last element. Now, only call if idx exists.

https://www.lua.org/pil/19.2.html


          fix(aos): only remove handler if name matches

33c6085

twilson63 approved these changes

View reviewed changes

Collaborator

twilson63 left a comment

LGTM

twilson63 merged commit 4c545e0 into main

3 checks passed

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

Labels

None yet