tas_input: Avoid minor copies in Read/WriteCommandButtons()

We don't need to copy the whole pair
This commit is contained in:
Lioncash 2021-12-13 11:36:48 -05:00
parent 734fb180bb
commit 54ca48e8b7

View file

@ -244,7 +244,7 @@ u64 Tas::ReadCommandButtons(const std::string& line) const {
std::string button_line; std::string button_line;
u64 buttons = 0; u64 buttons = 0;
while (std::getline(button_text, button_line, ';')) { while (std::getline(button_text, button_line, ';')) {
for (auto [text, tas_button] : text_to_tas_button) { for (const auto& [text, tas_button] : text_to_tas_button) {
if (text == button_line) { if (text == button_line) {
buttons |= static_cast<u64>(tas_button); buttons |= static_cast<u64>(tas_button);
break; break;
@ -256,7 +256,7 @@ u64 Tas::ReadCommandButtons(const std::string& line) const {
std::string Tas::WriteCommandButtons(u64 buttons) const { std::string Tas::WriteCommandButtons(u64 buttons) const {
std::string returns; std::string returns;
for (auto [text_button, tas_button] : text_to_tas_button) { for (const auto& [text_button, tas_button] : text_to_tas_button) {
if ((buttons & static_cast<u64>(tas_button)) != 0) { if ((buttons & static_cast<u64>(tas_button)) != 0) {
returns += fmt::format("{};", text_button); returns += fmt::format("{};", text_button);
} }