Incompatible list iterator comparison in RecastNavigation system.
At least on Windows, debug builds crash with the following stack trace due to a debug assertion failure:
https://media.discordapp.net/attachments/355192295055687681/521470760108294149/unknown.png
The responsible line is https://gitlab.com/OpenMW/openmw/blob/master/components/detournavigator/navmeshtilescache.hpp#L72
My understanding of the issue is that the move assignment operator is being called when the target is still in the default state, and its iterator is being compared against that of the source operand, which has been initialised properly. This means that a default valued iterator is being comapred against one from a real list, and that menas the iterators don't belong ot the same list and therefore comparison is illegal. This might be an implementation detail unique to MSVC, or I'd have expected Coverity to have picked up on it.
I'd assign the issue to elsid, but I don't know what his GitLab username is (if he even has one). Just in case it's the same, I'm going to try and ping him, but there's a chance that that'll tag some random stranger. @elsid