Skip to content
This repository has been archived by the owner on Dec 9, 2018. It is now read-only.

add synchronization to stdin/stdout/stderr #90

Open
japaric opened this issue Feb 3, 2017 · 3 comments
Open

add synchronization to stdin/stdout/stderr #90

japaric opened this issue Feb 3, 2017 · 3 comments

Comments

@japaric
Copy link
Owner

japaric commented Feb 3, 2017

The current implementation is not synchronized. This is OK for now because we don't have threads. Once we get threads the implementation will have to be updated. The code that uses synchronization is already there but we haven't switched to it yet.

This is blocked by TLS support. cc #30

@tbu-
Copy link
Collaborator

tbu- commented Feb 3, 2017

Not thread-safe is the wrong word. It is thread-safe (as in memory-safe), but lines from different threads can be interleaved.

@japaric japaric changed the title make stdin/stdout/stderr thread-safe add synchronization to stdin/stdout/stderr Feb 3, 2017
@japaric
Copy link
Owner Author

japaric commented Feb 3, 2017

You are right. Changed thread-safe to synchronization.

@tbu-
Copy link
Collaborator

tbu- commented Feb 7, 2017

Let's make this the tracking issue for io::stdout.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants