Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Replace Boost.Serialization with Cereal#158

Merged
hmenke merged 2 commits intopairinteraction:masterpairinteraction/pairinteraction:masterfrom
hmenke:cerealhmenke/pairinteraction:cerealCopy head branch name to clipboard
Jul 22, 2023
Merged

Replace Boost.Serialization with Cereal#158
hmenke merged 2 commits intopairinteraction:masterpairinteraction/pairinteraction:masterfrom
hmenke:cerealhmenke/pairinteraction:cerealCopy head branch name to clipboard

Conversation

@hmenke
Copy link
Member

@hmenke hmenke commented Jun 20, 2023

Part of #103.

This would remove the last Boost dependency that requires linking. Also since Cereal is header-only we would work around the problem with _GLIBCXX_DEBUG ABI incompatibilities.

TODO:

  • Check that we can use the cereal from the system package manager.
  • Clean up the serialization code (maybe using if constexpr).
  • Check whether we can use load_and_construct to avoid having to define default constructors.
  • Investigate how to serialize containers nicely, so that actual numbers show up in the JSON and not just base64 encoded binary.

@hmenke hmenke force-pushed the cereal branch 5 times, most recently from 7e73170 to 106ef38 Compare June 21, 2023 09:26
@hmenke hmenke force-pushed the cereal branch 3 times, most recently from 089b68d to ac4acd7 Compare July 6, 2023 09:25
@hmenke hmenke merged commit edd850e into pairinteraction:master Jul 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.