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.

Sign up for GitHub

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

Jump to bottom

Refer to 'a Lwt.t as promises rather than threads (proposal) #300

Closed
aantron opened this issue Dec 2, 2016 · 1 comment
Closed

Refer to 'a Lwt.t as promises rather than threads (proposal) #300

aantron opened this issue Dec 2, 2016 · 1 comment
Labels
docs proposal

Comments

@aantron
Copy link
Collaborator

aantron commented Dec 2, 2016

Calling 'a Lwt.ts threads causes a lot of confusion for beginners, who mistake Lwt threads for system threads, or at least expect aspects of system thread behavior. Many examples can be cited.

I think we should just use the common terminology of the day, and call Lwt a (monadic) promise library. We should completely eliminate all mention of threads outside of Lwt_preemptive or discussion of internals of Lwt_unix.

I would mind this when fixing/rewriting the docs.

Opinions welcome.
aantron added a commit that referenced this issue Dec 20, 2016
@aantron
Docs: start using term "promise."
be65cfa 
The manual and API docs are still using "thread," this will be changed
in the new manual (coming "soon").

Related #300.

[skip ci]
aantron added a commit that referenced this issue Dec 20, 2016
@aantron
Docs: start using term "promise."
8af751a 
The manual and API docs are still using "thread," this will be changed
in the new manual (coming "soon").

Related #300.

[skip ci]
aantron added a commit that referenced this issue Dec 20, 2016
@aantron
Docs: start using term "promise."
06638a9 
The manual and API docs are still using "thread," this will be changed
in the new manual (coming "soon").

Related #300.

[skip ci]
aantron added a commit that referenced this issue Dec 20, 2016
@aantron
Docs: start using term "promise."
8d64954 
The manual and API docs are still using "thread," this will be changed
in the new manual (coming "soon").

Related #300.

[skip ci]
@aantron aantron mentioned this issue Dec 22, 2016
Closed
@aantron
Copy link
Collaborator Author

aantron commented Oct 13, 2017

I think this is sufficiently addressed by the recent lwt.ml rewrite (#354) and lwt.mli rewrite (#469) to justify closing this issue. The remaining modules still use the term "thread," but they will be corrected soon.

@aantron aantron closed this as completed Oct 13, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
docs proposal
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@aantron