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

Adding setter and getter for WeldJointConstraint::mRelativeTransform #910

Merged
merged 1 commit into from
Aug 28, 2017

Conversation

mxgrey
Copy link
Member

@mxgrey mxgrey commented Aug 25, 2017

I couldn't find any reason to restrict access to setting mRelativeTransform. I also couldn't identify any other data members who would be impacted by changing mRelativeTransform, so the setter is just a straightforward copy.

This addresses #903.

@jslee02 jslee02 added this to the DART 6.3.0 milestone Aug 25, 2017
@codecov
Copy link

codecov bot commented Aug 25, 2017

Codecov Report

Merging #910 into release-6.3 will decrease coverage by <.01%.
The diff coverage is 0%.

@@               Coverage Diff               @@
##           release-6.3     #910      +/-   ##
===============================================
- Coverage        50.63%   50.62%   -0.01%     
===============================================
  Files              299      299              
  Lines            23411    23415       +4     
  Branches          3016     3016              
===============================================
  Hits             11854    11854              
- Misses           10226    10230       +4     
  Partials          1331     1331
Impacted Files Coverage Δ
dart/constraint/WeldJointConstraint.hpp 0% <ø> (ø) ⬆️
dart/constraint/WeldJointConstraint.cpp 0% <0%> (ø) ⬆️

@jslee02 jslee02 merged commit 7b5c6c1 into release-6.3 Aug 28, 2017
@jslee02 jslee02 deleted the grey/WeldJointConstraint-6.3 branch August 31, 2017 08:42
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