this code is running in MainThrd so there isn’t any Multithread Competition
Maybe the input method will trigger this function StartChangingText more than one time before “FinishChangingText”
I’d like to do more test and give more information I can get to close this bug.
Thanks for the info. We’ve had this issue open for a while but have never been able to get a solid repro for it. I’ll try out your repro and see how I get on.
The assert itself has been removed for 4.12 so you’ll no longer get the crash when using the IME, but I can’t say whether any weird behaviour will occur (this function handles starting an undo transaction, so the second call while there’s already a transaction active will just be ignored).