Skip to content

Make XmlRpcValue arguments const references#3419

Merged
rhaschke merged 1 commit intomoveit:masterfrom
ubi-agni:xmlrpc-casts
May 4, 2023
Merged

Make XmlRpcValue arguments const references#3419
rhaschke merged 1 commit intomoveit:masterfrom
ubi-agni:xmlrpc-casts

Conversation

@rhaschke
Copy link
Copy Markdown
Contributor

@rhaschke rhaschke commented May 4, 2023

This was broken upstream before, but seems to be fixed now (ros/ros_comm#1978)

This was broken upstream before, but seems to be fixed now.
@rhaschke rhaschke requested a review from mlautman as a code owner May 4, 2023 13:02
@codecov
Copy link
Copy Markdown

codecov bot commented May 4, 2023

Codecov Report

Patch coverage has no change and project coverage change: -0.20 ⚠️

Comparison is base (a4cf385) 61.93% compared to head (7d62609) 61.72%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3419      +/-   ##
==========================================
- Coverage   61.93%   61.72%   -0.20%     
==========================================
  Files         380      382       +2     
  Lines       33596    33788     +192     
==========================================
+ Hits        20804    20852      +48     
- Misses      12792    12936     +144     
Impacted Files Coverage Δ
moveit_core/utils/src/xmlrpc_casts.cpp 0.00% <0.00%> (ø)

... and 31 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@rhaschke rhaschke merged commit 8635f82 into moveit:master May 4, 2023
@rhaschke rhaschke deleted the xmlrpc-casts branch May 4, 2023 13:30
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.

2 participants