avoid copying input_events unnecessarily

clang-tidy was concerned about it so I guess we can use references.
RenechCDDA · Jul 1, 2024 · 7008778 · 7008778
1 parent 1a554c1
commit 7008778
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/src/npctalk_rules.cpp b/src/npctalk_rules.cpp
@@ -277,7 +277,7 @@ void follower_rules_ui_impl::rules_transfer_popup( bool &exporting_rules, bool &
 
 template<typename T>
 void follower_rules_ui_impl::checkbox( int rule_number, const T &this_rule,
-                                       input_event assigned_hotkey, input_event pressed_key )
+                                       input_event &assigned_hotkey, const input_event &pressed_key )
 {
     ImGui::PushID( rule_number );
     print_hotkey( assigned_hotkey );
@@ -303,7 +303,7 @@ void follower_rules_ui_impl::checkbox( int rule_number, const T &this_rule,
     ImGui::SameLine();
     auto label = _( "Default" );
     auto x = ImGui::GetWindowWidth() - ImGui::GetStyle().ScrollbarSize -
-             ImGui::GetStyle().WindowPadding.x - ImGui::CalcTextSize( label, NULL, true ).x;
+             ImGui::GetStyle().WindowPadding.x - ImGui::CalcTextSize( label, nullptr, true ).x;
     ImGui::SetCursorPosX( x );
     if( ImGui::Button( label ) ) {
         guy->rules.clear_flag( this_rule.rule );

diff --git a/src/npctalk_rules.h b/src/npctalk_rules.h
@@ -53,12 +53,12 @@ class follower_rules_ui_impl : public cataimgui::window
 
         // makes a checkbox for the rule
         template<typename T>
-        void checkbox( int rule_number, const T &this_rule, input_event assigned_hotkey,
-                       input_event pressed_key );
+        void checkbox( int rule_number, const T &this_rule, input_event &assigned_hotkey,
+                       const input_event &pressed_key );
 
         // makes one radio button per option in the map
         template<typename T>
-        void radio_group( const std::string header_id, const char *title, T *rule,
+        void radio_group( std::string header_id, const char *title, T *rule,
                           std::map<T, std::string> &values, input_event assigned_hotkey, input_event pressed_key );
 
         // Prepares for a rule option with multiple valid selections. Advances and wraps through