Every invocation this was copying the Mapper for no reason. Take a const
Good catch, this looks fine to me, but like the other patch, please wait for one of the other reviewers.
Is this still open? Seems good to go.
Yeah this seems fine
Same for this one, can I merge this on behalf of @lanza?
closing since the patch has been landed.