If identity pairs (i.e. old ID pointer being same as new one) was forbidden, then this should be asserted against in code defining remapping, not in code applying it. But it is actually sometimes usefull to allow/use identity pairs, so simply early-return on these instead of asserting.