Skip to content

Conversation

@lpinca
Copy link
Member

@lpinca lpinca commented Oct 1, 2020

Specify how to detect end-of-file when using fs.read() and
filehandle.read().

Fixes: #35363

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • documentation is changed or added
  • commit message follows commit guidelines

@nodejs-github-bot nodejs-github-bot added doc Issues and PRs related to the documentations. fs Issues and PRs related to the fs subsystem / file system. labels Oct 1, 2020
@lpinca lpinca added the request-ci Add this label to start a Jenkins CI on a PR. label Oct 1, 2020
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Oct 1, 2020
@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

Specify how to detect end-of-file when using `fs.read()` and
`filehandle.read()`.

PR-URL: nodejs#35445
Fixes: nodejs#35363
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Zeyu Yang <[email protected]>
Reviewed-By: Andrey Pechkurov <[email protected]>
@lpinca lpinca merged commit 0f9170e into nodejs:master Oct 4, 2020
@lpinca
Copy link
Member Author

lpinca commented Oct 4, 2020

Landed in 0f9170e.

@lpinca lpinca deleted the fix/issue-35363 branch October 4, 2020 06:03
danielleadams pushed a commit that referenced this pull request Oct 6, 2020
Specify how to detect end-of-file when using `fs.read()` and
`filehandle.read()`.

PR-URL: #35445
Fixes: #35363
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Zeyu Yang <[email protected]>
Reviewed-By: Andrey Pechkurov <[email protected]>
@danielleadams danielleadams mentioned this pull request Oct 6, 2020
joesepi pushed a commit to joesepi/node that referenced this pull request Jan 8, 2021
Specify how to detect end-of-file when using `fs.read()` and
`filehandle.read()`.

PR-URL: nodejs#35445
Fixes: nodejs#35363
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Zeyu Yang <[email protected]>
Reviewed-By: Andrey Pechkurov <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc Issues and PRs related to the documentations. fs Issues and PRs related to the fs subsystem / file system.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fs.read documentation does not specify how to detect EOF condition

9 participants