Replace boost with C++11 [DISCL-391] #130
Conversation
deflect/Stream.h
Outdated
|
|
||
| /** @name Asynchronous send API */ | ||
| //@{ | ||
| /** Future signaling success of asyncSend(). @version 1.1 */ |
There was a problem hiding this comment.
That breaks API imo. No longer version 1.1
|
Yes, I thought I could easily remove all when I started. But I couldn't find an elegant solution for the shared_ptr serialization. I could copy the Frame in Tide from std::shared_ptr to boost::shared_ptr before serializing, which wouldn't be too expensive because the segments are QByteArray (implicitly shared) but still look a bit awkward. The boost::signal could be replaced by a callback function which would be fine for our use case. I was not sure and decided to stop here, which is still a step in the right direction (the only problem is the one API breakage you pointed out). |
|
And I notice now that the API change requires a change in Equalizer because eq/deflect/Proxy.cpp because it explicitly uses boost in a make_ready_future() function... |
|
Updated: I decided that since this was already breaking the API I might as well do all the changes and make boost optional. |
rdumusc
changed the title
CMakeLists.txt
Outdated
| set(DEFLECT_DESCRIPTION "A fast C++ library for streaming pixels and events") | ||
| set(DEFLECT_MAINTAINER "Blue Brain Project <[email protected]>") | ||
| set(DEFLECT_LICENSE BSD) | ||
| set(DEFLECT_DEPENDENT_LIBRARIES Boost) |
doc/Changelog.md
Outdated
|
|
||
| * [130](https://github.com/BlueBrain/Deflect/pull/130) | ||
| Replaced boost by C++11. Boost is now an optional dependency and it is used | ||
| only by the tests. Minor API changes were introdued by this change. |
There was a problem hiding this comment.
Not so minor imo.
typo 'introdued'
boost program_options and unit_test are now optional (tests only)
|
+1 |
2 participants
Notes:
serializing it and does not mandate boost >= 1.56.
however it is only used internally by QmlStreamerImpl.cpp.