Skip to content

Commit a45c532

Browse files
committed
chore: fixes
Signed-off-by: Roman Gershman <roman@dragonflydb.io>
1 parent 2e637fc commit a45c532

File tree

2 files changed

+4
-6
lines changed

2 files changed

+4
-6
lines changed

src/common/backed_args.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ class BackedArguments {
3030
}
3131

3232
bool operator==(const iterator& other) const {
33-
return index_ == other.index_;
33+
return index_ == other.index_ && ba_ == other.ba_;
3434
}
3535

3636
bool operator!=(const iterator& other) const {

src/facade/memcache_parser.cc

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -342,12 +342,10 @@ auto MP::Parse(string_view str, uint32_t* consumed, Command* cmd) -> Result {
342342
tmp_args_.push_back(tokens_view[0]);
343343
tokens_view.remove_prefix(1);
344344
auto res = ParseStore(tokens_view, cmd);
345-
if (res != MP::OK) {
346-
return res;
345+
if (res == MP::OK) {
346+
MaterializeArgs(absl::MakeSpan(tmp_args_), cmd);
347347
}
348-
MaterializeArgs(absl::MakeSpan(tmp_args_), cmd);
349-
350-
return MP::OK;
348+
return res;
351349
}
352350

353351
if (cmd->type >= META_SET) {

0 commit comments

Comments
 (0)