-
Notifications
You must be signed in to change notification settings - Fork 202
Open
Description
Your library seems very promising, but I thought I would let you know a few ideas I had while looking at it:
- Would be really great to get https support
- I'd request to avoid the use of C++ exceptions. Many C++ devs don't use them very much (they're notoriously expensive!)
- The single header, although it looks "easier to use" at first, is really difficult to browse. Splitting the source file into multiple files is almost always the best idea.
- style nit: I feel like you're overusing
autoin the code. I mean, in this case,intis shorter ;) The general guideline I've heard is to useautowhere it "increases readability", very subjective :) - There is some extra data copy that could be avoided. Consider going to C-arrays/
memcpyfor passing around some of the data? - Would be nice to have an
asyncSendthat accepts a lambda onComplete handler (something likestd::function<void (Response)>)
Metadata
Metadata
Assignees
Labels
No labels