Address more simple Coverity defects

This commit is contained in:
Evil Eye 2025-08-14 11:19:13 +02:00
parent cb0316e6be
commit 9db9df95e2
4 changed files with 10 additions and 10 deletions

View File

@ -106,7 +106,7 @@ namespace MWGui
newSpell.mSelected = (MWBase::Environment::get().getWindowManager()->getSelectedSpell() == spell->mId);
newSpell.mActive = true;
newSpell.mCount = 1;
mSpells.push_back(newSpell);
mSpells.push_back(std::move(newSpell));
}
MWWorld::InventoryStore& invStore = mActor.getClass().getInventoryStore(mActor);

View File

@ -218,10 +218,10 @@ namespace MWLua
};
topicEntryBindingsClass["id"] = sol::readonly_property(
[](const MWDialogue::Entry& topicEntry) { return topicEntry.mInfoId.serializeText(); });
topicEntryBindingsClass["text"]
= sol::readonly_property([](const MWDialogue::Entry& topicEntry) { return topicEntry.mText; });
topicEntryBindingsClass["actor"]
= sol::readonly_property([](const MWDialogue::Entry& topicEntry) { return topicEntry.mActorName; });
topicEntryBindingsClass["text"] = sol::readonly_property(
[](const MWDialogue::Entry& topicEntry) -> std::string_view { return topicEntry.mText; });
topicEntryBindingsClass["actor"] = sol::readonly_property(
[](const MWDialogue::Entry& topicEntry) -> std::string_view { return topicEntry.mActorName; });
}
void addJournalClassJournalEntriesListBindings(sol::state_view& lua, const MWBase::Journal* journal)
@ -255,7 +255,7 @@ namespace MWLua
journalEntryBindingsClass["id"] = sol::readonly_property(
[](const MWDialogue::StampedJournalEntry& journalEntry) { return journalEntry.mInfoId.serializeText(); });
journalEntryBindingsClass["text"] = sol::readonly_property(
[](const MWDialogue::StampedJournalEntry& journalEntry) { return journalEntry.mText; });
[](const MWDialogue::StampedJournalEntry& journalEntry) -> std::string_view { return journalEntry.mText; });
journalEntryBindingsClass["questId"] = sol::readonly_property(
[](const MWDialogue::StampedJournalEntry& journalEntry) { return journalEntry.mTopic.serializeText(); });
journalEntryBindingsClass["day"] = sol::readonly_property(

View File

@ -164,7 +164,7 @@ namespace MWLua
else
return LuaUtil::toLuaIndex(index);
};
layersTable["insertAfter"] = [context](std::string afterName, std::string_view name, const sol::object& opt) {
layersTable["insertAfter"] = [context](std::string afterName, std::string name, const sol::object& opt) {
LuaUi::Layer::Options options;
options.mInteractive = LuaUtil::getValueOrDefault(LuaUtil::getFieldOrNil(opt, "interactive"), true);
context.mLuaManager->addAction(
@ -177,11 +177,11 @@ namespace MWLua
},
"Insert after UI layer");
};
layersTable["insertBefore"] = [context](std::string beforeName, std::string_view name, const sol::object& opt) {
layersTable["insertBefore"] = [context](std::string beforeName, std::string name, const sol::object& opt) {
LuaUi::Layer::Options options;
options.mInteractive = LuaUtil::getValueOrDefault(LuaUtil::getFieldOrNil(opt, "interactive"), true);
context.mLuaManager->addAction(
[=]() {
[beforeName = std::move(beforeName), name = std::move(name), options]() {
size_t index = LuaUi::Layer::indexOf(beforeName);
if (index == LuaUi::Layer::count())
throw std::logic_error(

View File

@ -786,7 +786,7 @@ namespace MWWorld
state.mSounds.push_back(sound);
}
mMagicBolts.push_back(state);
mMagicBolts.push_back(std::move(state));
return true;
}