-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Splitting C++ REST SDK #68
Comments
This is a really interesting idea, but it would be a large restructuring of the project so we need to be careful and ensure it's the right thing to do. As an experiment, adding macros to selectively disable parts of the library might be an interesting approach (we currently have a CPPREST_EXCLUDE_WEBSOCKETS macro which disables websockets; this also removes the need for Boost and OpenSSL on Windows). |
+1 I only need PPLX. It would be good if it could be split from the cpprestsdk project |
I was actually looking into making the cross-platform port of the PPL stand-alone. ras, you mention that it would require a large restructuring of the code. Is it not simply possible to copy the This section of the PPL is actually pretty useful and really should be available to everyone because there's no good libraries out there that seem to match what MS has created here. |
I have project with the same needs. My project also needed narrow character width support on Windows. I tackled the latter first and have completed infrastructure to isolate components include pplx and json. Please see #1204. In CMake you may depend on cpprest_json and/or cpprest_pplx. |
I was facing somewhat annoying "issue". sometimes I just need the PPLX, sometimes just to parse or create a piece of JSON. But whatever I need I have to add the whole C++ REST SDK NuGet package. Not a big issue but maybe it is a good idea to have the PPLX and JSON as standalone packages?
In addition, the PPLX is not a full port of PPL, for example it has no parallel algorithms or parallel_invoke functionality, any plans to add it?
The text was updated successfully, but these errors were encountered: