-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Replace QNetwork operations with libcurl + wolfssl #7771
Conversation
61bb8f8
to
12752c7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here's some stuff that'll fix the most oblivous errors on windows side. It still won't compile even with these fixed due to don't know how to make '3rdparty\wolfssl_cmake\wolfssl.lib'
, but I think that'll get sorted out once you fix the linux build.
You need to define NOMINMAX, either for whole project in ConfigureCompiler or for the files which use libcurl. If you define it for whole project, remove it from stdafx.h
to avoid multiple definition spam.
12752c7
to
a0567e9
Compare
5ee8ebe
to
82ee6e4
Compare
278b24a
to
af1fbbe
Compare
|
||
#define NOMINMAX | ||
#define CURL_STATICLIB | ||
#include <curl/curl.h> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CURL probably can be declared as a forward declaration under extern "C"{}, with an actual include in cpp file.
d766e3a
to
db81b77
Compare
63c8f62
to
ad9ac27
Compare
5234c07
to
df35e4f
Compare
} | ||
} | ||
|
||
// Parse expression(or at least used to, was nuked to remove the need for QtJsEngine) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Press F to pay respects.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
F
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
// F
closes #7575