Conversation
|
|
|
|
|
|
||
| using namespace utilities::time; | ||
| using namespace rsutils::time; // from rsutils/ | ||
| using namespace utilities::time; // from common/ |
There was a problem hiding this comment.
Not two classes.
We had one namespace, and the stuff in common/ added to it.
I didn't touch common/, so it stayed in the utilities namespace, while the stuff in rsutils moved.
common/ adds device-specific (L500 mainly) logic, which doesn't belong in rsutils.
There was a problem hiding this comment.
OK.. I must say it's confusing.
I can and I will. |
This is on purpose, at least in part -- here we need to override global mechanisms. Still, I'll take a look later. |
On the next one :) |
|
Thankyousir |
utilities->rsutils(project, paths, namespaces)utilities-pylibrealsense::to_string()(out of types.h) andrs2::to_string()(out of rendering.h) and move intorsutils::string::from()