diff --git a/docs/checks/README-empty-qstringliteral.md b/docs/checks/README-empty-qstringliteral.md index bf311c9..fe8ba4e 100644 --- a/docs/checks/README-empty-qstringliteral.md +++ b/docs/checks/README-empty-qstringliteral.md @@ -1,9 +1,9 @@ # empty-qstringliteral Suggests to use an empty `QString` instead of an empty `QStringLiteral`. The later causes unneeded code bloat. You should use `QString()` instead of `QStringLiteral()` and `QStringLiteral("")`. -Note: Beware that `QString().isNull()` is `true` while both `QStringLiteral().isEmpty()` and `QStringLiteral("").isNull()` are `false`. +Note: Beware that `QString().isNull()` is `true` while both `QStringLiteral().isNull()` and `QStringLiteral("").isNull()` are `false`. So be sure not to introduce subtle bugs when doing such replacements. In most cases it's simply a matter of using `isEmpty()` instead, which is `true` for all cases above.