diff options
author | Sergio Martins <smartins@kde.org> | 2018-12-03 19:00:19 +0000 |
---|---|---|
committer | Sergio Martins <smartins@kde.org> | 2018-12-03 19:00:19 +0000 |
commit | f34772d583b1f6f6e9efc89a5c3998d661fd34a4 (patch) | |
tree | 886cadb7419d0e3a63411c1e44962c19d7ae19ff /docs/checks | |
parent | ddfb80b7de992ebd77e9635bafba95ba5d7900f9 (diff) | |
parent | ad8e58e41537a9656b7924b5447fdb6e7bbad43e (diff) |
Merge branch '1.4'
Diffstat (limited to 'docs/checks')
-rw-r--r-- | docs/checks/README-inefficient-qlist-soft.md | 2 | ||||
-rw-r--r-- | docs/checks/README-inefficient-qlist.md | 5 |
2 files changed, 2 insertions, 5 deletions
diff --git a/docs/checks/README-inefficient-qlist-soft.md b/docs/checks/README-inefficient-qlist-soft.md index 1795c3a7..beebb3d1 100644 --- a/docs/checks/README-inefficient-qlist-soft.md +++ b/docs/checks/README-inefficient-qlist-soft.md @@ -2,6 +2,6 @@ A less aggressive version of the **inefficient-qlist** check. -Finds `QList<T>` where `sizeof(T) > 64`. `QVector<T>` should be used instead. +Finds `QList<T>` where `sizeof(T) > sizeof(void*)`. `QVector<T>` should be used instead. Only warns if the container is a local variable and isn't passed to any method or returned, unlike **inefficient-qlist**. This makes it easier to fix the warnings without concern about source and binary compatibility. diff --git a/docs/checks/README-inefficient-qlist.md b/docs/checks/README-inefficient-qlist.md index fc7dbf76..3bfe3f1c 100644 --- a/docs/checks/README-inefficient-qlist.md +++ b/docs/checks/README-inefficient-qlist.md @@ -1,8 +1,5 @@ # inefficient-qlist -Finds `QList<T>` where `sizeof(T) > 64`. `QVector<T>` should be used instead. - -This check uses 64 as the limit instead of `sizeof(void*)` so that it gives the same -results in 32-bit platforms. Making your 32-bit code portable to 64-bit without pessimizations. +Finds `QList<T>` where `sizeof(T) > sizeof(void*)`. `QVector<T>` should be used instead. This is a very noisy check and hence disabled by default. See **inefficient-qlist-soft** for a more useful check. |