Change 14163

ShadauxCat (ShadauxCat)
ShadauxCat committed this change into //guest/ShadauxCat/Sprawl/Mainline under Review 14164
View Review
Download .zip
-Renamed HashMap functions to follow coding style.
Only begin, end, find, and variants are left lowercase, in keeping with C++ algorithm and range-based for support.
-Fixed some accounting issues with list and forwardlist; size wasn't properly being maintained.
-Made a small pedantic change to ThreadManager to ensure that m_numThreadsSynced got reset to 0 before the NotifyAll() to eliminate the miniscule potential for deadlock it would cause if it happened after another thread had already woken up.

#review-14164
  • Files 8
  • Comments 0
8 edited 0 added 0 deleted
UnitTests/UnitTests_HashMap.cpp#8
Loading...
UnitTests/UnitTests_List.cpp#7
Loading...
collections/ForwardList.hpp#7
Loading...
collections/List.hpp#5
Loading...
collections/hashmap/HashMap_Variadic.hpp#6
Loading...
serialization/JSONSerializer.cpp#3
Loading...
serialization/JSONSerializer.hpp#2
Loading...
threading/threadmanager.cpp#3
Loading...
Tip: Use n and p to cycle through the changes.