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

End-of-file on stdin is not detected on Mac OS #453

Closed
sgjesse opened this issue Nov 15, 2011 · 6 comments
Closed

End-of-file on stdin is not detected on Mac OS #453

sgjesse opened this issue Nov 15, 2011 · 6 comments
Assignees
Labels
area-core-library SDK core library issues (core, async, ...); use area-vm or area-web for platform specific libraries. library-io
Milestone

Comments

@sgjesse
Copy link
Contributor

sgjesse commented Nov 15, 2011

rUNNING the following program with dart_bin on Mac OS does not terminate when Ctrl-D is pressed.

void main() {
  stdin.dataHandler = () => stderr.write(stdin.read());
}

On Linux and Windows it does terminate.

On Linux Ctrl-D is detected by POLLIN event with 0 bytes available when the filedescriptor is stdin and it is connected to a terminal.

On Mac OS however Ctrl-D is reported as a character by POLLIN and when checking ioctl for available bytes. However when read is actually called Ctrl-D is not read an one less character than expected is read.

@sgjesse
Copy link
Contributor Author

sgjesse commented Feb 21, 2012

Removed Area-VM label.
Added Area-IO label.

@madsager
Copy link
Contributor

madsager commented Jun 7, 2012

Added this to the Later milestone.

@sgjesse
Copy link
Contributor Author

sgjesse commented Apr 16, 2013

Removed this from the Later milestone.
Added this to the M5 milestone.

@sgjesse
Copy link
Contributor Author

sgjesse commented May 28, 2013

Removed this from the M5 milestone.
Added this to the M6 milestone.

@sgjesse
Copy link
Contributor Author

sgjesse commented Jun 25, 2013

@kevmoo
Copy link
Member

kevmoo commented May 14, 2014

Removed Area-IO label.
Added Area-Library, Library-IO labels.

@sgjesse sgjesse added Type-Defect area-core-library SDK core library issues (core, async, ...); use area-vm or area-web for platform specific libraries. library-io labels May 14, 2014
@sgjesse sgjesse self-assigned this May 14, 2014
@sgjesse sgjesse added this to the M6 milestone May 14, 2014
nex3 pushed a commit that referenced this issue Aug 31, 2016
Make tool/get_chrome_canary.sh work on Mac
copybara-service bot pushed a commit that referenced this issue Dec 12, 2022
Revisions updated by `dart tools/rev_sdk_deps.dart`.

intl (https://github.com/dart-lang/intl/compare/a127902..881e53e):
  881e53e  Fri Dec 9 08:01:26 2022 -0800  Copybara-Service  Merge pull request #346 from lukepighetti:patch-1
  e5ca40d  Fri Dec 9 07:00:14 2022 -0800  Copybara-Service  Merge pull request #453 from selcukguvel:doc-fix
  edbfc06  Fri Dec 9 15:34:30 2022 +0100  Moritz  Merge branch 'master' into patch-1
  b489fdd  Fri Dec 9 15:30:23 2022 +0100  Moritz  Merge branch 'master' into doc-fix
  60a0d8c  Fri Dec 9 04:17:04 2022 -0800  Googler  Internal change
  ef953fd  Mon Mar 7 16:30:31 2022 +0300  selcukguvel  Fix NumberFormat document readability issue - Show "var eurosInUSFormat = .." on a new line
  aa98b67  Tue Dec 29 15:31:53 2020 -0500  Luke Pighetti  Remove 7 year old unimplemented fields

test (https://github.com/dart-lang/test/compare/73cd754..09fb067):
  09fb0679  Thu Dec 8 14:52:33 2022 -0800  Jacob MacDonald  prep packages for publishing (#1814)
  c8ac4382  Thu Dec 8 14:07:05 2022 -0800  Jacob MacDonald  Prepare for Dart 3.0 api removals (#1812)

Change-Id: I9a7caed3165d3b65e96001aa029602a09355bce8
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/274542
Auto-Submit: Michael Thomsen <mit@google.com>
Reviewed-by: Alexander Thomas <athom@google.com>
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-core-library SDK core library issues (core, async, ...); use area-vm or area-web for platform specific libraries. library-io
Projects
None yet
Development

No branches or pull requests

3 participants