We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
We use assert to check the sanity of data inserted in the buffer:
https://github.com/robotology-playground/yarp-telemetry/blob/306c9a047655fc65bf8072006193291bde767346/src/libYARP_telemetry/src/yarp/telemetry/BufferManager.h#L75-L85
or to check if the BufferManager is constructed correctly: https://github.com/robotology-playground/yarp-telemetry/blob/306c9a047655fc65bf8072006193291bde767346/src/libYARP_telemetry/src/yarp/telemetry/BufferManager.h#L39-L43
The problem is that in Release the assert is trimmed out and can raise issues or bugs like this.
Release
assert
We have to find out an alternative, that maybe can be exceptions also if I am not a big fan of c++ exceptions
The text was updated successfully, but these errors were encountered:
No branches or pull requests
We use assert to check the sanity of data inserted in the buffer:
https://github.com/robotology-playground/yarp-telemetry/blob/306c9a047655fc65bf8072006193291bde767346/src/libYARP_telemetry/src/yarp/telemetry/BufferManager.h#L75-L85
or to check if the BufferManager is constructed correctly:
https://github.com/robotology-playground/yarp-telemetry/blob/306c9a047655fc65bf8072006193291bde767346/src/libYARP_telemetry/src/yarp/telemetry/BufferManager.h#L39-L43
The problem is that in
Release
theassert
is trimmed out and can raise issues or bugs like this.We have to find out an alternative, that maybe can be exceptions also if I am not a big fan of c++ exceptions
The text was updated successfully, but these errors were encountered: