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

Implement support for InputRange in vibe.web.rest & associates #2334

Closed
Geod24 opened this issue Jul 29, 2021 · 1 comment
Closed

Implement support for InputRange in vibe.web.rest & associates #2334

Geod24 opened this issue Jul 29, 2021 · 1 comment
Labels
C. Network Communication An issue which is related to network communication prio-high Will have an effect in the next three sprints
Milestone
6.1 TestNet Phase A

Comments

@Geod24
Copy link
Collaborator

Geod24 commented Jul 29, 2021

Currently we are forced to always return arrays when returning sets of data, which forces allocation.
We should be able to return an input range that can be consumed by Vibe.d at no or little cost.
Since we use interface and classes, the functions should use https://dlang.org/phobos/std_range_interfaces.html
This also will need to be implemented in LocalRest.
@Geod24 Geod24 added prio-high Will have an effect in the next three sprints C. Network Communication An issue which is related to network communication labels Jul 29, 2021
@Geod24 Geod24 added this to the 6. Stabilization milestone Jul 29, 2021
@Geod24
Copy link
Collaborator Author

Geod24 commented Oct 11, 2021

There's now support for InputStreamProxy (see #2530) so this is done.

@Geod24 Geod24 closed this as completed Oct 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
C. Network Communication An issue which is related to network communication prio-high Will have an effect in the next three sprints
Projects
None yet
Milestone
6.1 TestNet Phase A
Development

No branches or pull requests
1 participant
@Geod24