Remove deprecated constructors for using zeq::URI #141
Conversation
|
Changelog update and version bump missing. +1 otherwise. |
|
Hold on, not complete yet:
|
|
So, additionally all "todos" have to be resolved for this topic. |
|
The mentioned todos are part of this topic, yes. Not sure if it's all todos in the project. |
| BOOST_CHECK_EQUAL( hostPort.getScheme(), "tcp" ); | ||
| } | ||
|
|
||
| BOOST_AUTO_TEST_CASE(custom_schema_not_supported_yet) |
There was a problem hiding this comment.
that test is still valid imo.
There was a problem hiding this comment.
// TODO: only for compatibility for hbp schemas, remove after downstreams
// are fixed
I removed this to do from uri.cpp. What is in mind for this then ?
There was a problem hiding this comment.
The schema is left untouched, so actually only line 46 changes. The testname is then custom_schema. For the zeq::URI this is supported then, but not yet for the publisher/subscriber. Will do later then when needed. inproc could be interesting to have 'local' GUI only for instance.
No description provided.