New issue
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
Realsense2 warnings fixed #8821
Conversation
common/utilities/time/work_week.cpp
Outdated
@@ -39,8 +39,9 @@ namespace time { | |||
|
|||
work_week::work_week( unsigned year, unsigned ww ) | |||
{ | |||
if( ww == 0 || ww > work_weeks_between_years( year + 1, year ) ) | |||
{ | |||
unsigned int yearww = unsigned(work_weeks_between_years(year + 1, year)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please put back into the next line
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
src/usb/usb-request.h
Outdated
@@ -22,7 +22,7 @@ namespace librealsense | |||
{ | |||
public: | |||
virtual rs_usb_endpoint get_endpoint() const = 0; | |||
virtual int get_actual_length() const = 0; | |||
virtual uint64_t get_actual_length() const = 0; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Dangerous -- and need to keep the type for "length" uniform -- so either int
, size_t
, or uint64_t
I prefer to keep int
and not change interfaces for this. Please see if you can change the usages rather than the definition.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
No description provided.