Skip to content

New Stream API Support - #2281

Merged
yhirose merged 1 commit into
masterfrom
generator
Dec 2, 2025
Merged

yhirose merged 1 commit into
masterfrom
generator

Conversation

@yhirose

@yhirose yhirose commented Nov 27, 2025

Copy link
Copy Markdown
Owner

This PR is trying to implement #2269. It requires C++20 compiler or later which supports C++ coroutines.

@yhirose

yhirose commented Nov 27, 2025 •

Copy link
Copy Markdown
Owner Author

@ngxson the only thing of this new stream API is that it requires C++20 which supports C++ coroutines.

Here is README.

@yhirose
yhirose force-pushed the generator branch 5 times, most recently from 414be44 to a7cfdd3 Compare December 2, 2025 01:21
@yhirose yhirose changed the title New Stream API for C++20 New Stream API Support Dec 2, 2025
@yhirose
yhirose force-pushed the generator branch 3 times, most recently from 16381ea to bc058ea Compare December 2, 2025 02:07
@yhirose
yhirose merged commit 8bba34e into master Dec 2, 2025
16 of 17 checks passed
@yhirose

yhirose commented Dec 2, 2025

Copy link
Copy Markdown
Owner Author

@ngxson I discarded my first implementation for C++20, then implemented the stream API which works with C++11 according to your request at #2269.

@ngxson

ngxson commented Dec 2, 2025

Copy link
Copy Markdown

Thanks, indeed C++11 or 17 is better, but not necessarily as we already moved cpp-httplib to a dedicated unit in llama.cpp. We have some other vendors compiled with C++17 too, while the main project still uses 11, so it's probably fine even if internally you want to use C++20 features

@ngxson

ngxson commented Dec 2, 2025

Copy link
Copy Markdown

Just read through the documentation, this is more than enough for me. Thanks again 😄

@yhirose
yhirose deleted the generator branch December 5, 2025 00:20
noresources pushed a commit to noresources/cpp-httplib that referenced this pull request Dec 31, 2025
noresources pushed a commit to noresources/cpp-httplib that referenced this pull request Dec 31, 2025
noresources pushed a commit to noresources/cpp-httplib that referenced this pull request Dec 31, 2025
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.

2 participants