From 574477866fe3e82a87214848d3d4b505535f2d3b Mon Sep 17 00:00:00 2001 From: Filoppi Date: Tue, 11 May 2021 16:30:29 +0300 Subject: InputCommon: fix serialization of control expression with line breaks The control expression editor allows line breaks, but the serialization was losing anything after the first line break (/r /n). Instead of opting to encode them and decode them on serialization (which I tried but was not safe, as it would lose /n written in the string by users), I opted to replace them with a space. --- .../Core/InputCommon/ControllerEmu/ControlGroup/ControlGroup.cpp | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) (limited to 'Source/Core/InputCommon/ControllerEmu/ControlGroup/ControlGroup.cpp') diff --git a/Source/Core/InputCommon/ControllerEmu/ControlGroup/ControlGroup.cpp b/Source/Core/InputCommon/ControllerEmu/ControlGroup/ControlGroup.cpp index 84c7238e48..2862445392 100644 --- a/Source/Core/InputCommon/ControllerEmu/ControlGroup/ControlGroup.cpp +++ b/Source/Core/InputCommon/ControllerEmu/ControlGroup/ControlGroup.cpp @@ -117,7 +117,10 @@ void ControlGroup::SaveConfig(IniFile::Section* sec, const std::string& defdev, for (auto& c : controls) { // control expression - sec->Set(group + c->name, c->control_ref->GetExpression(), ""); + std::string expression = c->control_ref->GetExpression(); + // We can't save line breaks in a single line config. Restoring them is too complicated. + ReplaceBreaksWithSpaces(expression); + sec->Set(group + c->name, expression, ""); // range sec->Set(group + c->name + "/Range", c->control_ref->range * 100.0, 100.0); @@ -135,7 +138,9 @@ void ControlGroup::SaveConfig(IniFile::Section* sec, const std::string& defdev, } else { - sec->Set(base + name, ext->GetSelectionSetting().GetInputReference().GetExpression(), "None"); + std::string expression = ext->GetSelectionSetting().GetInputReference().GetExpression(); + ReplaceBreaksWithSpaces(expression); + sec->Set(base + name, expression, "None"); } for (auto& ai : ext->GetAttachmentList()) -- cgit v1.2.3