Parcourir la source

controller: Use std::move within ConvertToFrontendParameters()

Avoids unnecessary copies.
Lioncash il y a 5 ans
Parent
commit
215cfbb757
1 fichiers modifiés avec 3 ajouts et 3 suppressions
  1. 3 3
      src/core/hle/service/am/applets/controller.cpp

+ 3 - 3
src/core/hle/service/am/applets/controller.cpp

@@ -29,14 +29,14 @@ static Core::Frontend::ControllerParameters ConvertToFrontendParameters(
     npad_style_set.raw = private_arg.style_set;
     npad_style_set.raw = private_arg.style_set;
 
 
     return {
     return {
-        .min_players = std::max(s8(1), header.player_count_min),
+        .min_players = std::max(s8{1}, header.player_count_min),
         .max_players = header.player_count_max,
         .max_players = header.player_count_max,
         .keep_controllers_connected = header.enable_take_over_connection,
         .keep_controllers_connected = header.enable_take_over_connection,
         .enable_single_mode = header.enable_single_mode,
         .enable_single_mode = header.enable_single_mode,
         .enable_border_color = header.enable_identification_color,
         .enable_border_color = header.enable_identification_color,
-        .border_colors = identification_colors,
+        .border_colors = std::move(identification_colors),
         .enable_explain_text = enable_text,
         .enable_explain_text = enable_text,
-        .explain_text = text,
+        .explain_text = std::move(text),
         .allow_pro_controller = npad_style_set.pro_controller == 1,
         .allow_pro_controller = npad_style_set.pro_controller == 1,
         .allow_handheld = npad_style_set.handheld == 1,
         .allow_handheld = npad_style_set.handheld == 1,
         .allow_dual_joycons = npad_style_set.joycon_dual == 1,
         .allow_dual_joycons = npad_style_set.joycon_dual == 1,