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

nitcorn: intro an alternative to answer allowing more control and revamp the README file #1828

Merged
merged 4 commits into from
Nov 11, 2015

Conversation

xymus
Copy link
Contributor

@xymus xymus commented Nov 10, 2015

The new prepare_respond_and_close is an advanced alternative to answer. It allows mainly to delay answering to some requests. I may this to implement simple push notifications. It could also be used to do some parallel or IO intensive work, while processing other requests.

The new README should be more up to date and a better support to get to know the library. I'm not sure about including a code example, but I thought it could help. The file could still benefit from some work, but it's an improvement.

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>
Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>
Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

fix prepare
Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>
@privat
Copy link
Member

privat commented Nov 10, 2015

great code and doc. +1

privat added a commit that referenced this pull request Nov 11, 2015
…l and revamp the README file

The new `prepare_respond_and_close` is an advanced alternative to `answer`. It allows mainly to delay answering to some requests. I may this to implement simple push notifications. It could also be used to do some parallel or IO intensive work, while processing other requests.

The new README should be more up to date and a better support to get to know the library. I'm not sure about including a code example, but I thought it could help. The file could still benefit from some work, but it's an improvement.

Pull-Request: #1828
Reviewed-by: Jean Privat <jean@pryen.org>
@privat privat merged commit 75c0efe into nitlang:master Nov 11, 2015
@xymus xymus deleted the nitcorn branch November 15, 2016 23:18
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.

None yet

2 participants