4 Months Ago
Update: Further constraining Pool - Step 3 Now that all usecases of Pool's API have been redirected to new methods, I can further constrain and annotate the API. - Further constrained all container-like Free overloads to only accept ref-types with default ctor and IPooled interface - Marked FreeList<T> as obsolete (with errors) and piped it into FreeUnmanaged overload (this is dangerous/leaky, but I've updated all existing calls to Free/FreeList) - Marked ClearList as obsolete (with errors) - also inlined it's logic at the only callsite in the project - Removed Clear call from GetList, since now we always clear lists via overloads - Removed checks from main Free overload Tests: built all targets. No runtime tests. Ran around on CraggyIsland in 2 player multiplayer, did minor spawning of items, constructions, shooting, player damage.