Andrew Auclair
Andrew Auclair
Are you running into this issue with `/ZI` in Debug? https://github.com/pocoproject/poco/issues/2913#issuecomment-586155527
That would be a breaking change to any existing calls to `information` that use those types. The only real option that comes to mind is to use `static_cast(__LINE__)` instead of...
> IMO this isn't breaking change Opinions aside, this is a breaking change. Using a template doesn't change the output. The same thing would happen with just `void information(const std::string&...
That would cause the same issue. Although it would only cause the issue for when the type of `__LINE__` is `long`. I'm also not sure that would actually work if...
Here's what I think would be a good compromise. A breaking change is required to properly account for `long` `__LINE__` and the current logging functions that log file and line...
> What about hiding the implementation of `Poco::Manifest::insert` and `Poco::Manifest::clear`, to make sure `who new, who delete`, then it will not crash anymore. That can't be done. `Manifest` is a...
This change appears to fail with the Thread Sanitizer. The actual log has much more in it . ``` testActiveThreadLoadBalancing: ================== WARNING: ThreadSanitizer: data race (pid=33153) Read of size 8...
I think there's still an issue with this related to multi-configs because of the below code in the root CMakeLists.txt. I'm going to give it a look later today. ```cmake...
Should Poco consider migrating to `std::any` now that C++17 is supported?
@matejk ~~did you mean to delete devel? Doesn't seem right.~~ Nevermind. I just saw the discussion post. Had way too many notifications to go through.