Use QStringLiteral
ClosedPublic

Authored by nicolasfella on Feb 27 2020, 4:33 PM.

Details

Summary

Don't use naked strings

Test Plan

builds

Diff Detail

Repository
R155 KAccounts Integration
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
nicolasfella requested review of this revision.Feb 27 2020, 4:33 PM
nicolasfella created this revision.

Remove unintended changes

Remove unintended change

apol accepted this revision.Feb 28 2020, 9:44 AM
apol added a subscriber: apol.

if you want to have this, it could make sense to enforce it from cmake.

src/jobs/createaccount.cpp
195 ↗(On Diff #76566)

Use QLatin1Char here.

This revision is now accepted and ready to land.Feb 28 2020, 9:44 AM
This revision was automatically updated to reflect the committed changes.
nicolasfella marked an inline comment as done.Feb 28 2020, 11:02 AM