Pass "const QString &varName" instead of "QString varName" throughout the codebase
In a bunch of places, especially constructors, we are passing the whole object instead of the reference. This is against KDE's coding practices and should be fixed.
In a bunch of places, especially constructors, we are passing the whole object instead of the reference. This is against KDE's coding practices and should be fixed.