Skip to content
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.

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

When EOF (C-d) is received, quit interactive prompt #16

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Utkarsh-khambra
Copy link

@Utkarsh-khambra Utkarsh-khambra commented Dec 29, 2022

std::cin.clear() does clear the eofbit, but to get stdin back to working state we also need to call clearerr, but I've made changes to just quit interactive mode when EOF is received.

(fixes #15)

@whisperity
Copy link
Owner

Oh, hah! Thank you. I'll refrain from merging this for now, because I'm doing a massive refactor (see library-refactoring, but afterwards I do expect the patch to be a trivial rebase!

@whisperity whisperity added Kind: Bug 🐞 Something isn't working Target: Frontend 💻 The outside user-facing code labels Dec 30, 2022
@whisperity whisperity changed the title When EOF is received, quit monomux interactive mode. When EOF (C-d) is received, quit interactive prompt Dec 30, 2022
@whisperity
Copy link
Owner

Just checking in to say that my life has become hectic for the ending of the semester so this'll be kept on hold for a while. I've also started working on redoing the command-line interface (together with that library refactor) to break off the dependency on getopt (as it has a weird disjoint API that bit us already, see v0.1.1) altogether. With that the interactiveness of things might be touched which will affect this functionality.

But it might be that this all will happen only during the summer... 🙁

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Kind: Bug 🐞 Something isn't working Target: Frontend 💻 The outside user-facing code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Interactive prompt stuck on infinite prinout loop after C-d
2 participants