Thread safety of Router::FindExit()
In Simulation::UpdateRoutesAndLocations()
(which calls Pedestrian::FindRoute()
), Router::FindExit()
is called without an omp critical
construct, implying it is thread safe.
However, in Pedestrian::Relocate()
, the call to Router::FindExit()
is inside a critical construct.
Either the critical construct is unnecessary (if all routers are thread safe) or it is missing in Pedestrain::FindRoute()
.
This holds for the latest version at the time of writing (e9aeb106, I believe).