X Tutup
Skip to content

read may return -1 on error#548

Merged
randy3k merged 1 commit intomasterfrom
fix/stdin_read
May 22, 2022
Merged

read may return -1 on error#548
randy3k merged 1 commit intomasterfrom
fix/stdin_read

Conversation

@randy3k
Copy link
Member

@randy3k randy3k commented May 22, 2022

@milanglacier

Could you test this PR? I am thinking that read may return -1 in some cases, so the output is duplicated.

close #368

@randy3k
Copy link
Member Author

randy3k commented May 22, 2022

It seems that R 4.2 doesn't like setlocale in Windows and unlink is not able to resolve the filename.
@renkun-ken

@milanglacier
Copy link

This PR seems working fine on my side (macOS 12.3 with R 4.2)
tested on a rmd file of 1000 lines and a R file of 1500 lines

@randy3k randy3k merged commit b564896 into master May 22, 2022
@randy3k randy3k deleted the fix/stdin_read branch May 22, 2022 21:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Language Server crashing sporadically

2 participants

X Tutup