Skip to content

Commit

Permalink
Rename "archive" into "persist"
Browse files Browse the repository at this point in the history
  • Loading branch information
alex-sparus committed May 23, 2024
1 parent eca43c1 commit c6802f2
Show file tree
Hide file tree
Showing 52 changed files with 1,874 additions and 1,994 deletions.
2 changes: 1 addition & 1 deletion CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ option(ENABLE_GUILE "enable building guile module" off)
option(ENABLE_BOOST_COROUTINE "run benchmarks with boost coroutine" off)

option(immer_BUILD_TESTS "Build tests" ON)
option(immer_BUILD_ARCHIVE_TESTS "Build experimental archive tests" off)
option(immer_BUILD_PERSIST_TESTS "Build experimental persist tests" off)
option(immer_BUILD_EXAMPLES "Build examples" ON)
option(immer_BUILD_DOCS "Build docs" ON)
option(immer_BUILD_EXTRAS "Build extras" ON)
Expand Down
6 changes: 3 additions & 3 deletions extra/fuzzer/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ endif()

file(GLOB_RECURSE immer_fuzzers "*.cpp")
foreach(TMP_PATH ${immer_fuzzers})
string(FIND ${TMP_PATH} archive EXCLUDE_DIR_FOUND)
string(FIND ${TMP_PATH} persist EXCLUDE_DIR_FOUND)
if(NOT ${EXCLUDE_DIR_FOUND} EQUAL -1)
list(REMOVE_ITEM immer_fuzzers ${TMP_PATH})
endif()
Expand All @@ -36,6 +36,6 @@ foreach(_file IN LISTS immer_fuzzers)
endif()
endforeach()

if(immer_BUILD_ARCHIVE_TESTS)
add_subdirectory(archive)
if(immer_BUILD_PERSIST_TESTS)
add_subdirectory(persist)
endif()
File renamed without changes.
Original file line number Diff line number Diff line change
Expand Up @@ -12,8 +12,8 @@
#include <immer/flex_vector.hpp>

#include <fmt/ranges.h>
#include <immer/extra/archive/rbts/load.hpp>
#include <immer/extra/archive/rbts/save.hpp>
#include <immer/extra/persist/rbts/load.hpp>
#include <immer/extra/persist/rbts/save.hpp>

#include <array>

Expand All @@ -38,13 +38,12 @@ extern "C" int LLVMFuzzerTestOneInput(const std::uint8_t* data,
using size_t = std::uint8_t;

const auto check_save_and_load = [&](const auto& vec) {
auto ar = immer::archive::rbts::make_save_archive_for(vec);
auto vector_id = immer::archive::container_id{};
std::tie(ar, vector_id) =
immer::archive::rbts::save_to_archive(vec, ar);
auto ar = immer::persist::rbts::make_input_pool_for(vec);
auto vector_id = immer::persist::container_id{};
std::tie(ar, vector_id) = immer::persist::rbts::add_to_pool(vec, ar);

auto loader =
immer::archive::rbts::make_loader_for(vec, to_load_archive(ar));
immer::persist::rbts::make_loader_for(vec, to_output_pool(ar));
auto loaded = loader.load(vector_id);
require_eq(vec, loaded);
};
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
#include <fuzzer/fuzzer_input.hpp>

#include <immer/extra/archive/champ/champ.hpp>
#include <immer/extra/persist/champ/champ.hpp>

#include <array>
#include <bitset>
Expand Down Expand Up @@ -77,21 +77,20 @@ extern "C" int LLVMFuzzerTestOneInput(const std::uint8_t* data,
case op_check: {
const auto bitset = std::bitset<var_count>(read<size_t>(in));
auto ids =
std::vector<std::pair<std::size_t, immer::archive::node_id>>{};
auto archive =
immer::archive::champ::container_archive_save<set_t>{};
std::vector<std::pair<std::size_t, immer::persist::node_id>>{};
auto archive = immer::persist::champ::container_input_pool<set_t>{};
for (auto index = std::size_t{}; index < var_count; ++index) {
if (bitset[index]) {
auto set_id = immer::archive::node_id{};
auto set_id = immer::persist::node_id{};
std::tie(archive, set_id) =
immer::archive::champ::save_to_archive(vars[index],
archive);
immer::persist::champ::add_to_pool(vars[index],
archive);
ids.emplace_back(index, set_id);
}
}

auto loader = immer::archive::champ::container_loader<set_t>{
to_load_archive(archive)};
auto loader = immer::persist::champ::container_loader<set_t>{
to_output_pool(archive)};

for (const auto& [index, set_id] : ids) {
const auto& set = vars[index];
Expand Down
12 changes: 6 additions & 6 deletions flake.nix
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@

arximboldi-cereal = pkgs.callPackage ./nix/cereal.nix {inherit arximboldi-cereal-src;};

archive-inputs = with pkgs; [
persist-inputs = with pkgs; [
spdlog
arximboldi-cereal
xxHash
Expand Down Expand Up @@ -79,8 +79,8 @@
checkPhase = ''
ctest -D ExperimentalMemCheck
valgrind --quiet --error-exitcode=99 --leak-check=full --errors-for-leak-kinds=all \
--suppressions=./test/extra/archive/valgrind.supp \
./test/extra/archive/archive-tests
--suppressions=./test/extra/persist/valgrind.supp \
./test/extra/persist/persist-tests
'';
});
};
Expand All @@ -103,7 +103,7 @@
fzf
starship
]
++ archive-inputs;
++ persist-inputs;

shellHook =
self.checks.${system}.pre-commit-check.shellHook
Expand Down Expand Up @@ -147,7 +147,7 @@

unit-tests = (withLLVM self.packages.${system}.immer).overrideAttrs (prev: {
name = "immer-unit-tests";
buildInputs = with pkgs; [catch2_3 boehmgc boost fmt] ++ archive-inputs;
buildInputs = with pkgs; [catch2_3 boehmgc boost fmt] ++ persist-inputs;
nativeBuildInputs = with pkgs; [cmake ninja];
dontBuild = false;
doCheck = true;
Expand All @@ -159,7 +159,7 @@
cmakeFlags = [
"-DCMAKE_BUILD_TYPE=Debug"
"-Dimmer_BUILD_TESTS=ON"
"-Dimmer_BUILD_ARCHIVE_TESTS=ON"
"-Dimmer_BUILD_PERSIST_TESTS=ON"
"-Dimmer_BUILD_EXAMPLES=OFF"
"-DCXX_STANDARD=20"
];
Expand Down
54 changes: 0 additions & 54 deletions immer/extra/archive/errors.hpp

This file was deleted.

Loading

0 comments on commit c6802f2

Please sign in to comment.