-
Notifications
You must be signed in to change notification settings - Fork 8
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
♻️ Use std::function instead of mbed::Callback #1255
♻️ Use std::function instead of mbed::Callback #1255
Conversation
eb7d47a
to
88b4d88
Compare
File comparision analysis report🔖 Info
Click to show memory sections
📝 SummaryClick to show summary
🗺️ Map files diff outputClick to show diff list
|
File comparision analysis report🔖 Info
Click to show memory sections
📝 SummaryClick to show summary
🗺️ Map files diff outputClick to show diff list
|
Kudos, SonarCloud Quality Gate passed! |
Codecov Report
@@ Coverage Diff @@
## develop #1255 +/- ##
========================================
Coverage 96.20% 96.20%
========================================
Files 148 148
Lines 3713 3715 +2
========================================
+ Hits 3572 3574 +2
Misses 141 141
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
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.
thanks for this PR! good to see mbed::callback going away in a lot of places :)
I've added a few suggestions where I think that mbed::callback can be completely removed and replaced by simple lambdas.
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.
Nice 👍
- Validated on robot
Validations