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 instruction order to return early #26

Merged

padenot merged 1 commit into padenot:main from chrisguttandin:avoid-unnecessary-assignments

Dec 20, 2023

Contributor

chrisguttandin commented Dec 20, 2023

This PR just re-orders the capacity check inside ParameterWriter.enqueue_change() to avoid two unnecessary assignments in case it fails.


          Fix instruction order to return early

267ce70

padenot merged commit c2058e3 into padenot:main

2 checks passed

Owner

padenot commented Dec 20, 2023

Thanks!

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

Labels

None yet