Skip to content

Commit 902f3af

Browse files
Nyeriahclaude
andauthored
fix(WG): bind SkipClasses config to a local before Tokenize (#170)
Acore::Tokenize returns string_views into its argument and deletes its rvalue overloads, so passing GetOption's temporary std::string directly fails to compile. Store it in a named local so the views stay valid. Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
1 parent ad08dd7 commit 902f3af

1 file changed

Lines changed: 2 additions & 1 deletion

File tree

src/CFBG.cpp

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,8 @@ void CFBG::LoadConfig()
102102
_IsEnableWGReapplyOnResurrect = sConfigMgr->GetOption<bool>("CFBG.Battlefield.ReapplyOnResurrect.Enable", true);
103103

104104
_wgSkipClasses.clear();
105-
for (auto const& token : Acore::Tokenize(sConfigMgr->GetOption<std::string>("CFBG.Battlefield.SkipClasses", ""), ',', false))
105+
std::string const skipClasses = sConfigMgr->GetOption<std::string>("CFBG.Battlefield.SkipClasses", "");
106+
for (auto const& token : Acore::Tokenize(skipClasses, ',', false))
106107
{
107108
if (Optional<uint8> playerClass = Acore::StringTo<uint8>(token))
108109
_wgSkipClasses.insert(*playerClass);

0 commit comments

Comments
 (0)