Some minor changes from code review
This commit is contained in:
parent
37b7666262
commit
51204163df
|
@ -462,6 +462,7 @@ TEST_CASE("fdbclient/Tuple/makeTuple") {
|
|||
ASSERT(t1.getType(5) == Tuple::UTF8);
|
||||
ASSERT(t1.getType(6) == Tuple::NULL_TYPE);
|
||||
ASSERT(t1.getType(7) == Tuple::VERSIONSTAMP);
|
||||
ASSERT(t1.size() == 8);
|
||||
|
||||
return Void();
|
||||
}
|
||||
}
|
||||
|
|
|
@ -30,7 +30,7 @@
|
|||
struct Tuple {
|
||||
struct UnicodeStr {
|
||||
StringRef str;
|
||||
UnicodeStr(StringRef str) : str(str) {}
|
||||
explicit UnicodeStr(StringRef str) : str(str) {}
|
||||
};
|
||||
|
||||
Tuple() {}
|
||||
|
@ -97,7 +97,7 @@ struct Tuple {
|
|||
|
||||
// Use a fold expression to append each argument using the << operator.
|
||||
// https://en.cppreference.com/w/cpp/language/fold
|
||||
(t << ... << args);
|
||||
(t << ... << std::forward<Types>(args));
|
||||
|
||||
return t;
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue