Hi - initial looks at this shows that it works - although I’d like to have some of my folks take a closer look.
I think my plan going forward will be to get this MR approved and merged first - since we definitely use the by-reference methods. Once that is done, I’ll do a followup to change some of the signatures that pass a pointer, because while we don’t actively use them, I would like to have them available (since we have had them that way in our fork for years).
I’ll comment back when I have an update.