[clazy] [Bug 367484] reserve-candidate warnings for QVarLengthArray

Sergio Martins via KDE Bugzilla bugzilla_noreply at kde.org
Sun Aug 21 20:06:35 BST 2016


https://bugs.kde.org/show_bug.cgi?id=367484

Sergio Martins <smartins at kde.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
      Latest Commit|                            |http://commits.kde.org/claz
                   |                            |y/e879f0ec34147484c179f320d
                   |                            |fd0e77b367435de
             Status|CONFIRMED                   |RESOLVED

--- Comment #5 from Sergio Martins <smartins at kde.org> ---
Git commit e879f0ec34147484c179f320dfd0e77b367435de by Sergio Martins.
Committed on 21/08/2016 at 19:04.
Pushed by smartins into branch 'master'.

reserve-candidates: Don't warn for QVarLengthArray

Unlike the other containers QVLA doesn't need reserve in the majority
of cases since it uses the stack, resulting in false-positives.

There's still the edge case of it using the heap, but that would
take a considerable amount of effort to catch, and besides, users
using QVLA usually know what they are doing.

M  +1    -1    QtUtils.cpp
M  +11   -3    tests/reserve-candidates/main.cpp

http://commits.kde.org/clazy/e879f0ec34147484c179f320dfd0e77b367435de

-- 
You are receiving this mail because:
You are the assignee for the bug.



More information about the Unassigned-bugs mailing list