This isn't quite correct - getsvalue() is just a 'get', i.e. it doesn't copy the data so it will potentially get clobbered by the next line... I suggest doing:
```c++
MCExecPoint t_temp_ep;
t_temp_ep . setsvalue(t_other_ep . getsvalue());
t_temp_ep . nativetoutf16();
```
Then use t_temp_ep in the next comparison. This way, 't_other_ep' still contains the native version for the return (if roundtrip succeeds).