Implptr Get function

[nkoenig]

🎉 New feature

Summary

Adds a get function to the ImplPtr class so that you can do things like:

class Implementation
{
  public: void Run();
  public: std::thread runThread;
};

class MainClass
{
  public: void Load()
  {
    this->dataPtr->runThread = std::thread(&Implementation::Run, this->dataPtr.Get());
  }
}; 

Test it

Run the tests.

Checklist

• Signed all commits for DCO
• Added tests
• Added example and/or tutorial
• Updated documentation (as needed)
• Updated migration guide (as needed)
• codecheck passed (See contributing)
• All tests passed (See test coverage)
• While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge

[codecov-commenter]

Codecov Report

Merging #27 (6419a1b) into ign-utils1 (e9c439b) will increase coverage by 4.41%.
The diff coverage is 100.00%.

@@              Coverage Diff               @@
##           ign-utils1      #27      +/-   ##
==============================================
+ Coverage       89.58%   94.00%   +4.41%     
==============================================
  Files               3        3              
  Lines              48       50       +2     
==============================================
+ Hits               43       47       +4     
+ Misses              5        3       -2     

Impacted Files	Coverage Δ
include/ignition/utils/ImplPtr.hh	100.00% <ø> (ø)
include/ignition/utils/detail/ImplPtr.hh	91.17% <100.00%> (+6.80%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e9c439b...6419a1b. Read the comment docs.

[chapulina]

Can this target ign-utils1?

[nkoenig]

Can this target ign-utils1?

Done

[azeey]

One minor comment regarding what seems to be duplicate tests. Otherwise, LGTM!

[osrf-triage]

This pull request has been mentioned on Gazebo Community. There might be relevant details there:

https://community.gazebosim.org/t/new-ignition-releases-2022-03-01-citadel-edifice-fortress/1313/1