QueuePushHandler should not pollute main namespace of RSB
|Assignee:||J. Wienke||% Done:|
Currently the queue push handler resides in the top-level namespace of RSB, which looks kind of awkward to me as it is a utility class, which is not necessary for RSB to function properly. I would vote to move it to a util namespace or something like that. If we don't do it now, the API break will get event worse later if more of these special handlers are there.
This also includes EventQueuePushHandler
- Target version changed from rsb-0.9 to rsb-0.10
- Status changed from New to In Progress
- Assignee set to J. Wienke
- Status changed from In Progress to Resolved
- % Done changed from 0 to 100
Since this constitutes an incompatible API change, can you add it to the section for version 0.10 in
news.rst in the manual?
Backwards compatibility is so far maintained with a deprecation warning. Do you think this is necessary then?
Maybe a note could be added so people can start to migrate?
A deprecation compiler warning is issues in case you are using the old includes. I think we discussed that this is enough so far.
Also available in: Atom