Parameter 'oldCmdName' is passed by value. It could be passed as a const reference which is usually faster and recommended in C++.

Parameter 'fontType' is passed by value. It could be passed as a const reference which is usually faster and recommended in C++.

Parameter 'args' is passed by value. It could be passed as a const reference which is usually faster and recommended in C++.

Condition 'runningCount!=0' is always true

The function 'handleQuitAction' is never used.

Parameter 'name' is passed by value. It could be passed as a const reference which is usually faster and recommended in C++.

Parameter 'deleteKey' is passed by value. It could be passed as a const reference which is usually faster and recommended in C++.

The scope of the variable 'isFocus' can be reduced. Warning: Be careful when fixing this message, especially when there are inner loops. Here is an example where cppcheck will write that the scope for 'i' can be reduced:\012void f(int x)\012{\012 int i = 0;\012 if (x) {\012 // it's safe to move 'int i = 0;' here\012 for (int n = 0; n < 10; ++n) {\012 // it is possible but not safe to move 'int i = 0;' here\012 do_something(&i);\012 }\012 }\012}\012When you see this message it is always safe to reduce the variable scope 1 level.

Variable 'isFocus' is assigned a value that is never used.