diff options
author | Tom Cherry <tomcherry@google.com> | 2020-06-25 15:02:35 +0000 |
---|---|---|
committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | 2020-06-25 15:02:35 +0000 |
commit | 3f1381fbcd170f40f22d8b38e735d795e4a755fa (patch) | |
tree | e0f97b79d0d9fbfe771b1d67813b84a0a6e6cadb /logd/SerializedLogChunk.cpp | |
parent | b3a10e52201e3602bf4c4a4120cace63e0bb2304 (diff) | |
parent | c58d1e4aec4d59b112fa1cc2aa99ed4eb0a25010 (diff) |
Merge changes I9638e90b,Ib2636dfc am: c58d1e4aec
Original change: https://android-review.googlesource.com/c/platform/system/core/+/1349344
Change-Id: I54b5873749ec46f6489d6e6f8224759239b0b25e
Diffstat (limited to 'logd/SerializedLogChunk.cpp')
-rw-r--r-- | logd/SerializedLogChunk.cpp | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/logd/SerializedLogChunk.cpp b/logd/SerializedLogChunk.cpp index 2516003c14..e444856a40 100644 --- a/logd/SerializedLogChunk.cpp +++ b/logd/SerializedLogChunk.cpp @@ -25,14 +25,13 @@ SerializedLogChunk::~SerializedLogChunk() { } void SerializedLogChunk::Compress() { - if (compressed_log_.empty()) { - CompressionEngine::GetInstance().Compress({contents_.data(), write_offset_}, - compressed_log_); + if (compressed_log_.size() == 0) { + CompressionEngine::GetInstance().Compress(contents_, write_offset_, compressed_log_); LOG(INFO) << "Compressed Log, buffer max size: " << contents_.size() << " size used: " << write_offset_ << " compressed size: " << compressed_log_.size(); } - contents_.resize(0); + contents_.Resize(0); } // TODO: Develop a better reference counting strategy to guard against the case where the writer is @@ -41,7 +40,8 @@ void SerializedLogChunk::IncReaderRefCount() { if (++reader_ref_count_ != 1 || writer_active_) { return; } - CompressionEngine::GetInstance().Decompress(compressed_log_, contents_, write_offset_); + contents_.Resize(write_offset_); + CompressionEngine::GetInstance().Decompress(compressed_log_, contents_); } void SerializedLogChunk::DecReaderRefCount(bool compress) { @@ -90,7 +90,7 @@ bool SerializedLogChunk::ClearUidLogs(uid_t uid, log_id_t log_id, LogStatistics* // Clear the old compressed logs and set write_offset_ appropriately for DecReaderRefCount() // to compress the new partially cleared log. if (new_write_offset != write_offset_) { - compressed_log_.clear(); + compressed_log_.Resize(0); write_offset_ = new_write_offset; } |