[DiscordArchive] Doesn't ```MMapManager::GetNavMeshQuery``` needs a lock when inserting into ```mmap->navMeshQueries`
[DiscordArchive] Doesn't ```MMapManager::GetNavMeshQuery``` needs a lock when inserting into ```mmap->navMeshQueries`
Archived author: robinsch • Posted: 2024-01-17T20:36:26.989000+00:00
Original source
https://github.com/TrinityCore/TrinityCo...or.cpp#L45
[Embed: TrinityCore/src/server/game/Movement/PathGenerator.cpp at 3.3.5 · T...]
TrinityCore Open Source MMO Framework (master = 10.2.0.52808, 3.3.5 = 3.3.5a.12340, wotlk_classic = 3.4.3.52237) - TrinityCore/TrinityCore
https://github.com/TrinityCore/TrinityCo...erator.cpp
Archived author: robinsch • Posted: 2024-01-17T20:38:25.202000+00:00
Original source
in case `navMeshQueries.find` fails, it will insert into `navMeshQueries`. This should be able to happen from multiple threads (instances)
Archived author: Tea • Posted: 2024-01-17T20:38:46.961000+00:00
Original source
oh i know why i got confused
Archived author: Tea • Posted: 2024-01-17T20:38:52.015000+00:00
Original source
i fixed this in master branch
Archived author: robinsch • Posted: 2024-01-17T20:38:57.393000+00:00
Original source
oh nice
Archived author: Tea • Posted: 2024-01-17T20:39:11.428000+00:00
Original source
https://github.com/TrinityCore/TrinityCo...r.cpp#L349 no insert here
[Embed: TrinityCore/src/common/Collision/Management/MMapManager.cpp at mast...]
TrinityCore Open Source MMO Framework (master = 10.2.0.52808, 3.3.5 = 3.3.5a.12340, wotlk_classic = 3.4.3.52237) - TrinityCore/TrinityCore
https://github.com/TrinityCore/TrinityCo...anager.cpp
Archived author: Tea • Posted: 2024-01-17T20:41:26.526000+00:00
Original source
it all begins with this commit https://github.com/TrinityCore/TrinityCo...0c75ac0b38
[Embed: Core/Maps: Move terrain data handling out of Map class · TrinityCor...]
Partial port of cmangos/mangos-wotlk@ff5232c64809207b5fa59d62e9870f58a36b6f3f
https://github.com/TrinityCore/TrinityCo...0c75ac0b38