faramir118/mangos — comment on commit 53ed5f371e887039902547ea680a1375219232e4
File:  line 0
By sixsixnine:

I just mean not exposing ACE locks to outside world; its just ugly and creates unnecessary dependencies.
I'm not sure what would be the best way to do it.
Proper is to have the lock itself inside dtNavMesh;so it will lock itself when needed, but changing detour code is bad idea for us (or maybe not ... making it thread-safe wrapper class exposing only functions we need).
Maybe just leaving it as is is good enough... not sure.
Need to discuss it a little bit ...

About RSA's implementation, I wonder why implicit unlocking (using scopes) is used on shared locks, this is one ugly style.