feat: 终端日志规范改造 · linuxdeepin/deepin-terminal@cfe8ab0

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.

The function 'slotFileChooseDialog' 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 'direction' is passed by value. It could be passed as a const reference which is usually faster and recommended in C++.

Consider using std::find_if algorithm instead of a raw loop.

The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/

Provide feedback