cleanups of _SafeReadRepAccessor - mostly taking ptr instead of reference to match _SafeReadWriteRepAccessor