Remove some side-effecting asserts #38

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
2 participants
Contributor

kmcallister commented Mar 6, 2012

If someone built Mosh with -DNDEBUG, these effects would go away.

Owner

keithw commented Mar 6, 2012

Thanks for this -- nice catches. I am going to try to merge the OS X port first though before making code changes.

Contributor

kmcallister commented Mar 8, 2012

Rebased for the new selfpipe code.

Owner

keithw commented Mar 8, 2012

Thanks -- merged #1 and did #2 for (hopefully) all the side-effecting asserts in the codebase.

keithw closed this Mar 8, 2012

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