Fix 'performance-unnecessary-value-param' in Logger
Fix 'performance-unnecessary-value-param' in Logger

Most Logger's shift operators call .write() which "mutates" passed
object except three operators which simply use QString overload. This
produces clang-tidy warning about unnecessary copy. Use direct call to
.write() to suppress the warning and have consistent function signatures

Change-Id: I1c42e16948147315e40edf263e887ebeee4a4388
Reviewed-by: Christian Kandeler <3aa99dcdd3f0cac61fb81c2a11771c0cc2497607@qt.io>

qt-labs/qbs,qt-labs/qbs,qt-labs/qbs,qtproject/qt-labs-qbs,qt-labs/qbs,qt-labs/qbs,qtproject/qt-labs-qbs,qtproject/qt-labs-qbs,qtproject/qt-labs-qbs,qt-labs/qbs,qtproject/qt-labs-qbs,qt-labs/qbs,qtproject/qt-labs-qbs,qtproject/qt-labs-qbs