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

Take idle socket setup out of assert wrapper. #2566

Merged

elliefm merged 1 commit into cyrusimap:master from zlynx:assert-fix

Oct 24, 2018

Contributor

zlynx commented Oct 23, 2018

This fixes having side effects inside of assert calls. Issue #2564

Nothing critical about this, just seeing this in the code bugged me.


          Take idle socket setup out of assert wrapper.

9a14f22

elliefm merged commit 9c3c9c3 into cyrusimap:master

Contributor

elliefm commented Oct 24, 2018

Agreed, thanks!

zlynx deleted the assert-fix branch

October 24, 2018 23:31

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

Labels

None yet