Applying clang-tidy suggested fixes.

This commit is contained in:
Ralf W. Grosse-Kunstleve 2021-06-22 12:07:25 -07:00 committed by Ralf W. Grosse-Kunstleve
parent adfd3e1700
commit b1d9f0d28f

View File

@ -44,14 +44,14 @@ struct SftSharedPtrStash {
std::vector<std::shared_ptr<Sft>> stash; std::vector<std::shared_ptr<Sft>> stash;
explicit SftSharedPtrStash(int ser_no) : ser_no{ser_no} {} explicit SftSharedPtrStash(int ser_no) : ser_no{ser_no} {}
void Add(const std::shared_ptr<Sft> &obj) { void Add(const std::shared_ptr<Sft> &obj) {
if (obj->history.size()) { if (!obj->history.empty()) {
obj->history += "_Stash" + std::to_string(ser_no) + "Add"; obj->history += "_Stash" + std::to_string(ser_no) + "Add";
} }
stash.push_back(obj); stash.push_back(obj);
} }
void AddSharedFromThis(Sft *obj) { void AddSharedFromThis(Sft *obj) {
auto sft = obj->shared_from_this(); auto sft = obj->shared_from_this();
if (sft->history.size()) { if (!sft->history.empty()) {
sft->history += "_Stash" + std::to_string(ser_no) + "AddSharedFromThis"; sft->history += "_Stash" + std::to_string(ser_no) + "AddSharedFromThis";
} }
stash.push_back(sft); stash.push_back(sft);
@ -74,7 +74,7 @@ struct SftTrampoline : Sft, py::trampoline_self_life_support {
long pass_shared_ptr(const std::shared_ptr<Sft> &obj) { long pass_shared_ptr(const std::shared_ptr<Sft> &obj) {
auto sft = obj->shared_from_this(); auto sft = obj->shared_from_this();
if (sft->history.size()) { if (!sft->history.empty()) {
sft->history += "_PassSharedPtr"; sft->history += "_PassSharedPtr";
} }
return sft.use_count(); return sft.use_count();