all repos — fluxbox @ 63f89cb4336a7232de84a6c0999833267d732175

custom fork of the fluxbox windowmanager 

Pass pre_command to CommandDialog as const reference to avoid unnecessary copying.
Mario J. Rugiero mrugiero@gmail.com
commit

63f89cb4336a7232de84a6c0999833267d732175

parent

79a358346a650a40c0e0f0409b264bf3e3cb23a4

2 files changed, 2 insertions(+), 2 deletions(-)

jump to
M src/CommandDialog.cc → src/CommandDialog.cc 
@@ -40,7 +40,7 @@ using std::less;
 using std::out_of_range;
 
 CommandDialog::CommandDialog(BScreen &screen, const string &title,
-                             const string precommand) : 
+                             const string &precommand) : 
     TextDialog(screen, title),
     m_precommand(precommand) { }
M src/CommandDialog.hh → src/CommandDialog.hh 
@@ -35,7 +35,7 @@ */
 class CommandDialog: public TextDialog {
 public:
     CommandDialog(BScreen &screen, const std::string &title,
-                  const std::string pre_command = "");
+                  const std::string &pre_command = "");
 
     /**
      * Sets the command to be executed after the command is done.