Skip to content

Commit b140375

Browse files
committed
ttl/ttl_test.cc: prefer prefix ++operator for non-primitive types
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
1 parent 43c789c commit b140375

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

utilities/ttl/ttl_test.cc

+4-4
Original file line numberDiff line numberDiff line change
@@ -120,7 +120,7 @@ class TtlTest {
120120
static FlushOptions flush_opts;
121121
WriteBatch batch;
122122
kv_it_ = kvmap_.begin();
123-
for (int i = 0; i < num_ops && kv_it_ != kvmap_.end(); i++, kv_it_++) {
123+
for (int i = 0; i < num_ops && kv_it_ != kvmap_.end(); i++, ++kv_it_) {
124124
switch (batch_ops[i]) {
125125
case PUT:
126126
batch.Put(kv_it_->first, kv_it_->second);
@@ -145,7 +145,7 @@ class TtlTest {
145145
static FlushOptions flush_opts;
146146
kv_it_ = kvmap_.begin();
147147
advance(kv_it_, start_pos_map);
148-
for (int i = 0; kv_it_ != kvmap_.end() && i < num_entries; i++, kv_it_++) {
148+
for (int i = 0; kv_it_ != kvmap_.end() && i < num_entries; i++, ++kv_it_) {
149149
ASSERT_OK(cf == nullptr
150150
? db_ttl_->Put(wopts, kv_it_->first, kv_it_->second)
151151
: db_ttl_->Put(wopts, cf, kv_it_->first, kv_it_->second));
@@ -207,7 +207,7 @@ class TtlTest {
207207
kv_it_ = kvmap_.begin();
208208
advance(kv_it_, st_pos);
209209
std::string v;
210-
for (int i = 0; kv_it_ != kvmap_.end() && i < span; i++, kv_it_++) {
210+
for (int i = 0; kv_it_ != kvmap_.end() && i < span; i++, ++kv_it_) {
211211
Status s = (cf == nullptr) ? db_ttl_->Get(ropts, kv_it_->first, &v)
212212
: db_ttl_->Get(ropts, cf, kv_it_->first, &v);
213213
if (s.ok() != check) {
@@ -252,7 +252,7 @@ class TtlTest {
252252
} else { // dbiter should have found out kvmap_[st_pos]
253253
for (int i = st_pos;
254254
kv_it_ != kvmap_.end() && i < st_pos + span;
255-
i++, kv_it_++) {
255+
i++, ++kv_it_) {
256256
ASSERT_TRUE(dbiter->Valid());
257257
ASSERT_EQ(dbiter->value().compare(kv_it_->second), 0);
258258
dbiter->Next();

0 commit comments

Comments
 (0)