Skip to content
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

eliminate [-Wdepreated-declarations] warning message #54

Merged
merged 1 commit into from
Oct 5, 2023

Conversation

takasehideki
Copy link
Member

This PR is just eliminating warning message, so I will merge this without proper test.

/var/mbed/mros2/src/mros2.cpp: In function 'void mros2::spin()':
/var/mbed/mros2/src/mros2.cpp:301:33: warning: 'void rtos::ThisThread::sleep_for(uint32_t)' is deprecated: Pass a chrono duration, not an integer millisecond count. For example use `5s` rather than `5000`. [since mbed-os-6.0.0] [-Wdeprecated-declarations]
  301 |       ThisThread::sleep_for(1000);
      |                                 ^
In file included from /var/mbed/mbed-os/rtos/./include/rtos/rtos.h:30,
                 from /var/mbed/mbed-os/mbed.h:23,
                 from /var/mbed/mros2/src/mros2.cpp:6:
/var/mbed/mbed-os/rtos/./include/rtos/ThisThread.h:216:6: note: declared here
  216 | void sleep_for(uint32_t millisec);
      |      ^~~~~~~~~

```
/var/mbed/mros2/src/mros2.cpp: In function 'void mros2::spin()':
/var/mbed/mros2/src/mros2.cpp:301:33: warning: 'void rtos::ThisThread::sleep_for(uint32_t)' is deprecated: Pass a chrono duration, not an integer millisecond count. For example use `5s` rather than `5000`. [since mbed-os-6.0.0] [-Wdeprecated-declarations]
  301 |       ThisThread::sleep_for(1000);
      |                                 ^
In file included from /var/mbed/mbed-os/rtos/./include/rtos/rtos.h:30,
                 from /var/mbed/mbed-os/mbed.h:23,
                 from /var/mbed/mros2/src/mros2.cpp:6:
/var/mbed/mbed-os/rtos/./include/rtos/ThisThread.h:216:6: note: declared here
  216 | void sleep_for(uint32_t millisec);
      |      ^~~~~~~~~
```
@takasehideki takasehideki merged commit 09617d7 into main Oct 5, 2023
@takasehideki takasehideki deleted the eliminate_mros2_cpp_warning branch October 5, 2023 10:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant