Skip to content

Fix reading redirected input from stdin#810

Merged
lhecker merged 1 commit intomainfrom
dev/lhecker/stdin
Apr 23, 2026
Merged

Fix reading redirected input from stdin#810
lhecker merged 1 commit intomainfrom
dev/lhecker/stdin

Conversation

@lhecker
Copy link
Copy Markdown
Member

@lhecker lhecker commented Apr 22, 2026

This broke in #556, which moved the stdin reopen from init() into
switch_modes(). But open_stdin_if_redirected() was called inside
handle_args(), which runs before switch_modes().
So, after #556, it would never see the reopened stdin.

@lhecker lhecker added I-bug It shouldn't be doing this. P-high High priority, fix ASAP. Example: The application crashes. labels Apr 22, 2026
@lhecker lhecker enabled auto-merge (squash) April 22, 2026 10:50
@lhecker lhecker requested review from DHowett and removed request for carlos-zamora and qbradley April 23, 2026 12:39
@lhecker lhecker merged commit 9e7b938 into main Apr 23, 2026
8 checks passed
@lhecker lhecker deleted the dev/lhecker/stdin branch April 23, 2026 15:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

I-bug It shouldn't be doing this. P-high High priority, fix ASAP. Example: The application crashes.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants