Ole Christian Eidheim

Results 316 comments of Ole Christian Eidheim

Yes, that would work, we can add the meson.build file(s) only.

Thank you, but regarding the warnings, what compiler are you using?

For now I added https://github.com/eidheim/Simple-Web-Server/commit/50ce7510efe58ffea1e77351c0dc7a32eda5f46a. I don't enforce a whitespace policy, as long as there are no cleanup whitespace commits:) Regarding the warnings, these are warnings that are not shown...

`size_t` was chosen due to being an unsigned datatype.

Thank you, but could you elaborate on what you mean by: the client will be broken too?

Sorry for the late reply, but thank you for this. I have not yet had time to study the changes, but hopefully I will get a look at it soon.

I went a slightly different direction in https://github.com/eidheim/Simple-Web-Server/tree/base_class. Not sure what solution is best, but at least I know it is doable:)

Yes, for instance: ```c++ #include "client_https.hpp" #include "server_https.hpp" #include int main() { auto io_service = std::make_shared(); std::vector servers; servers.emplace_back(new SimpleWeb::Server()); servers.back()->io_service = io_service; servers.back()->config.port = 8080; servers.back()->default_resource["GET"] = [](std::shared_ptr response,...

For the time being, I changed it so that one now can write: ```c++ #include "client_https.hpp" #include "server_https.hpp" #include int main() { auto io_service = std::make_shared(); std::vector servers; servers.emplace_back(new SimpleWeb::Server());...

I managed to simplify https://github.com/eidheim/Simple-Web-Server/tree/base_class further by splitting the Response class instead of the Connection class.