MoveIt is sprinkled with ...SharedPtr::reset(new ...) code and SharedPtr(new ...) constructors.
It is slightly (though in most cases not noticeably) faster and cleaner to use std::shared_ptr<...>() instead.

Anyone is welcome to help clean this up and create a pull-request along the lines of #2756 for wider parts of the code base. 💪