Skip to content
Navigation Menu
Sign in
Appearance settings
Platform
AI CODE CREATION
GitHub Copilot
Write better code with AI
GitHub Copilot app
Direct agents from issue to merge
MCP Registry
Integrate external tools
DEVELOPER WORKFLOWS
Actions
Automate any workflow
Codespaces
Instant dev environments
Issues
Plan and track work
Code Review
Manage code changes
Code Quality
Enforce quality at merge
APPLICATION SECURITY
GitHub Advanced Security
Find and fix vulnerabilities
Code security
Secure your code as you build
Secret protection
Stop leaks before they start
EXPLORE
Why GitHub
Documentation
Blog
Changelog
Marketplace
View all features
Solutions
BY COMPANY SIZE
Enterprises
Small and medium teams
Startups
Nonprofits
BY USE CASE
App Modernization
DevSecOps
DevOps
CI/CD
View all use cases
BY INDUSTRY
Healthcare
Financial services
Manufacturing
Government
View all industries
View all solutions
Resources
EXPLORE BY TOPIC
AI
Software Development
DevOps
Security
View all topics
EXPLORE BY TYPE
Customer stories
Events & webinars
Ebooks & reports
Business insights
GitHub Skills
SUPPORT & SERVICES
Documentation
Customer support
Community forum
Trust center
Partners
View all resources
Open Source
COMMUNITY
GitHub Sponsors
Fund open source developers
PROGRAMS
Security Lab
Maintainer Community
GitHub Stars
Archive Program
REPOSITORIES
Topics
Trending
Collections
Enterprise
ENTERPRISE SOLUTIONS
Enterprise platform
AI-powered developer platform
AVAILABLE ADD-ONS
GitHub Advanced Security
Enterprise-grade security features
Copilot for Business
Enterprise-grade AI features
Premium Support
Enterprise-grade 24/7 support
Pricing
Search
/
Sign in
Sign up
Appearance settings
You signed in with another tab or window.
Reload
to refresh your session.
You signed out in another tab or window.
Reload
to refresh your session.
You switched accounts on another tab or window.
Reload
to refresh your session.
Dismiss alert
{{ message }}
Uh oh!
There was an error while loading.
Please reload this page
.
ros2
/
rclcpp
Public
Notifications
You must be signed in to change notification settings
Fork
564
Star
805
Code
Issues
236
Pull requests
86
Actions
Security and quality
0
Insights
Additional navigation options
Code
Issues
Pull requests
Actions
Security and quality
Insights
Fix action server deadlock (#1285)
- #1313
#1313
Merged
clalancette
merged 8 commits into
ros2:master
ros2/rclcpp:master
from
daisukes:foxy
daisukes/rclcpp:foxy
Copy head branch name to clipboard
Jan 14, 2021
Conversation
Commits
8
(8)
Checks
Files changed
Merged
Fix action server deadlock (#1285)
#1313
clalancette
merged 8 commits into
ros2:master
ros2/rclcpp:master
from
daisukes:foxy
daisukes/rclcpp:foxy
Copy head branch name to clipboard
Commits
Commits on Nov 21, 2020
add a test case to reproduce deadlock situation in rclcpp_action
Show description for 760c980
daisukes
committed
760c980
View commit details
Copy full SHA for 760c980
Browse repository at this point
Commits on Dec 3, 2020
unlock action_server_reentrant_mutex_ before calling user callback functions
Show description for d2bd783
daisukes
committed
d2bd783
View commit details
Copy full SHA for d2bd783
Browse repository at this point
Commits on Jan 11, 2021
Revert "unlock action_server_reentrant_mutex_ before calling user callback functions"
Show description for e1ee318
daisukes
committed
e1ee318
View commit details
Copy full SHA for e1ee318
Browse repository at this point
minimize lock scopes to avoid deadlock
Show description for 6feedbe
daisukes
committed
6feedbe
View commit details
Copy full SHA for 6feedbe
Browse repository at this point
Commits on Jan 12, 2021
remove an unnecessary lock
Show description for 054bf57
daisukes
committed
054bf57
View commit details
Copy full SHA for 054bf57
Browse repository at this point
make action_server_reentrant_mutex_ scope smaller
Show description for 5ae0e9a
daisukes
committed
5ae0e9a
View commit details
Copy full SHA for 5ae0e9a
Browse repository at this point
Merge remote-tracking branch 'master' into foxy
daisukes
committed
ca4c851
View commit details
Copy full SHA for ca4c851
Browse repository at this point
Commits on Jan 13, 2021
use std::atomic<bool> to make pimpl flags to atomic
Show description for 5adb529
daisukes
committed
5adb529
View commit details
Copy full SHA for 5adb529
Browse repository at this point
You can’t perform that action at this time.