Skip to content

Commit

Permalink
Add missing fixes for PR suggestion.
Browse files Browse the repository at this point in the history
  • Loading branch information
davidlion committed Dec 20, 2024
1 parent d139bd5 commit 68c4c36
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@
#include "../../FileWriter.hpp"
#include "../../TraceableException.hpp"
#include "../Compressor.hpp"
#include "Constants.hpp"

namespace clp::streaming_compression::lzma {
/**
Expand All @@ -33,6 +34,7 @@ class Compressor : public ::clp::streaming_compression::Compressor {

// Constructors
Compressor() : Compressor{cDefaultCompressionLevel, cDefaultDictionarySize, LZMA_CHECK_CRC64} {}

Compressor(int compression_level, size_t dict_size, lzma_check check)
: m_lzma_stream{compression_level, dict_size, check} {}

Expand Down
7 changes: 1 addition & 6 deletions components/core/tests/test-StreamingCompression.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,6 @@
#include "../src/clp/streaming_compression/Compressor.hpp"
#include "../src/clp/streaming_compression/Decompressor.hpp"
#include "../src/clp/streaming_compression/lzma/Compressor.hpp"
#include "../src/clp/streaming_compression/lzma/Constants.hpp"
#include "../src/clp/streaming_compression/passthrough/Compressor.hpp"
#include "../src/clp/streaming_compression/passthrough/Decompressor.hpp"
#include "../src/clp/streaming_compression/zstd/Compressor.hpp"
Expand Down Expand Up @@ -132,11 +131,7 @@ TEST_CASE("StreamingCompression", "[StreamingCompression]") {
}

SECTION("LZMA compression") {
compressor = std::make_unique<clp::streaming_compression::lzma::Compressor>(
clp::streaming_compression::lzma::cDefaultCompressionLevel,
clp::streaming_compression::lzma::cDefaultDictionarySize,
LZMA_CHECK_CRC64
);
compressor = std::make_unique<clp::streaming_compression::lzma::Compressor>();
compress(std::move(compressor), uncompressed_buffer.data());
}

Expand Down

0 comments on commit 68c4c36

Please sign in to comment.