Skip to content

Commit

Permalink
MB-47604: Pass CompactionConfig by value and use std::move
Browse files Browse the repository at this point in the history
So that in a future patch which tweaks other parts of this ctor
clang-tidy doesn't complain about this.

Change-Id: Ide518acfe577ebe0ad4e3bf05b87a9d332f17bd7
Reviewed-on: http://review.couchbase.org/c/kv_engine/+/163193
Tested-by: Build Bot <[email protected]>
Reviewed-by: Richard de Mellow <[email protected]>
  • Loading branch information
BenHuddleston committed Oct 11, 2021
1 parent 1fbba81 commit c7ca055
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions engines/ep/src/kvstore/kvstore.h
Original file line number Diff line number Diff line change
Expand Up @@ -143,11 +143,11 @@ enum class PurgedItemType {

struct CompactionContext {
CompactionContext(Vbid vbid,
const CompactionConfig& config,
CompactionConfig config,
uint64_t purgeSeq,
std::optional<time_t> timeToExpireFrom = {})
: vbid(vbid),
compactConfig(config),
compactConfig(std::move(config)),
rollbackPurgeSeqno(purgeSeq),
timeToExpireFrom(timeToExpireFrom) {
}
Expand Down

0 comments on commit c7ca055

Please sign in to comment.