pool entry update #350

muellerr · 2021-01-13T12:05:47+01:00

muellerr commented

2021-01-13 12:05:47 +01:00

Using pool entries with the new initializer list was still kind of awkward because the old constructor
would ignore the initializer list length. I refactored the ctor to deduce the number
of pool entries from the length of the initializer list. I think this is more intuitive.
I also provided setter/getter functions instead of expsoing pointers or other data directly.
Still need to check where I needed to change code, but I think it was mostly internal stuff (for setter/getter functions).

It might be a better idea to push this with the local pool update because of the internal API change

Using pool entries with the new initializer list was still kind of awkward because the old constructor would ignore the initializer list length. I refactored the ctor to deduce the number of pool entries from the length of the initializer list. I think this is more intuitive. I also provided setter/getter functions instead of expsoing pointers or other data directly. Still need to check where I needed to change code, but I think it was mostly internal stuff (for setter/getter functions). It might be a better idea to push this with the local pool update because of the internal API change

muellerr added the

API Change

label 2021-01-13 12:05:47 +01:00

muellerr added 1 commit 2021-01-13 12:05:48 +01:00

pool entry update eb6a851935

muellerr changed title from ~~pool entry update~~ to WIP: pool entry update

2021-01-13 12:06:40 +01:00

muellerr changed title from ~~WIP: pool entry update~~ to pool entry update

2021-01-13 15:33:10 +01:00

muellerr referenced this pull request

2021-01-13 15:34:07 +01:00

Local Pool Update #349