apache / datasketches-cpp

Core C++ Sketch Library
https://datasketches.apache.org
Apache License 2.0
223 stars 71 forks source link

use allocator properly in varopt union #340

Closed jmalkin closed 1 year ago

jmalkin commented 1 year ago

Tried to use a newer C++ standard and discovered varopt used deprecated allocator methods. Those were relatively small, but I also discovered that the union didn't use the allocator properly at all.

coveralls commented 1 year ago

Pull Request Test Coverage Report for Build 4030236638


Totals Coverage Status
Change from base Build 4029599866: 0.002%
Covered Lines: 2329
Relevant Lines: 2481

💛 - Coveralls