Bug fixes that weren't caught until I tried to merge this with libiop.
Removed the file binary_utils.hpp since for now we're going to have it live in libiop.
Separated binary field class methods into tcc (templated) and cpp (nontemplated) to fix linker errors.
Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.
[ √ ] Targeted PR against correct branch (develop)
[ √ ] Linked to Github issue with discussion and accepted design OR have an explanation in the PR that describes this work.
[ √ ] Wrote unit tests
[ √ ] Updated relevant documentation in the code
[ √ ] Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
[ √ ] Re-reviewed Files changed in the Github PR explorer
Description
Bug fixes that weren't caught until I tried to merge this with libiop.
Removed the file
binary_utils.hpp
since for now we're going to have it live in libiop.Separated binary field class methods into tcc (templated) and cpp (nontemplated) to fix linker errors.
Before we can merge this PR, please make sure that all the following items have been checked off. If any of the checklist items are not applicable, please leave them but write a little note why.
Unreleased
section inCHANGELOG.md
Files changed
in the Github PR explorer