Try to avoid panicking in PosixParser if there's no /dev/tty #185
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
When using go-prompt before
/dev/
is populated, it willpanic()
fromNewStandardInputParser()
because/dev/
isn't populated yet.This will use stdin (file descriptor 0) as a fallback if
/dev/tty
doesn't exist.Furthermore
GetWinSize()
will return the default console size if theioctl()
call fails. This can happen if the default stdin is not aterminal, but a pipe as fixed in commit 846777c and described in
issue #88.